Juju Charms Collection
glance-simplestreams-sync package

Code review comment for lp://qastaging/~bjornt/charms/xenial/glance-simplestreams-sync/no-python2

  1. Xenial Xerus (16.04)
  2. no-python2
  3. Merge into landscape
Revision history for this message
Björn Tillenius (bjornt) wrote :

> Why not just add python into the DEPS array, as it stands right now we'll
> attempt installation of python for every dep (wasteful)
>
> That said, +1

It's bash, I don't want to change it :) I copied this from the keystone charm. I'd rather be consistent.

To be clear, python is not installed for every dependency. If you look closer, 'python' isn't explicitly installed, only 'python-$dep'.

« Back to merge proposal