Merge lp:~frankban/charms/precise/juju-gui/remove-npm into lp:~juju-gui/charms/precise/juju-gui/trunk
Proposed by
Francesco Banconi
Status: | Merged |
---|---|
Merged at revision: | 169 |
Proposed branch: | lp:~frankban/charms/precise/juju-gui/remove-npm |
Merge into: | lp:~juju-gui/charms/precise/juju-gui/trunk |
Diff against target: |
30 lines (+4/-3) 2 files modified
hooks/utils.py (+3/-2) revision (+1/-1) |
To merge this branch: | bzr merge lp:~frankban/charms/precise/juju-gui/remove-npm |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
charmers | Pending | ||
Review via email: mp+208842@code.launchpad.net |
Description of the change
Fix GUI charm broken by npm update.
To post a comment you must log in.
Reviewers: mp+208842_ code.launchpad. net,
Message:
Please take a look.
Description:
Fix GUI charm broken by npm update.
https:/ /code.launchpad .net/~frankban/ charms/ precise/ juju-gui/ remove- npm/+merge/ 208842
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/67990044/
Affected files (+6, -3 lines):
A [revision details]
M hooks/utils.py
M revision
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>
Index: revision
=== modified file 'revision'
--- revision 2014-02-21 10:24:19 +0000
+++ revision 2014-02-28 13:07:33 +0000
@@ -1,1 +1,1 @@
-106
+107
Index: hooks/utils.py
=== modified file 'hooks/utils.py'
--- hooks/utils.py 2014-02-04 20:07:31 +0000
+++ hooks/utils.py 2014-02-28 15:33:29 +0000
@@ -114,7 +114,7 @@
JUJU_PEM = 'juju.includes- private- key.pem' DEPENDENCIES = (
DEB_BUILD_
- 'bzr', 'g++', 'git', 'imagemagick', 'make', 'nodejs', 'npm',
+ 'bzr', 'g++', 'git', 'imagemagick', 'make', 'nodejs',
)
@@ -628,8 +628,9 @@ mkstemp( prefix= 'make-distfile- ', dir=logdir)
fd, name = tempfile.
log('Output from "make distfile" sent to %s' % name)
+ # Passing HOME is required by node during npm packages installation. source_ dir, 'distfile', 'BRANCH_ IS_GOOD= true', }'.format( os.path. expanduser( '~')), stdout=fd, stderr=fd)
run('make', '-C',
juju_gui_
- stdout=fd, stderr=fd)
+ 'HOME={
return first_path_in_dir(
os.path. join(juju_ gui_source_ dir, 'releases'))