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-libs/intel-neo, dev-libs/opencl-clang, dev-util/intel-graphics-compiler, and dev-util/spirv-llvm-translator: Version Bumps (can now be build with llvm 11) #18284

Closed
wants to merge 7 commits into from

Conversation

Titaniumtown
Copy link

Note: This is my first PR to this project!

This PR updates: dev-libs/intel-neo, dev-libs/opencl-clang, dev-util/intel-graphics-compiler, and dev-util/spirv-llvm-translator
Currently you can't install any of those packages while having llvm/clang 11.0.0 installed, this PR updates those packages so it is possible to build. Also, these ebuilds come from my own Gentoo overlay.

Hope this PR can be accepted!

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @Titaniumtown
Areas affected: ebuilds
Packages affected: dev-libs/intel-neo, dev-libs/opencl-clang, dev-util/intel-graphics-compiler, dev-util/spirv-llvm-translator

dev-libs/intel-neo: @gentoo/proxy-maint (maintainer needed)
dev-libs/opencl-clang: @gentoo/proxy-maint (maintainer needed)
dev-util/intel-graphics-compiler: @gentoo/proxy-maint (maintainer needed)
dev-util/spirv-llvm-translator: @gentoo/proxy-maint (maintainer needed)

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

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 maintainer-needed There is at least one affected package with no maintainer. Review it if you can. assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Nov 16, 2020
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2020-11-19 16:14 UTC
Newest commit scanned: 8bae901
Status: ❌ broken

New issues caused by PR:
https://qa-reports.gentoo.org/output/gentoo-ci/ebf1d97738/output.html#dev-libs/intel-neo

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/ebf1d97738/output.html

Copy link
Member

@juippis juippis left a comment

Choose a reason for hiding this comment

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

While the ebuilds look rather good, a sign-off is required from you to each of your commits. Please see https://www.gentoo.org/glep/glep-0076.html#certificate-of-origin

And because you'll have to amend your commits, you could edit your git commit summaries to match our requirements, https://www.gentoo.org/glep/glep-0066.html#commit-messages (just check the git log on repo really)

@thesamesam
Copy link
Member

@Titaniumtown Hey! Can you have a look at juippis' comment above?

We also need to address the CI issue.

@Titaniumtown
Copy link
Author

@thesamesam Sorry, I have been busy lately. I'll try to do it when I get the time.

@ConiKost
Copy link
Contributor

It seems, that all versions of this PR are already in tree, so closing here.

@ConiKost ConiKost closed this Feb 27, 2021
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). maintainer-needed There is at least one affected package with no maintainer. Review it if you can. no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants