-
Notifications
You must be signed in to change notification settings - Fork 9
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
release.py fails for ignition packages #1102
Comments
Fix #1102 Signed-off-by: Jose Luis Rivero <[email protected]>
This is still failing for me. I tried to release ign-sensors6 and this is what I got $ release.py -r 1 --dry-run ign-sensors6 6.8.0 testtoken
Downloading releasing info for ign-sensors6
Linux distributions in the -release repository:
+ focal ('amd64', 'armhf', 'arm64')
+ jammy ('amd64', 'armhf', 'arm64')
Safety checks:
+ OK No underscore in package name
+ OK Package names in changelog and control
+ OK Package versions in changelog
+ OK Package release versions in changelog
+ OK sdformat version in proper sdformat package
!! Automatic calculation of source_repo_uri failed.
* git remote origin is: https://github.com/gazebosim/gz-sensors
* Package name generated org/repo: gazebosim/ignition-sensors
>> Please use --source-repo-uri parameter |
This is an expected effect of the failure of the safety measure to be sure what is the source repository desired by crossing the results of:
Since both methods don't match the script refuse to continue and suggests:
So:
should make it. To improve the user experience I can tweak the script to deal with these |
After releasing ign-gazebo6 using PR #1109 I'm going to proceed to rename the ign- debbuilder to use the new gz- names. |
Renamed jobs: rendering6 and ign-gazebo6 were already renamed.
|
Should be now ready. |
Tested with ign-math6:
Running the same command with a version of
release.py
before #1014 succeeds. For references, the working version is in cf72fde.cc @iche033.
The text was updated successfully, but these errors were encountered: