lexsup.c (parse_args: OPTION_CALL_SHARED): Revise comment describing defaults.
authorNick Clifton <nickc@redhat.com>
Wed, 8 Oct 2003 13:52:33 +0000 (13:52 +0000)
committerNick Clifton <nickc@redhat.com>
Wed, 8 Oct 2003 13:52:33 +0000 (13:52 +0000)
Change default to report unresolved symbols in object files as well.

ld/ChangeLog
ld/lexsup.c

index 28c6218e083cd1876dc8d0e384d9693c1e05469a..f12a8a8719feb6870322e33182373c6c6a5533d0 100644 (file)
@@ -1,3 +1,9 @@
+2003-10-08  Nick Clifton  <nickc@redhat.com>
+
+       * lexsup.c (parse_args: OPTION_CALL_SHARED): Revise comment
+       describing defaults.  Change default to report unresolved symbols
+       in object files as well.
+
 2003-10-08  Hans-Peter Nilsson  <hp@bitrange.com>
 
        * ldexp.c (fold_name) <case DEFINED>: Set section of result to
index 556a5ba08298f8b8d2741781453325aa0f40712b..b8d0be4cba474d22821d13e6b5b6b000b60607ac 100644 (file)
@@ -661,16 +661,15 @@ parse_args (unsigned argc, char **argv)
        case OPTION_CALL_SHARED:
          config.dynamic_link = TRUE;
          /* When linking against shared libraries, the default behaviour is
-            to report any unresolved references.  This is for backwards
-            comptability with previous linker behaviour.  Whilst strictly
-            speaking it is not a failure to encounter unresolved symbols at
-            link time - the symbol *might* be available at load time - it is
-            nevertheless necessary for the correct execution of the autoconf
-            package.  (It needs to be able to detect functions that are not
-            provided by the host OS).  Since this package is used by lots of
-            projects, maintaining the old linker behaviour is important.  */
+            to report any unresolved references.  Although strictly speaking
+            it is not a failure to encounter unresolved symbols at link time
+            - the symbol *might* be available at load time - it is a strong
+            indication that the resulting executable will not work.  Plus it
+            is necessary for the correct execution of the autoconf package,
+            which needs to be able to detect functions that are not provided
+            by the host OS.  */
          if (link_info.unresolved_syms_in_objects == RM_NOT_YET_SET)
-           link_info.unresolved_syms_in_objects = RM_IGNORE;
+           link_info.unresolved_syms_in_objects = how_to_report_unresolved_symbols;
          if (link_info.unresolved_syms_in_shared_libs == RM_NOT_YET_SET)
            link_info.unresolved_syms_in_shared_libs = how_to_report_unresolved_symbols;
          break;