Hi,
Tom Tromey <[EMAIL PROTECTED]> writes:
> >>>>> "Derek" == Derek R Price <[EMAIL PROTECTED]> writes:
> Derek> One of the other CVS developers reported a bug in depcomp on
> Derek> BSD/OS. Apparently the included /bin/sh doesn't set $? inside
> Derek> of the conditional. His original message and fix are attached.
>
> I checked in a variant of this patch.
> depcomp has changed since the patch was made.
>
> Larry, can you try the new depcomp to see if it works?
> I can email it to you if you like.
The 'gcc3' code looks somewhat suspect :-) I think it's preferable to
use less obtuse code. The rest of the file uses
if test "$stat" != "0"; then exit $stat; fi
So, the cases fixed above should use code like
...
if test $stat -ne 0; then
rm -f "$tmpdepfile"
exit $stat
fi
...
Anyway, here's a fix for the one problem.
from Raja R Harinath <[EMAIL PROTECTED]>
* depcomp (gcc3): Invert test condition.
Index: depcomp
===================================================================
RCS file: /cvs/automake/automake/depcomp,v
retrieving revision 1.20
diff -u -p -u -r1.20 depcomp
--- depcomp 2001/04/07 20:22:07 1.20
+++ depcomp 2001/04/07 20:49:36
@@ -53,7 +53,7 @@ gcc3)
## we want. Yay!
"$@" -MT "$object" -MF "$tmpdepfile" -MD -MP
stat=$?
- if test $stat -ne 0; then :
+ if test $stat -eq 0; then :
else
rm -f "$tmpdepfile"
exit $stat
- Hari
--
Raja R Harinath ------------------------------ [EMAIL PROTECTED]
"When all else fails, read the instructions." -- Cahn's Axiom
"Our policy is, when in doubt, do the right thing." -- Roy L Ash
