[MPlayer-dev-eng] [PATCH 2/3] configure: iconv: Remove hack that added -L/usr/lib

Alexander Strasser eclipse7 at gmx.net
Sat Nov 14 22:20:19 CET 2015


Seems not really a good idea to do such modifications to
the selection of the libraries that will be linked.

Also adding -L/usr/lib might invalidate previous results
of tests because now a different lib will finally be linked
in the end.

I got caught by this hack when compiling with a mingw compiler
and adding /usr/lib linker search path prioritized msys/cygwin
versions of the libs.

Signed-off-by: Alexander Strasser <eclipse7 at gmx.net>
---
 configure | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/configure b/configure
index 57e9cb7..6326134 100755
--- a/configure
+++ b/configure
@@ -4103,10 +4103,7 @@ int main(void) {
 }
 EOF
   _iconv=no
-  # NOTE: -L/usr/lib is a hack to avoid issues due to a
-  # broken libiconv that e.g. macports installs into /opt/local/lib
-  # which might get addded to the search path later by e.g. SDL
-  for ld_tmp in "" "-L/usr/lib -liconv" "-liconv" "-liconv $ld_dl" ; do
+  for ld_tmp in "" "-liconv" "-liconv $ld_dl" ; do
     cc_check $ld_tmp && extra_ldflags="$extra_ldflags $ld_tmp" &&
       ld_iconv="$ld_tmp" && _iconv=yes && break
   done
-- 


More information about the MPlayer-dev-eng mailing list