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

plugins: fix undefined references #413

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

orbea
Copy link

@orbea orbea commented Mar 17, 2024

Many of the plugins have undefined references, but with GNU libtool it silently ignores the -no-undefined flag so the build succeeds. With slibtool it correctly sets -Wl,--no-undefined in the linker flags causing the build to fail.

Please view the commit messages for more details. The alternative to removing -no-undefined in the last commit is to turn most of mate-settings-daemon into a convenience library and link that into most of the plugins, but that doesn't seem correct.

This was reported for Gentoo: https://bugs.gentoo.org/922326

Many of the plugins depend on the libX11 and libXi linker flags, but
they are missing.

With GNU libtool the -no-undefined flag is silently ignored so the build
succeeds, but with slibtool is correctly sets -Wl,--no-undefined in the
linker flags.

Gentoo issue: https://bugs.gentoo.org/922326
Many of the plugins depend on mate_settings_plugin_get_type(), but this
symbol is only available at runtime in the mate-settings-daemon program.

With GNU libtool the -no-undefined flag gets silently ignored and the
build succeeds, but with slibtool it correctly sets -Wl,--no-undefined
in the LDFLAGS causing the build to fail.

To solve this simply removing -no-undefined is enough.

Gentoo issue: https://bugs.gentoo.org/922326
@lukefromdc lukefromdc requested a review from a team April 1, 2024 18:56
@lukefromdc
Copy link
Member

I simply don't know enough about the build system to properly evaluate this, so leaving it for other team members

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

Successfully merging this pull request may close these issues.

None yet

2 participants