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

Proposed by Robie Basak
Status: Merged
Merged at revision: 69
Proposed branch: lp://staging/~racb/juju-release-tools/lxc-template-dependency
Merge into: lp://staging/~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://staging/~racb/juju-release-tools/lxc-template-dependency
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+245888@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

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

review: Approve (code)
Revision history for this message
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.

Revision history for this message
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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches