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-analyzer/wireshark: add 4.0.14, 4.2.4, drop 4.1.0 #34685

Closed
wants to merge 1 commit into from

Conversation

csfore
Copy link
Contributor

@csfore csfore commented Jan 7, 2024

  • 4.0.14 security bump, also add 4.2.4
  • New USE flag for 4.2.x, http3
  • Unkeyword 4.2.4 to match net-libs/http3
  • Drop 4.1.0 since it was a dev release

Bug: https://bugs.gentoo.org/921528

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @csfore
Areas affected: ebuilds
Packages affected: net-analyzer/wireshark

net-analyzer/wireshark: @thesamesam

Linked bugs

Bugs linked: 921528


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 assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. security PR that needs to be merged promptly as it addresses security issues labels Jan 7, 2024
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-01-07 03:37 UTC
Newest commit scanned: c0086c3
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/cbb5f0dab5/output.html

@hhoffstaette
Copy link
Contributor

I just found this PR and was wondering if we could add 4.2.3 as well? It works fine in my local overlay.

@csfore
Copy link
Contributor Author

csfore commented Apr 2, 2024

@hhoffstaette Works for me! I'll push it in a bit.

@csfore csfore changed the title net-analyzer/wireshark: add 4.0.12 net-analyzer/wireshark: add 4.0.14, 4.2.4 Apr 2, 2024
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-04-02 22:41 UTC
Newest commit scanned: fec607c
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/97108fc510/output.html

@hhoffstaette
Copy link
Contributor

👍

@csfore
Copy link
Contributor Author

csfore commented Apr 3, 2024

Should 4.1.0 be dropped as well since 4.2.4 is being added?

@hhoffstaette
Copy link
Contributor

Should 4.1.0 be dropped as well since 4.2.4 is being added?

My understanding is that 4.1.x was a development series and 4.2.x is the stable release, so IMHO yes.

@thesamesam
Copy link
Member

I'll try take a look today. Sorry for the wait.

@csfore
Copy link
Contributor Author

csfore commented Apr 4, 2024

No worries, this might have to wait until net-libs/nghttp3 can be keyworded to match wireshark though since that's why CI is failing.

@thesamesam
Copy link
Member

You should p.use.mask as appropriate or drop keywords until rekeyworded. Dropping keywords might be better here.

See https://devmanual.gentoo.org/keywording/index.html#keywording-on-upgrades.

@csfore
Copy link
Contributor Author

csfore commented Apr 4, 2024

Got it, should 4.1.0 be dropped as well?

@thesamesam
Copy link
Member

Yes, Holger was spot on. It was an unkw'd dev release.

- 4.0.14 security bump, also add 4.2.4
- New USE flag for 4.2.x, http3
- Unkeyword 4.2.4 to match net-libs/http3
- Drop 4.1.0 since it was a dev release

Bug: https://bugs.gentoo.org/921528
Signed-off-by: Christopher Fore <csfore@posteo.net>
@csfore csfore changed the title net-analyzer/wireshark: add 4.0.14, 4.2.4 net-analyzer/wireshark: add 4.0.14, 4.2.4, drop 4.1.0 Apr 4, 2024
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-04-04 01:50 UTC
Newest commit scanned: d3dd866
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/b10e482375/output.html

@gentoo-bot gentoo-bot closed this in 0050375 Apr 4, 2024
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. security PR that needs to be merged promptly as it addresses security issues
Projects
None yet
5 participants