Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1783)

Side by Side Diff: Makefile

Issue 10264047: Remove unnecessary HTTP requests from npm install.
Patch Set: Remove unnecessary HTTP requests from npm install. Created 10 years, 9 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | [revision details] » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Makefile debugging hack: uncomment the two lines below and make will tell you 1 # Makefile debugging hack: uncomment the two lines below and make will tell you
2 # more about what is happening. The output generated is of the form 2 # more about what is happening. The output generated is of the form
3 # "FILE:LINE [TARGET (DEPENDENCIES) (NEWER)]" where DEPENDENCIES are all the 3 # "FILE:LINE [TARGET (DEPENDENCIES) (NEWER)]" where DEPENDENCIES are all the
4 # things TARGET depends on and NEWER are all the files that are newer than 4 # things TARGET depends on and NEWER are all the files that are newer than
5 # TARGET. DEPENDENCIES will be colored green and NEWER will be blue. 5 # TARGET. DEPENDENCIES will be colored green and NEWER will be blue.
6 # 6 #
7 #OLD_SHELL := $(SHELL) 7 #OLD_SHELL := $(SHELL)
8 #SHELL = $(warning [$@ ($^) ($?) ])$(OLD_SHELL) 8 #SHELL = $(warning [$@ ($^) ($?) ])$(OLD_SHELL)
9 9
10 # Build a target for JavaScript files. The find command excludes directories 10 # Build a target for JavaScript files. The find command excludes directories
(...skipping 190 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 $(SPRITE_SOURCE_FILES) 201 $(SPRITE_SOURCE_FILES)
202 node_modules/grunt/bin/grunt spritegen 202 node_modules/grunt/bin/grunt spritegen
203 203
204 $(NON_SPRITE_IMAGES): 204 $(NON_SPRITE_IMAGES):
205 mkdir -p build-shared/juju-ui/assets/images 205 mkdir -p build-shared/juju-ui/assets/images
206 cp -r app/assets/images/non-sprites/* build-shared/juju-ui/assets/images / 206 cp -r app/assets/images/non-sprites/* build-shared/juju-ui/assets/images /
207 207
208 install-npm-packages: $(NODE_TARGETS) 208 install-npm-packages: $(NODE_TARGETS)
209 209
210 $(NODE_TARGETS): package.json 210 $(NODE_TARGETS): package.json
211 » npm install 211 » npm install --cache-min=999999999
212 # Keep all targets up to date, not just new/changed ones. 212 # Keep all targets up to date, not just new/changed ones.
213 for dirname in $(NODE_TARGETS); do touch $$dirname ; done 213 for dirname in $(NODE_TARGETS); do touch $$dirname ; done
214 @# Check to see if we made what we expected to make, and warn if we did 214 @# Check to see if we made what we expected to make, and warn if we did
215 @# not. Note that we calculate FOUND_TARGETS here, in this way and not 215 @# not. Note that we calculate FOUND_TARGETS here, in this way and not
216 @# in the standard Makefile way, because we need to see what 216 @# in the standard Makefile way, because we need to see what
217 @# node_modules were created by this target. Makefile variables and 217 @# node_modules were created by this target. Makefile variables and
218 @# substitutions, even when using $(eval...) within a target, happen 218 @# substitutions, even when using $(eval...) within a target, happen
219 @# initially, before the target is run. Therefore, if this were a 219 @# initially, before the target is run. Therefore, if this were a
220 @# simple Makefile variable, it would be empty after a first run, and 220 @# simple Makefile variable, it would be empty after a first run, and
221 @# you would always see the warning message in that case. We have to 221 @# you would always see the warning message in that case. We have to
(...skipping 377 matching lines...) Expand 10 before | Expand all | Expand 10 after
599 599
600 # targets are alphabetically sorted, they like it that way :-) 600 # targets are alphabetically sorted, they like it that way :-)
601 .PHONY: beautify build build-files build-devel clean clean-all clean-deps \ 601 .PHONY: beautify build build-files build-devel clean clean-all clean-deps \
602 clean-docs code-doc debug devel docs dist gjslint help \ 602 clean-docs code-doc debug devel docs dist gjslint help \
603 install-npm-packages jshint lint lint-yuidoc main-doc npm-cache \ 603 install-npm-packages jshint lint lint-yuidoc main-doc npm-cache \
604 npm-cache-file prep prod recess server spritegen test test-debug \ 604 npm-cache-file prep prod recess server spritegen test test-debug \
605 test-misc test-prep test-prod undocumented view-code-doc view-docs \ 605 test-misc test-prep test-prod undocumented view-code-doc view-docs \
606 view-main-doc 606 view-main-doc
607 607
608 .DEFAULT_GOAL := all 608 .DEFAULT_GOAL := all
OLDNEW
« no previous file with comments | « no previous file | [revision details] » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b