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

Restricting 20.04 to python3. #34

Closed
nuclearsandwich opened this issue Nov 26, 2019 · 1 comment · Fixed by #35
Closed

Restricting 20.04 to python3. #34

nuclearsandwich opened this issue Nov 26, 2019 · 1 comment · Fixed by #35

Comments

@nuclearsandwich
Copy link
Contributor

For the upcoming 20.04-based ROS releases we are trying to keep Python 2 packages out of our bootstrap repository. Currently this script releases packages into all suites specified in the Suite option for each package.
There is some support in the --include parameter for a separate Suite3 option which is only used on python3 targets but that support does not currently extend to the regular release scripts here and stdeb itself does not currently support this config clause and so does not generate Suite3-aware .changes files.

One workaround for that problem is to immediately use the --include option to push python3 packages into Suite3 targets after making a new release. For example:

## in stdeb.cfg
# Suite: xenial bionic
# Suite3: focal

# Publish a new package to all suites in `Suite` and pypi.
ros_release_python --upload

# Now include it in suites only in Suite3
ros_release_python --include --upload deb3

This works but it will be very slow due to the added publishing cycle.
It also means that if Suite3 support is ever added to stdeb we may not notice when our older suites that are currently python2 and python3 stop receiving python3 updates because they are not present in Suite3. We could add all suites to Suite3 which would slow the script down further, although because the snapshot and publish is only run once per include target it would only make a bad problem worse rather than making a bad problem excruciating.

@dirk-thomas
Copy link
Member

The following repos have been updated to specify Suite3:

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 a pull request may close this issue.

2 participants