Skip to content

Commit

Permalink
configure: Fix libs order in check_ld()
Browse files Browse the repository at this point in the history
With patch and commentary from Lawrence Rust:

'The configure script defines the function check_ld which tests if a
named function can be compiled with specified headers and libraries.
However, the libraries are placed after $extralibs in the linker
invocation which can cause the link to fail if the tested libraries
depend on $extralibs.'
  • Loading branch information
Mark Kendall committed Feb 18, 2011
1 parent 92b55c9 commit ef68bf7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mythtv/configure
Expand Up @@ -760,7 +760,7 @@ check_ld(){
test "${f}" = "${f#-l}" && flags="$flags $f" || libs="$libs $f" test "${f}" = "${f#-l}" && flags="$flags $f" || libs="$libs $f"
done done
check_cc $($filter_cflags $flags) || return check_cc $($filter_cflags $flags) || return
check_cmd $ld $LDFLAGS $flags -o $TMPE $TMPO $extralibs $libs check_cmd $ld $LDFLAGS $flags -o $TMPE $TMPO $libs $extralibs
} }


check_cppflags(){ check_cppflags(){
Expand Down

0 comments on commit ef68bf7

Please sign in to comment.