- fix bug where we linked again -lm even though it is not needed.
authoraldot <aldot@69ca8d6d-28ef-0310-b511-8ec308f3f277>
Sun, 2 Sep 2007 14:51:54 +0000 (14:51 +0000)
committeraldot <aldot@69ca8d6d-28ef-0310-b511-8ec308f3f277>
Sun, 2 Sep 2007 14:51:54 +0000 (14:51 +0000)
  For BBOX_LIB_LIST="crypt m" trylink ... with just applet true, we pulled in
  libm because in this case we tried to use invalid flags (plain "-l" without
  a lib) which of course failed, thus the script thought that -lm was needed.

  The fix is not to pass "-l" without a lib if we are about to check if any
  or the last remaining lib is really needed.

git-svn-id: svn://busybox.net/trunk/busybox@19760 69ca8d6d-28ef-0310-b511-8ec308f3f277

scripts/trylink

index 4eaa334..6947307 100755 (executable)
@@ -39,8 +39,17 @@ while test "$BBOX_LIB_LIST"; do
     for one in $BBOX_LIB_LIST; do
        without_one=`echo " $BBOX_LIB_LIST " | sed "s/ $one / /g" | xargs`
        l_list=`echo "$without_one" | sed -e 's/ / -l/g' -e 's/^/-l/'`
-       $debug && echo "Trying -l options: $l_list"
-       if try "-Wl,--start-group $l_list -Wl,--end-group" "$@"; then
+       # If l_list is just "-l" without a lib, then make sure to test the
+       # correct thing to fail: just using -l will fail, so the last lib
+       # (usually m in my case) will incorrectly be added as needed.
+       if test "x$without_one" != "x"; then
+               l_list="-Wl,--start-group $l_list -Wl,--end-group"
+       else
+               # without_one is empty, so l_list has to be empty too
+               l_list=""
+       fi
+       $debug && echo "Trying -l options: '$l_list'"
+       if try "$l_list" "$@"; then
                echo "Library $one is not needed"
                BBOX_LIB_LIST="$without_one"
                all_needed=false