Merge lp:~bac/charms/precise/juju-gui/constraint-parsing into lp:~juju-gui/charms/precise/juju-gui/trunk

Proposed by Brad Crittenden
Status: Merged
Merged at revision: 155
Proposed branch: lp:~bac/charms/precise/juju-gui/constraint-parsing
Merge into: lp:~juju-gui/charms/precise/juju-gui/trunk
Diff against target: 26 lines (+2/-2)
2 files modified
Makefile (+1/-1)
server-requirements.pip (+1/-1)
To merge this branch: bzr merge lp:~bac/charms/precise/juju-gui/constraint-parsing
Reviewer Review Type Date Requested Status
charmers Pending
Review via email: mp+202677@code.launchpad.net

Description of the change

Bump juju-deployer to pre-release fix.

New version of juju-deployer handles constraints with size specifiers like
'4G'.

https://codereview.appspot.com/49720044/

To post a comment you must log in.
Revision history for this message
Brad Crittenden (bac) wrote :

Reviewers: mp+202677_code.launchpad.net,

Message:
Please take a look.

Description:
Bump juju-deployer to pre-release fix.

New version of juju-deployer handles constraints with size specifiers
like
'4G'.

https://code.launchpad.net/~bac/charms/precise/juju-gui/constraint-parsing/+merge/202677

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/49720044/

Affected files (+2, -0 lines):
   M Makefile
   A [revision details]
   D deps/juju-deployer-0.2.9.tar.gz
   A deps/juju-deployer-0.3.1-prerelease.tar.gz
   M server-requirements.pip

Index: Makefile
=== modified file 'Makefile'
--- Makefile 2014-01-16 20:32:55 +0000
+++ Makefile 2014-01-21 15:08:17 +0000
@@ -28,7 +28,7 @@
  setup:
   tests/00-setup
   # Ensure the correct version of pip has been installed
- tests/.venv/bin/pip --version | grep "pip 1.4" || exit 1
+ tests/.venv/bin/pip --version | grep "pip 1.[4-9]" || exit 1

  sysdeps:
   sudo apt-get install --yes $(SYSDEPS)

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: server-requirements.pip
=== modified file 'server-requirements.pip'
--- server-requirements.pip 2013-11-13 18:01:03 +0000
+++ server-requirements.pip 2014-01-22 14:19:45 +0000
@@ -25,4 +25,4 @@
  tornado==3.1.1
  websocket-client==0.12.0
  jujuclient==0.15
-juju-deployer==0.2.9
+juju-deployer==0.3.1-prerelease

Index: deps/juju-deployer-0.2.9.tar.gz
=== removed file 'deps/juju-deployer-0.2.9.tar.gz'
Binary files deps/juju-deployer-0.2.9.tar.gz 2013-11-13 16:36:45 +0000 and
deps/juju-deployer-0.2.9.tar.gz 1970-01-01 00:00:00 +0000 differ

Index: deps/juju-deployer-0.3.1-prerelease.tar.gz
=== added file 'deps/juju-deployer-0.3.1-prerelease.tar.gz'
Binary files deps/juju-deployer-0.3.1-prerelease.tar.gz 1970-01-01 00:00:00
+0000 and deps/juju-deployer-0.3.1-prerelease.tar.gz 2014-01-22 14:19:45
+0000 differ

Revision history for this message
Francesco Banconi (frankban) wrote :
Revision history for this message
Brad Crittenden (bac) wrote :

*** Submitted:

Bump juju-deployer to pre-release fix.

New version of juju-deployer handles constraints with size specifiers
like
'4G'.

R=frankban
CC=
https://codereview.appspot.com/49720044

https://codereview.appspot.com/49720044/

Revision history for this message
Brad Crittenden (bac) wrote :
Revision history for this message
Francesco Banconi (frankban) wrote :
Revision history for this message
Francesco Banconi (frankban) wrote :
Revision history for this message
Brad Crittenden (bac) wrote :

*** Submitted:

Bump juju-deployer to pre-release fix.

New version of juju-deployer handles constraints with size specifiers
like
'4G'.

R=frankban
CC=
https://codereview.appspot.com/49720044

https://codereview.appspot.com/49720044/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Makefile'
2--- Makefile 2014-01-16 20:32:55 +0000
3+++ Makefile 2014-01-22 14:23:27 +0000
4@@ -28,7 +28,7 @@
5 setup:
6 tests/00-setup
7 # Ensure the correct version of pip has been installed
8- tests/.venv/bin/pip --version | grep "pip 1.4" || exit 1
9+ tests/.venv/bin/pip --version | grep "pip 1.[4-9]" || exit 1
10
11 sysdeps:
12 sudo apt-get install --yes $(SYSDEPS)
13
14=== removed file 'deps/juju-deployer-0.2.9.tar.gz'
15Binary files deps/juju-deployer-0.2.9.tar.gz 2013-11-13 16:36:45 +0000 and deps/juju-deployer-0.2.9.tar.gz 1970-01-01 00:00:00 +0000 differ
16=== added file 'deps/juju-deployer-0.3.1-prerelease.tar.gz'
17Binary files deps/juju-deployer-0.3.1-prerelease.tar.gz 1970-01-01 00:00:00 +0000 and deps/juju-deployer-0.3.1-prerelease.tar.gz 2014-01-22 14:23:27 +0000 differ
18=== modified file 'server-requirements.pip'
19--- server-requirements.pip 2013-11-13 18:01:03 +0000
20+++ server-requirements.pip 2014-01-22 14:23:27 +0000
21@@ -25,4 +25,4 @@
22 tornado==3.1.1
23 websocket-client==0.12.0
24 jujuclient==0.15
25-juju-deployer==0.2.9
26+juju-deployer==0.3.1-prerelease

Subscribers

People subscribed via source and target branches