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/jwhois: EAPI8, fix #446472 #33566

Closed
wants to merge 2 commits into from
Closed

Conversation

mm1ke
Copy link
Contributor

@mm1ke mm1ke commented Oct 28, 2023

Hi,

this updates net-misc/jwhois to EAPI8. It also fixes bug #446472
While looking at the ebuild i also got following QA Warnings.

 * QA Notice: Found the following implicit function declarations in configure logs:
 *   /var/tmp/portage/net-misc/jwhois-4.0-r2/work/jwhois-4.0/config.log:626 - inet_pton
 *   /var/tmp/portage/net-misc/jwhois-4.0-r2/work/jwhois-4.0/config.log:629 - exit

I didn't had any idea how to correctly fix this, however there is an upstream PR which addresses this issue [1].
I've decided to add these two patches from there, jwhois-4.0-avoid-implicit-declarations.patch fixes the QA Warning, jwhois-4.0-add-timeout_init-prototype.patch should fix compilation errors with future compilers.

[1] jonasob/jwhois#35

Signed-off-by: Michael Mair-Keimberger mmk@levelnine.at

Closes: https://bugs.gentoo.org/446472

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @mm1ke
Areas affected: ebuilds
Packages affected: net-misc/jwhois

net-misc/jwhois: @gentoo/base-system

Linked bugs

Bugs linked: 446472


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 Oct 28, 2023
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-10-28 12:56 UTC
Newest commit scanned: 7f6ba37
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/778b158aee/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-11-15 20:21 UTC
Newest commit scanned: 4708a1b
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/6feb9c16cb/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-11-21 17:56 UTC
Newest commit scanned: d1be141
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/3f139091b3/output.html

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-12-08 11:32 UTC
Newest commit scanned: f93cfc8
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/88764b8fbd/output.html

Signed-off-by: Michael Mair-Keimberger <mmk@levelnine.at>

Closes: https://bugs.gentoo.org/446472
Signed-off-by: Michael Mair-Keimberger <mmk@levelnine.at>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-12-19 20:08 UTC
Newest commit scanned: 6ac5a36
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/f8fc8cb217/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.
Projects
None yet
3 participants