juju / charm-tools

Tools for charm authors and maintainers
Other
42 stars 64 forks source link

charm build is broken in bionic when installed from the apt repositories #402

Open pengale opened 6 years ago

pengale commented 6 years ago

To reproduce this issue, install charm-tools from the apt repositories on a 18.04/Bionic box, and attempt to run charm build.

It will fail while running pip3 with the following message:

    no such option: -d

A workaround is to install charm tools from a snap. In environments where this is not possible, however, the tools are simply broken.

marcoceppi commented 6 years ago

The deb should have been pulled from the archive before 18.04 - it's out of date and no longer maintained.

On Thu, May 24, 2018, 12:56 Pete Vander Giessen notifications@github.com wrote:

To reproduce this issue, install charm-tools from the apt repositories on a 18.04/Bionic box, and attempt to run charm build.

It will fail while running pip3 with the following message:

no such option: -d

A workaround is to install charm tools from a snap. In environments where this is not possible, however, the tools are simply broken.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/juju/charm-tools/issues/402, or mute the thread https://github.com/notifications/unsubscribe-auth/AAET1Vj2YUapqmUzguPxzp9Ii0BPu77Oks5t1wJ3gaJpZM4UMzUo .

johnsca commented 6 years ago

@petevg I don't actually see it in apt search charm on bionic. Is it possible it was brought in by a PPA? I think it was in the Juju PPA at one point; maybe we need to remove it from that?

@marcoceppi We've had complaints about it being broken in xenial as well, assuming pip gets updated. I'm not sure what the best solution for this is, but it seems like just saying "it's out of date and no longer supported" isn't great.

marcoceppi commented 6 years ago

We should pull the packages and have the new package snap install.

On Thu, May 24, 2018, 15:49 Cory Johns notifications@github.com wrote:

@petevg https://github.com/petevg I don't actually see it in apt search charm on bionic. Is it possible it was brought in by a PPA? I think it was in the Juju PPA at one point; maybe we need to remove it from that?

@marcoceppi https://github.com/marcoceppi We've had complaints about it being broken in xenial as well, assuming pip gets updated. I'm not sure what the best solution for this is, but it seems like just saying "it's out of date and no longer supported" isn't great.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/juju/charm-tools/issues/402#issuecomment-391876316, or mute the thread https://github.com/notifications/unsubscribe-auth/AAET1bZmDAuCwpEzIlUa-xzQ2C-zbN5Bks5t1ysCgaJpZM4UMzUo .

pengale commented 6 years ago

It's possible that the person who reported the bug to me was pulling from a ppa.

We still have docs telling people to apt install charm-tools, apparently. I like the idea of fixing the package by having it do a snap install.