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

sys-libs/readline: USE=utils: port to modern C (C99) #36621

Closed
wants to merge 1 commit into from

Conversation

gabifalk
Copy link
Contributor

@gabifalk gabifalk commented May 9, 2024

GCC 14 enables -Werror=implicit-function-declaration and -Werror=implicit-int by default.


Please check all the boxes that apply:

  • I can submit this contribution in agreement with the Copyright Policy.
  • I have certified the above via adding a Signed-off-by line to every commit in the pull request.
  • This contribution has not been created with the assistance of Natural Language Processing artificial intelligence tools, in accordance with the AI policy.
  • I have run pkgcheck scan --commits --net to check for issues with my commits.

Please note that all boxes must be checked for the pull request to be merged.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @gabifalk
Areas affected: ebuilds
Packages affected: sys-libs/readline

sys-libs/readline: @gentoo/base-system

Linked bugs

Bugs linked: 870424, 919256


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. labels May 9, 2024
@@ -0,0 +1,291 @@
From d09284d53d1a4d1d58207b1240986329285d01ca Mon Sep 17 00:00:00 2001
Copy link
Member

@thesamesam thesamesam May 9, 2024

Choose a reason for hiding this comment

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

Have you sent this to Chet (ideally on bug-readline)? If so, include a link. If not, please do ;)

(... or, I guess, the version with configure.in, I don't remember if the tarball has it or not; at least making him aware of the problem is important)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This patch modifies the generated configure script. A proper change for upstream will likely involve porting to a newer version of autoconf. Additionally, I don't have a copyright assignment.

Copy link
Member

@thesamesam thesamesam May 9, 2024

Choose a reason for hiding this comment

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

OK, just send an email reporting it to him then. Chet doesn't ever really apply patches anyway, he just takes inspiration (or not) from them.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-09 19:10 UTC
Newest commit scanned: acc885d
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/99063c8651/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-09 21:40 UTC
Newest commit scanned: 72327f2
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/1193fe3699/output.html

GCC 14 enables -Werror=implicit-function-declaration and
-Werror=implicit-int by default.

Closes: https://bugs.gentoo.org/870424
Closes: https://bugs.gentoo.org/919256
Link: https://lists.gnu.org/archive/html/bug-readline/2024-05/msg00000.html
Signed-off-by: Gabi Falk <gabifalk@gmx.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-09 23:59 UTC
Newest commit scanned: d146b47
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/4ba055fd8c/output.html

@gabifalk gabifalk deleted the patch-readline-rlfe-c99 branch May 11, 2024 12:53
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.
Projects
None yet
4 participants