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

dev-python/reportlab: revbump 3.6.11-r1, add sparc SIGBUS patch #26970

Closed
wants to merge 1 commit into from

Conversation

matoro
Copy link
Contributor

@matoro matoro commented Aug 23, 2022

This actually fixes a real bug that normally only would have manifested
under specific conditions (possibly crafted input file) but was revealed
due to sparc's alignment requirements. Initially misidentified as a
cpython bug [1] but the cpython devs helpfully clarified and pointed in
the right direction [2].

[1] python/cpython#94141
[2] python/cpython#94149

Bug: https://bugs.gentoo.org/857408
Bug: https://bugs.gentoo.org/865051

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @matoro
Areas affected: ebuilds
Packages affected: dev-python/reportlab

dev-python/reportlab: @gentoo/python

Linked bugs

Bugs linked: 857408, 865051

Missing GCO sign-off

Please read the terms of Gentoo Certificate of Origin and acknowledge them by adding a sign-off to all your commits.


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. no signoff One or more commits do not indicate GCO sign-off. labels Aug 23, 2022
@@ -0,0 +1,32 @@
From 9ba2380a68e891667a2659e16d0f04ebe0ecfd19 Mon Sep 17 00:00:00 2001
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Include ink to the upstream commit here too please

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There isn't one this time. They don't do development over git, it's in SVN and it's not open for patches. I tried to submit to their -users mailing list but it rejected my mail server.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh bleh. Okay. Sorry, I remember now, tried to submit something ages ago.

Can you add a comment to that effect at the top? We should ask someone to forward it - @mgorny was able to get through in the past (I couldn't).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@thesamesam
Copy link
Member

great work, by the way

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-23 04:07 UTC
Newest commit scanned: 7d78fcd
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/664e542682/output.html

This actually fixes a real bug that normally only would have manifested
under specific conditions (possibly crafted input file) but was revealed
due to sparc's alignment requirements.  Initially misidentified as a
cpython bug [1] but the cpython devs helpfully clarified and pointed in
the right direction [2].

[1] python/cpython#94141
[2] python/cpython#94149

Bug: https://bugs.gentoo.org/857408
Bug: https://bugs.gentoo.org/865051
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-23 04:51 UTC
Newest commit scanned: 7640ac3
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/a01a032857/output.html

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. no signoff One or more commits do not indicate GCO sign-off.
Projects
None yet
4 participants