Merge lp:~racb/juju-release-tools/lxc-template-dependency into lp:~juju-qa/juju-release-tools/packaging-juju-core-default

Proposed by Robie Basak on 2015-01-08
Status: Merged
Merged at revision: 69
Proposed branch: lp:~racb/juju-release-tools/lxc-template-dependency
Merge into: lp:~juju-qa/juju-release-tools/packaging-juju-core-default
Diff against target: 13 lines (+2/-1)
1 file modified
debian/control (+2/-1)
To merge this branch: bzr merge lp:~racb/juju-release-tools/lxc-template-dependency
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code 2015-01-08 Approve on 2015-01-08
Review via email: mp+245888@code.launchpad.net
To post a comment you must log in.
Curtis Hovey (sinzui) wrote :

Thank you. I think this fix addresses part of the problem reported in bug 1407699

review: Approve (code)
Robie Basak (racb) wrote :

On Thu, Jan 08, 2015 at 05:46:30PM -0000, Curtis Hovey wrote:
> Review: Approve code
>
> Thank you. I think this fix addresses part of the problem reported in
> bug 1407699

Ah. I only considered Trusty. For Precise, the dependency should be
"cloud-image-utils | cloud-utils" as Scott says. Do you want to modify
that as you pull this in?

I just uploaded cloud-image-utils only to Vivid, but I can bring it up
to sync with this branch on next upload - no reason to diverge as it'll
work fine either way.

Curtis Hovey (sinzui) wrote :

I am making the change now. Glad we discussed this.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2014-10-06 15:15:17 +0000
3+++ debian/control 2015-01-08 17:41:37 +0000
4@@ -41,7 +41,8 @@
5
6 Package: juju-local
7 Architecture: all
8-Depends: cpu-checker,
9+Depends: cloud-image-utils,
10+ cpu-checker,
11 dbus,
12 juju-core (>= ${source:Version}),
13 lxc (>=1.0.0~alpha1-0ubuntu14),

Subscribers

People subscribed via source and target branches