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
Add -no-pie flag to CMake #4365
Conversation
|
If PIE is a requirement, disabling PIE isn't going to make Dolphin work on Ubuntu 16.10, is it? I thought it only was enabled by default, not required. |
|
@JosJuice It's enabled by default, requirement I guess isn't the right word. |
|
What's the difference between |
|
@ligfx |
|
Ah, cool, thanks. Looks like Clang on macOS, at least, uses |
|
@cristianrm650 Did you actually review the PR, or did you press the review button without reviewing? |
|
so where do you add this flag? I am compiling it via the command line and cant seem to figure this out, Kinda new to compiling things and using cmake |
|
If you pulled the PR's repository, it should just compile as normal. Otherwise, use the patch from this PR to edit the CMakeLists.txt file, then compile as you normally would. |
|
FYI, r2rX on the forums also confirms that it works. |
|
There are too many ubuntu users. Let's just merge this and wait for some other compilers to cry ... |
|
Hey all, I wanted to confirm that this PR (pre & post merge) resolved the ability to compile Dolphin in Ubuntu 16.10. Thanks alot! |
|
Why is this still broken? |
|
Getting this also... |
|
"/usr/local/lib/libmbedcrypto.a" You try to link with a static compiled library, which you have manually installed (/usr/local). Please just delete them. |
This works around the issue of our x64 JIT not working with PIE, which is now enabled by default in Ubuntu as of 16.10.
This change is