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

lmms 1.2.0: text relocation in /usr/lib/lmms/RemoteVstPlugin.exe.so #5024

Open
dvzrv opened this issue Jun 9, 2019 · 2 comments
Open

lmms 1.2.0: text relocation in /usr/lib/lmms/RemoteVstPlugin.exe.so #5024

dvzrv opened this issue Jun 9, 2019 · 2 comments

Comments

@dvzrv
Copy link

dvzrv commented Jun 9, 2019

When building lmms 1.2.0 for Arch Linux, our package check tool namcap reports text relocations in /usr/lib/lmms/RemoteVstPlugin.exe.so:

lmms W: ELF file ('usr/lib/lmms/RemoteVstPlugin.exe.so') has text relocations.

Is that library build with -fPIC?

@tresf
Copy link
Member

tresf commented Jun 10, 2019

Related #4806.

No, I don't think so. Looking back at the codebase, I believe the whole project used to be in 2006, but must have gotten refactored out at some point.

From what I'm reading, all plugins should build with this flag. Do you get more of these? If not, it may be due to the fact that we compile this particular plugin by hand instead of using whatever defaults that cmake uses. Long term, we should convert this to ExternalProject_Add instead.

@PhysSong
Copy link
Member

Long term, we should convert this to ExternalProject_Add instead.

IIRC, @lukas-w refactored RemoteVstPlugin build logic to use that in #4421.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants