Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a deprecation/status message to the README #649

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ajkavanagh
Copy link
Collaborator

This is to effectively disuade people to use it and get them to look elsewhere for writing new charms (i.e. point them to the Operator Framework). Also add details of which track to use for which version of charms.

This is to effectively disuade people to use it and get them to look
elsewhere for writing new charms (i.e. point them to the Operator
Framework).  Also add details of which track to use for which version of
charms.
Copy link
Collaborator

@fnordahl fnordahl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a welcome addition to the guidance to use other tools for new charms that was added to the documentation when we created 3.x ref 679a9f0.

The text does include a couple of things that needs fixing though, see in-line.

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated
versions, the tool has been split into two tracks:

1. For charms targetting Ubuntu focal (20.04) or earlier: 2.x
2. For charms targetting Ubuntu jammy (22.04) or later: 3.x
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a gross oversimplification of the problem at hand, and I do not think this will help guide people into making good decisions.

Why not instead cover the fact that charm-tools 2.x is hard coded to build using Python 3.6, which is equivalent of using a Ubuntu 18.04 (Bionic) base, and that charm-tools 3.x uses the Python from your build environment?

The Focal setuptools problem as revealed by the upstream Jinja2 issue is of course worth noting, and patches are of course welcome to actually fix the root of the issue as outlined in #646.

These correct the bases section, add a binary wheels section and
re-drafts the reasoning behind the 2.x/3.x split for snap tracks.
@ajkavanagh ajkavanagh requested a review from fnordahl June 13, 2023 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants