Skip to content

media-libs/opencv: limit cuda-12.4 again#36437

Closed
negril wants to merge 1 commit intogentoo:masterfrom
negril:media-libs/opencv/4.9.0_relimit_cuda
Closed

media-libs/opencv: limit cuda-12.4 again#36437
negril wants to merge 1 commit intogentoo:masterfrom
negril:media-libs/opencv/4.9.0_relimit_cuda

Conversation

@negril
Copy link
Copy Markdown
Contributor

@negril negril commented Apr 26, 2024

Still requires fixes from NVIDIA/cccl#1522

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

Still requires fixes from NVIDIA/cccl#1522

Bug: https://bugs.gentoo.org/928747
Signed-off-by: Paul Zander <negril.nx+gentoo@gmail.com>
@gentoo-bot
Copy link
Copy Markdown

Pull Request assignment

Submitter: @negril
Areas affected: ebuilds
Packages affected: media-libs/opencv

media-libs/opencv: @negril, @gentoo/proxy-maint

Linked bugs

Bugs linked: 928747


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 self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Apr 26, 2024
@gentoo-repo-qa-bot
Copy link
Copy Markdown
Collaborator

Pull request CI report

Report generated at: 2024-04-26 16:39 UTC
Newest commit scanned: 9248ba3
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/b7a0c1bdaf/output.html

Comment on lines +344 to +347
if has_version ">=dev-util/nvidia-cuda-toolkit-12.4" && use cuda; then
# TODO https://github.com/NVIDIA/cccl/pull/1522
eapply "${FILESDIR}/${PN}_contrib-4.9.0-cuda-12.4.patch"
fi
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What is the point of this if we restrict the version to lower then 12.4?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

None yet. It's just the start of the work to support CUDA-12.4. Still needs the fixes for the cccl mishap. But I feel like it's easier to start from there for anyone looking then to have to try to solve that as well.

@negril negril deleted the media-libs/opencv/4.9.0_relimit_cuda branch June 29, 2024 21:16
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. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants