pkgconfig 0.25 is built with --enable-indirect-deps, causing breakage in apps that link against libsdl.

[endgame.dos@…]

I've noticed that pkg-config is built with --enable-indirect-deps, which causes Libs.private to get unconditionally printed (even when --static is not passed to pkg-config). This causes runtime failures in programs linked against SDL, because the output of pkg-config --libs sdl looks something like this:

-Wl,-framework,Cocoa /opt/local/lib/libSDLmain.a /opt/local/lib/libSDL.a -Wl,-framework,OpenGL -Wl,-framework,ApplicationServices -Wl,-framework,Carbon -Wl,-framework,AudioToolbox -Wl,-framework,AudioUnit -Wl,-framework,IOKit -L/opt/local/lib -lSDLmain -lSDL

In particular, this is linking the SDL .dylib and the SDL static lib. This causes runtime warnings that the classes SDLTranslatorResponder, SDL_QuartzView, SDL_QuartzWindowDelegate and SDL_QuartzWindow are implemented in two places, followed by a crash.

Removing --enable-indirect-deps from the pkgconfig portfile allows SDL apps to correctly link and run.

[michaelld (Michael Dickens)]

Cc Me!

[jmroot (Joshua Root)]

Please remember to cc the maintainer.

[ryandesign (Ryan Carsten Schmidt)]

I have run into this same issue myself when working on the port for PlasmaClient. See r71898.

However, I'm worried about making a change that will affect so many ports. I'd appreciate some guidance on why this option was added to pkgconfig in the first place, and if that reason still applies.

Also, if we make this change in pkgconfig, then theoretically we'd have to revbump all ports that use pkgconfig, which is a ton of them.

[endgame.dos@…]

Strangely enough, I don't see this any more:

$ which pkg-config
/opt/local/bin/pkg-config
$ pkg-config --cflags --libs sdl
-D_GNU_SOURCE=1 -D_THREAD_SAFE -I/opt/local/include/SDL  -Wl,-framework,Cocoa -L/opt/local/lib -lSDLmain -lSDL  

Was the libsdl port changed recently? The other libs are still there in libs.private in the .pc file.

[endgame.dos@…]

Sorry, disregard the above. I'm using a locally modified Portfile for pkgconfig.

[ryandesign (Ryan Carsten Schmidt)]

Replying to ryandesign@…:

I'd appreciate some guidance on why this option was added to pkgconfig in the first place, and if that reason still applies.

Looks like the option was added in r12236 to "Fix the problem where the configure scripts for GTK2 apps wasn't finding GTK2 even though it was installed." I do not know if this reason still applies.

Separately, I have received email from Jeremy Huddleston, saying any port that needs this option is broken and the problem should be fixed in that port, and that sounds reasonable.

Now it just remains to decide whether I simply remove the flag and revbump pkgconfig, similar to what was done in r12236 to add it, or whether I need to also revbump all ports that use pkgconfig.

[exg (Emanuele Giaquinta)]

Cc Me!

[jmroot (Joshua Root)]

You don't need to rev bump anything that is working.

[jmroot (Joshua Root)]

r75297 - invoking maintainer timeout as this was preventing me from updating libsdl_ttf.

[ryandesign (Ryan Carsten Schmidt)]

Ok, here's hoping things continue to work after that change. :)

[drkp (Dan Ports)]

This appears to have broken the build for emacs +x11 (#28083). It looks like it's not getting the right linker flags for freetype and fontconfig.

I'm totally willing to believe that the problem is with emacs not pkg-config, but I'd appreciate any suggestions about how to fix it.

[drkp (Dan Ports)]

The xournal build failure in #28085 is probably also related.