On Mon, 6 Jan 2014, Luca Barbato wrote:

On 06/01/14 16:49, Martin Storsjö wrote:
On Mon, 6 Jan 2014, Martin Storsjö wrote:

On Mon, 6 Jan 2014, Luca Barbato wrote:

Should make work properly linking against static libraries dynamically
linking secondary libraries.
---

Apparently yet another freetype cute feat.

Nothing at all related to freetype in this, but just to proper
pkg-config use...


configure | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index 7c3a9f8..5846b60 100755
--- a/configure
+++ b/configure
@@ -918,8 +918,9 @@ check_pkg_config(){
    funcs="$3"
    shift 3
    check_cmd $pkg_config --exists --print-errors $pkg || return
-    pkg_cflags=$($pkg_config --cflags $pkg)
-    pkg_libs=$($pkg_config --libs $pkg)
+    enabled static && pkgconf_flags="--static"
+    pkg_cflags=$($pkg_config --cflags $pkgconf_flags $pkg)
+    pkg_libs=$($pkg_config --libs $pkgconf_flags $pkg)

I'm not sure if this is right - we should pass --static to pkg-config
if we explicitly are linking statically to freetype (which we
seldom/never do)

Ok, so now I read the irc backlog and noticed that this was exactly what
the user was doing - explicitly adding --extra-ldflags=-static is what
triggers this issue, not building static libav libs.

So "enabled static" is not the right condition for this - looking for
-static in the ldflags is probably the most correct way although that's
also a little ugly.


What about adding a --extra-pkgflags option and be done with that?

I guess it would work, but it would be pretty nonobvious to use and I doubt many would figure it out.

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to