Web lists-archives.com

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




Committed.

On 2015-11-14 22:20 +0100, Alexander Strasser wrote:
> 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@xxxxxxx>
> ---
>  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
_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng