Re: Missing inversion in Makefile (ee9be06)
- From
Philip Oakley <philipoakley@iee.org>
- Date
- Dec 27, 2014, 19:07 UTC
- Message-ID
- <4C7BB65E9EE445F08645582230EA9782@PhilipOakley>
- In-Reply-To
- <549F0355.5020805@kdbg.org>
From: "Johannes Sixt" <j6t@kdbg.org>
Show 33 quoted lines
> Am 27.12.2014 um 19:49 schrieb Philip Oakley:
>> 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 existing code looks correct to me. cmp succeeds when the files are
> identical and fails when they are different: When it succeeds (files
> are
> equal), the mv is not executed. When it fails, either because a file
> does not exist or they are different, the mv is executed.
>Thanks. The inverse logic had me confused. It's like 7400's again, for those that remember;-)
I was getting errors from `cd $git_dir && make -n MSVC=1 V=1 2>MakeDryErrs.txt 1>MakeDry.txt` (borrowed from 'msvc-build') which reported the PM.stamp as a problem, with the quoted code being the last part of the MakeDry.txt (and no PM.stamp seen).
Now that I've been poking and investigating the error's stopped! It's all getting rather frustrating. Time to go again on a clean and rebuild..
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.