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.

Sign up for GitHub

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

Jump to bottom

Update python-numpy key. #31552

Merged
merged 1 commit into from
Dec 23, 2021
Merged

Update python-numpy key. #31552

merged 1 commit into from
Dec 23, 2021

Conversation

nuclearsandwich
Copy link
Member

  • Remove out-of-support Ubuntu distributions.
  • This package is no longer available in Alpine Edge.
  • This package is no longer available in Debian as of Bullseye.
  • This package is no longer available in Ubuntu as of Jammy.

@nuclearsandwich
Update python-numpy key.
196145f 
* Remove out-of-support Ubuntu distributions.
* This package is no longer available in Alpine Edge.
* This package is no longer available in Debian as of Bullseye.
* This package is no longer available in Ubuntu as of Jammy.
@nuclearsandwich nuclearsandwich added the rosdep Issue/PR is for a rosdep key label Dec 22, 2021
@nuclearsandwich nuclearsandwich self-assigned this Dec 22, 2021
@nuclearsandwich nuclearsandwich requested a review from a team as a code owner December 22, 2021 19:51
@nuclearsandwich nuclearsandwich merged commit 579dfab into master Dec 23, 2021
@nuclearsandwich nuclearsandwich deleted the nuclearsandwich/python-numpy branch December 23, 2021 04:51
@smits
Copy link
Contributor

smits commented Dec 23, 2021

FYI, this broke a lot of eol distro users builds. Since rosdep has the --include-eol-distros flag it makes sense to keep the eol entries.

This was referenced Dec 23, 2021
Merged
Closed
tkmtnt7000 added a commit to tkmtnt7000/rosdistro that referenced this pull request Dec 23, 2021
@tkmtnt7000
Revert "Update python-numpy key. (ros#31552)"
40ea31c 
This reverts commit 579dfab.
@tkmtnt7000 tkmtnt7000 mentioned this pull request Dec 23, 2021
Closed
@knorth55
Copy link
Contributor

i dont see the reason why you removed all eol keys in this PR, just for supporting jammy.

@knorth55 knorth55 mentioned this pull request Dec 23, 2021
Closed
@tkmtnt7000 tkmtnt7000 mentioned this pull request Dec 23, 2021
Closed
@knorth55 knorth55 mentioned this pull request Dec 23, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cottsay cottsay cottsay approved these changes

Assignees

@nuclearsandwich nuclearsandwich

Labels
rosdep Issue/PR is for a rosdep key
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nuclearsandwich @smits @knorth55 @cottsay