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

Proposed by Matt Bruzek
Status: Merged
Merged at revision: 130
Proposed branch: lp:~mbruzek/juju-release-tools/packaging-juju2-default
Merge into: lp:~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:~mbruzek/juju-release-tools/packaging-juju2-default
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Review via email: mp+299614@code.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
1=== modified file 'debian/rules'
2--- debian/rules 2016-06-30 22:04:52 +0000
3+++ debian/rules 2016-07-09 14:33:11 +0000
4@@ -56,6 +56,7 @@
5 dh_auto_install
6
7 override_dh_link:
8+ dh_link -pjuju-2.0 usr/share/bash-completion/completions/juju-2.0 usr/share/bash-completion/completions/juju
9 dh_link -pjuju-2.0 usr/lib/juju-$(VERSION)/bin/juju usr/bin/juju
10 dh_link -pjuju-2.0 usr/lib/juju-$(VERSION)/bin/jujud usr/bin/jujud
11 dh_link -pjuju-2.0 usr/lib/juju-$(VERSION)/bin/juju-metadata usr/bin/juju-metadata

Subscribers

People subscribed via source and target branches