Missing inversion in Makefile (ee9be06)

From: Philip Oakley <philipoakley@iee.org>
Date: Dec 27, 2014, 17:56 UTC
Message-ID: <E3DB9AD2A8914C379FB3371494B0B816@PhilipOakley>

Hi,

In ee9be06 (perl: detect new files in MakeMaker builds, 2012-07-27) there is a step to detect if there has been an update to the PM.* files, however it appears that the logic is inverted in the comparison.

I need some extra eye's on this to be sure I have it right (I'm trying to debug an old Windows breakage...).

The resultant output of a make dry run included (on my m/c)..:

  find perl -type f -name '*.pm' | sort >perl/PM.stamp+ && \
   { cmp perl/PM.stamp+ perl/PM.stamp >/dev/null 2>/dev/null || mv 
perl/PM.stamp+ perl/PM.stamp; } && \
   rm -f perl/PM.stamp+
  make -C perl  PERL_PATH='/usr/bin/perl' prefix='/c/Documents and 
Settings/Philip' perl.mak

Shouldn't it be `{ ! cmp ` so that when the files are not identical, the move is performed?

https://github.com/git/git/blob/ee9be06770223238c6a22430eb874754dd22dfb0/Makefile#L2097

the code is now at https://github.com/git/git/blob/master/Makefile#L1697

I'd guess that this was something that was tricky to test as once it works it stays working.

On a clean Msysgit development install the PM.stamp file does not exist and the cmp exits with error code 2, while identical files return 0.

Msysgit list also copied should someone there have seen this before. -- Philip

-- 
-- 
*** Please reply-to-all at all times ***
*** (do not pretend to know who is subscribed and who is not) ***
*** Please avoid top-posting. ***
The msysGit Wiki is here: https://github.com/msysgit/msysgit/wiki - Github accounts are free.

You received this message because you are subscribed to the Google
Groups "msysGit" group.
To post to this group, send email to msysgit@googlegroups.com
To unsubscribe from this group, send email to
msysgit+unsubscribe@googlegroups.com
For more options, and view previous threads, visit this group at
http://groups.google.com/group/msysgit?hl=en_US?hl=en

--- 
You received this message because you are subscribed to the Google Groups "Git for Windows" group.
To unsubscribe from this group and stop receiving emails from it, send an email to msysgit+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.