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

net-misc/asterisk: prune vulnerable. #26919

Closed
wants to merge 5 commits into from

Conversation

jkroonza
Copy link
Contributor

Along with no longer used files/ files.

Bug: https://bugs.gentoo.org/838391
Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon jaco@uls.co.za

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @jkroonza
Areas affected: ebuilds
Packages affected: net-misc/asterisk

net-misc/asterisk: @jkroonza, @gentoo/proxy-maint

Linked bugs

Bugs linked: 838391


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Aug 18, 2022
@jkroonza jkroonza force-pushed the asterisk-vuln-remove branch 2 times, most recently from e56f94b to 376f07a Compare August 19, 2022 09:53
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-19 10:01 UTC
Newest commit scanned: 376f07a
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/bf06d5ee76/output.html

@ajakk
Copy link
Member

ajakk commented Aug 19, 2022

Could you split this up into one commit per package?

@jkroonza
Copy link
Contributor Author

Could you split this up into one commit per package?

I could, but the g729 and opus cannot stand on their own regardless. Should I also split the different asterisk slots into their own prunes? Ie, total 5 commits?

@ajakk
Copy link
Member

ajakk commented Aug 20, 2022

Sure, if you think that's necessary

net-misc/asterisk:13 is going away due to security concerns.

Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon <jaco@uls.co.za>
net-misc/asterisk:13 is going away due to security.

Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon <jaco@uls.co.za>
Bug: https://bugs.gentoo.org/838391
Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon <jaco@uls.co.za>
Bug: https://bugs.gentoo.org/838391
Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon <jaco@uls.co.za>
Bug: https://bugs.gentoo.org/838391
Package-Manager: Portage-3.0.30, Repoman-3.0.3
Signed-off-by: Jaco Kroon <jaco@uls.co.za>
@jkroonza
Copy link
Contributor Author

Sure, if you think that's necessary

Necessary? No. But by the logic of the packages being split into separate remove commits it's also better to handle the different slots in different commits. Done.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-23 08:41 UTC
Newest commit scanned: af4f762
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/5a6cc5cb90/output.html

@jkroonza jkroonza deleted the asterisk-vuln-remove branch August 27, 2022 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
5 participants