[vlc-devel] configure doesn't include nsl in LIBS on illumos

Jon Tibble meths at btinternet.com
Fri Jan 3 21:40:10 CET 2014


Hi,

On illumos/Solaris based systems the socket libraries come as a pair and 
should be linked to together.

The attached patch adds nsl on illumos if the check for connect in 
socket succeeds.

Regards,
Jon
-------------- next part --------------
commit df524bb044190369869293759734d1f849a7d4bf
Author: Jon Tibble <meths at btinternet.com>
Date:   Fri Jan 3 20:02:46 2014 +0000

    VLC on illumos should link both socket and nsl

diff --git a/configure.ac b/configure.ac
index 9e2714a..580d1c8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -586,6 +586,9 @@ AC_SEARCH_LIBS(connect, [socket], [
   AS_IF([test "$ac_cv_search_connect" != "none required"], [
     SOCKET_LIBS="$ac_cv_search_connect"
   ])
+  AS_IF([test "${SYS}" = "solaris"], [
+    SOCKET_LIBS="$SOCKET_LIBS -lnsl"
+  ])
 ], [
   AS_IF([test "${SYS}" = "mingw32"], [
     SOCKET_LIBS="-lws2_32"


More information about the vlc-devel mailing list