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

x11-drivers/*: add workaround for hardened toolchain (forgotten with EAPI 7 bump) #9412

Closed
wants to merge 4 commits into from

Conversation

puleglot
Copy link
Contributor

@puleglot puleglot commented Aug 1, 2018

@gentoo-bot
Copy link

Pull Request assignment

Areas affected: ebuilds
Packages affected: x11-drivers/xf86-video-amdgpu

x11-drivers/xf86-video-amdgpu: @gentoo/x11

Bugs linked: 661502

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

@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 Aug 1, 2018
@puleglot
Copy link
Contributor Author

puleglot commented Aug 1, 2018

We probably need the same fix for all x11-drivers/* recently ported to EAPI 7

@mattst88
Copy link
Contributor

mattst88 commented Aug 1, 2018

This doesn't seem like an appropriate fix. What is your theory of the problem?

@puleglot
Copy link
Contributor Author

puleglot commented Aug 1, 2018

Hardened toolchain adds -z,now to the linker flags. That's why xorg-2.eclass contains the following code:

[[ ${PN} == xf86-video-* || ${PN} == xf86-input-* ]] && DRIVER="yes"
...
xorg-2_flags_setup() {
...
    [[ ${PN} = xorg-server || -n ${DRIVER} ]] && append-ldflags -Wl,-z,lazy

This is a workaround needed for hardened toolchain.

Closes: https://bugs.gentoo.org/661502
This is a workaround needed for hardened toolchain.
This is a workaround needed for hardened toolchain.
This is a workaround needed for hardened toolchain.
@puleglot puleglot changed the title x11-drivers/xf86-video-amdgpu: append -Wl,-z,lazy to LDFLAGS x11-drivers/*: add a workaround for hardened toolchain (forgotten with EAPI 7 bump) Aug 1, 2018
@puleglot puleglot changed the title x11-drivers/*: add a workaround for hardened toolchain (forgotten with EAPI 7 bump) x11-drivers/*: add workaround for hardened toolchain (forgotten with EAPI 7 bump) Aug 1, 2018
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-08-01 19:55 UTC
Newest commit scanned: 8ff357b
Status: ✅ good

No issues found

@mattst88
Copy link
Contributor

mattst88 commented Aug 1, 2018

Ouch. That makes sense now. The intent was to avoid using xorg-2 unnecessarily, but it seems that it really does offer some important things that are easy to forget. I suspect the best thing to do is to remove the EAPI=7 bumps and wait until xorg-2 has EAPI=7 support.

@puleglot puleglot closed this Aug 1, 2018
@hanetzer
Copy link
Contributor

hanetzer commented Aug 1, 2018

@mattst88 fsck. Something to put into xorg-3, then. I was thinking maybe
some distinct sub-eclasses (like xorg-driver.eclass and others if needed)
would be a good idea, instead of one monolithic one for everything xorg
related.

@puleglot puleglot deleted the xf86-video-amdgpu-fix branch October 28, 2018 10:54
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
5 participants