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

Synchronize control files: part 1 #15

Merged
merged 4 commits into from
Jan 13, 2023
Merged

Conversation

scpeters
Copy link
Contributor

Part of #14.

The ubuntu/debian/control file is currently unused in this repository; typically it is symbolically linked to one or all of the distro-specific folders, but focal and jammy both have their own control files. The files have diverged from each other, with much of the ignition->gz renaming not having been applied to the focal and jammy control files. This synchronizes the ign->gz changes to the control files in e157789 and synchronizes the dependencies used by each package in d15093f.

Part of #14.

Signed-off-by: Steve Peters <[email protected]>
* jammy:  libgz-physics6-bullet-dev adds depend on
          libgz-common5-geospatial-dev
* ubuntu: libgz-physics6-core-dev adds depend on
          libgz-math7-eigen3-dev

Signed-off-by: Steve Peters <[email protected]>
@j-rivero
Copy link
Contributor

j-rivero commented Dec 22, 2022

Changes does not look bad to me. Usually we really want to preserve at least one symlink to the ubuntu/debian/control file, probably the latest distribution, in this case jammy, so the next ones tries to use the common file.

I don't see the ignition transitional packages in the current control files so we probably need to import them from jammy.

@scpeters
Copy link
Contributor Author

I don't see the ignition transitional packages in the current control files so we probably need to import them from jammy.

I see the transitional packages at the bottom of all the control files:

@j-rivero
Copy link
Contributor

j-rivero commented Jan 9, 2023

I see the transitional packages at the bottom of all the control files:

* https://github.com/gazebo-release/gz-physics6-release/blob/main/focal/debian/control#L264

* https://github.com/gazebo-release/gz-physics6-release/blob/main/jammy/debian/control#L265

* https://github.com/gazebo-release/gz-physics6-release/blob/main/ubuntu/debian/control#L290

arg, sorry I was looking in the wrong place.

@j-rivero j-rivero merged commit 4dadc68 into main Jan 13, 2023
@j-rivero j-rivero deleted the scpeters/sync_control_part1 branch January 13, 2023 15:54
scpeters added a commit that referenced this pull request Jan 25, 2023
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.

2 participants