Merge lp:~bac/juju-gui/spritegen-trivial into lp:juju-gui/experimental
Proposed by
Brad Crittenden
Status: | Merged |
---|---|
Merged at revision: | 225 |
Proposed branch: | lp:~bac/juju-gui/spritegen-trivial |
Merge into: | lp:juju-gui/experimental |
Diff against target: |
13 lines (+2/-1) 1 file modified
Makefile (+2/-1) |
To merge this branch: | bzr merge lp:~bac/juju-gui/spritegen-trivial |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+133063@code.launchpad.net |
Description of the change
Fix Makefile error wrt spritegen
If app/assets/sprite exists then regenerating sprites failed since the
logic assumed the directory would not be there. Remove the directory
before copying the contents of the newly generated sprite data.
Self-reviewed.
To post a comment you must log in.
Reviewers: mp+133063_ code.launchpad. net,
Message:
Please take a look.
Description:
Fix Makefile error wrt spritegen
If app/assets/sprite exists then regenerating sprites failed since the
logic assumed the directory would not be there. Remove the directory
before copying the contents of the newly generated sprite data.
https:/ /code.launchpad .net/~bac/ juju-gui/ spritegen- trivial/ +merge/ 133063
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/6814092/
Affected files:
M Makefile
A [revision details]
Index: Makefile
=== modified file 'Makefile'
--- Makefile 2012-10-31 10:57:16 +0000
+++ Makefile 2012-11-06 13:16:34 +0000
@@ -28,7 +28,8 @@
$(SPRITE_ GENERATED_ FILES): node_modules/grunt node-spriteshee t $(SPRITE_ SOURCE_ FILES) modules/ grunt/bin/ grunt spritegen
node_modules/
@node_
- @mv bin/sprite app/assets/sprite/
+ @rm -Rf app/assets/sprite/
+ @mv bin/sprite app/assets
$(NODE_TARGETS): package.json
@npm install
Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision: <email address hidden>