juju-quickstart

Merge lp://qastaging/~marcoceppi/juju-quickstart/recommend-juju into lp://qastaging/juju-quickstart/packaging

  1. recommend-juju
  2. Merge into packaging-trunk
Proposed by Marco Ceppi
Status: Merged
Merged at revision: 44
Proposed branch: lp://qastaging/~marcoceppi/juju-quickstart/recommend-juju
Merge into: lp://qastaging/juju-quickstart/packaging
Diff against target: 25 lines (+7/-0)
2 files modified
debian/changelog (+6/-0)
debian/control (+1/-0)
To merge this branch: bzr merge lp://qastaging/~marcoceppi/juju-quickstart/recommend-juju
Reviewer Review Type Date Requested Status
Fabrice Matrat (community) Approve
Jay R. Wren (community) Approve
Review via email: mp+283745@code.qastaging.launchpad.net

Description of the change

Juju Quickstart should recommend Juju

To post a comment you must log in.
Revision history for this message
Jay R. Wren (evarlast) wrote :

LGTM

review: Approve
Revision history for this message
Fabrice Matrat (fabricematrat) wrote :

LGTM

review: Approve
Revision history for this message
Antonio Rosales (arosales) wrote :

Given the instructions at:
https://jujucharms.com/get-started

state to first install python-software-properties should we also make this a recommends? This would enable folks to just type in:

sudo apt-get install juju-quickstart (given they have already added the ppa)

-thanks,
Antonio

Revision history for this message
Antonio Rosales (arosales) wrote :

Comment from Jay that answers my earlier comment:

"there is a catch-22, or dependency cycle there which would mean adding the recommends is a no-op as the python-software-properties from recommends must have already been installed for the add-apt-repository to be used for juju-quickstart to be installed and its dependencies and recommends considered."

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

to all changes: