Merge lp://staging/~mbruzek/juju-release-tools/packaging-juju2-default into lp://staging/~juju-qa/juju-release-tools/packaging-juju2-default

Proposed by Matt Bruzek
Status: Merged
Merged at revision: 130
Proposed branch: lp://staging/~mbruzek/juju-release-tools/packaging-juju2-default
Merge into: lp://staging/~juju-qa/juju-release-tools/packaging-juju2-default
Diff against target: 11 lines (+1/-0)
1 file modified
debian/rules (+1/-0)
To merge this branch: bzr merge lp://staging/~mbruzek/juju-release-tools/packaging-juju2-default
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Review via email: mp+299614@code.staging.launchpad.net

Description of the change

This is the change that was discussed with Martin and Nicholas. To symbolically link juju to the juju-2.0 file. Please evaluate this as a solution to https://bugs.launchpad.net/juju-core/+bug/1600257

However there were alternate solutions discussed, to work with juju1 packaging, such as renaming the source file juju-2.0 to juju if I remember correctly.

If this solution is determined to be less than optimal, go ahead and close it. I wanted to present my proposal against the right branch so we could evaluate the pros and cons of this approach.

To post a comment you must log in.
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Ugh, sorry Matt. So much going on with the release, I had forgotten about this.

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Thanks again!

review: Approve

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