[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: rm -f

From: Ralf Wildenhues
Subject: Re: rm -f
Date: Sat, 22 Dec 2007 18:47:41 +0100
User-agent: Mutt/1.5.13 (2006-08-11)

Proposed patch below should fix all instances.  Please test, thanks.
I'll probably apply after the holidays.

Merry christmas everyone!

2007-12-22  Ralf Wildenhues  <address@hidden>

        * libltdl/config/ltmain.m4sh (func_mode_link): When creating
        reloadable object files, avoid issuing plain `rm -f' without
        further arguments, even if the line length limit is ridiculously
        low as in the `Run tests with low max_cmd_len' test.  This
        failure is exposed on NetBSD.
        Report by Patrick Welche.

Index: libltdl/config/ltmain.m4sh
RCS file: /cvsroot/libtool/libtool/libltdl/config/ltmain.m4sh,v
retrieving revision 1.92
diff -u -r1.92 ltmain.m4sh
--- libltdl/config/ltmain.m4sh  24 Nov 2007 08:00:21 -0000      1.92
+++ libltdl/config/ltmain.m4sh  22 Dec 2007 17:41:15 -0000
@@ -6184,7 +6184,10 @@
              # reloadable object file.  All subsequent reloadable object
              # files will link in the last one created.
              test -z "$concat_cmds" || concat_cmds=$concat_cmds~
-             eval concat_cmds=\"\${concat_cmds}$reload_cmds $objlist 
$last_robj~\$RM $last_robj\"
+             eval concat_cmds=\"\${concat_cmds}$reload_cmds $objlist 
+             if test -n "$last_robj"; then
+               eval concat_cmds=\"\${concat_cmds}~\$RM $last_robj\"
+             fi
              delfiles="$delfiles $output"
@@ -6199,7 +6202,7 @@
              # Append the command to create the export file.
              test -z "$concat_cmds" || concat_cmds=$concat_cmds~
              eval concat_cmds=\"\$concat_cmds$export_symbols_cmds\"
-             if test -n "$output"; then
+             if test -n "$last_robj"; then
                eval concat_cmds=\"\$concat_cmds~\$RM $last_robj\"

reply via email to

[Prev in Thread] Current Thread [Next in Thread]