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-java/jna-5.10.0: DEPEND on x11-libs/libXt #23708

Closed
wants to merge 1 commit into from

Conversation

Leo3418
Copy link
Contributor

@Leo3418 Leo3418 commented Jan 9, 2022

The 'awt' USE flag has not been added to 5.10.0, which is effectively
like this USE flag is enabled by default (dev-java/jna[awt(+)]).  Bug
830812 shows that x11-libs/libXt is still required for 5.10.0.  Another
long-existing bug 762940 suggests that if the ebuild had the 'awt' USE
flag and it was disabled then libXt might be redundant, but that is not
applicable to this case.

As of the commit time, based on experiments in containers created from
the gentoo/stage3 Docker image, declaration of x11-libs/libXt as a
dependency would only cause 3 additional packages to be installed in a
new copy of stage3, none of which would take a very long time to merge,
so the benefit of making the dependency optional via the 'awt' USE flag
might not be very significant.

Bug: https://bugs.gentoo.org/762940
Closes: https://bugs.gentoo.org/830812
Signed-off-by: Yuan Liao <liaoyuan@gmail.com>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @Leo3418
Areas affected: ebuilds
Packages affected: dev-java/jna

dev-java/jna: @gentoo/java

Linked bugs

Bugs linked: 762940, 830812


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 Jan 9, 2022
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-01-09 00:30 UTC
Newest commit scanned: 6b53b55
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/4275c51b2e/output.html

@gentoo-bot gentoo-bot closed this in 1a0f8af Jan 9, 2022
@fordfrog
Copy link
Contributor

fordfrog commented Jan 9, 2022

@Leo3418 merged, thanks!

@Leo3418 Leo3418 deleted the bug-830812 branch January 9, 2022 22:55
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