Re: [PATCH] small patch for depend2.am

[Art Haas]

On Wed, Aug 14, 2002 at 10:23:07AM -0600, Tom Tromey wrote:
> 
> Thanks for the patch.  I'm checking it in.  Apologies for the bug.
> 

Thanks, but I've just today found another problem in the same file.
The generated dependencies were being stored in files ending ".Po",
but the Makefile produced via automake is trying to import
these files as though they ended in ".Plo", so the dependencies
are essentially lost.

Here's a new diff that renames the generated dependency file
to ".Plo". This diff also has the extra ";" fix again.

Thanks.

Art Haas

Index: depend2.am
===================================================================
RCS file: /cvs/automake/automake/lib/am/depend2.am,v
retrieving revision 1.45
diff -u -r1.45 depend2.am
--- depend2.am  10 Jul 2002 05:33:31 -0000      1.45
+++ depend2.am  14 Aug 2002 17:17:48 -0000
@@ -81,10 +81,10 @@
 ?!GENERIC?%LTOBJ%: %SOURCE%
 if %FASTDEP%
 ## In fast-dep mode, we can always use -o.
-       if %LTCOMPILE% -MT %LTOBJ% -MD -MP -MF "%DEPBASE%.Tpo" \
-         %-c% -o %LTOBJ% `test -f '%SOURCE%' || echo '$(srcdir)/'`%SOURCE% \
-       then mv "%DEPBASE%.Tpo" "%DEPBASE%.Po"; \
-       else rm -f "%DEPBASE%.Tpo"; exit 1; \
+       if %LTCOMPILE% -MT %LTOBJ% -MD -MP -MF "%DEPBASE%.TPlo" \
+         %-c% -o %LTOBJ% `test -f '%SOURCE%' || echo '$(srcdir)/'`%SOURCE% ; \
+       then mv "%DEPBASE%.TPlo" "%DEPBASE%.Plo"; \
+       else rm -f "%DEPBASE%.TPlo"; exit 1; \
        fi
 else !%FASTDEP%
 if %AMDEP%

-- 
They that can give up essential liberty to obtain a little temporary
safety deserve neither liberty nor safety.
 -- Benjamin Franklin, Historical Review of Pennsylvania, 1759