Re: [PATCH v2] t5000, t5003: do not use test_cmp to compare binary files
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 4, 2014, 18:22 UTC
- Message-ID
- <xmqq8upcv8jj.fsf@gitster.dls.corp.google.com>
- In-Reply-To
- <20140604155752.GA23226@camelia.ucw.cz>
Stepan Kasal <kasal@ucw.cz> writes:
Show 29 quoted lines
> test_cmp() is primarily meant to compare text files (and display the > difference for debug purposes). > > Raw "cmp" is better suited to compare binary files (tar, zip, etc.). > > On MinGW, test_cmp is a shell function mingw_test_cmp that tries to > read both files into environment, stripping CR characters (introduced > in commit 4d715ac0). > > This function usually speeds things up, as fork is extremly slow on > Windows. But no wonder that this function is extremely slow and > sometimes even crashes when comparing large tar or zip files. > > Signed-off-by: Stepan Kasal <kasal@ucw.cz> > --- > > Hi Thomas, > On Wed, Jun 04, 2014 at 02:59:44PM +0200, Thomas Braun wrote: >> Using test_cmp_bin instead of cmp would result in then four assertions >> for comparing arbitrary data >> test_cmp >> test_i18ncmp >> test_cmp_text >> test_cmp_bin >> where I think the purpose of each function is clear from its name. > > [test_cmp_text does not exist (yet)] > > OK, I agree, hence this modified version of the patch.
Yeah, I think the above reasoning is sound. And I do not think we ever need to have test_cmp_text -- our payload and our messages compared by tests to make sure our expectations hold are text by default.
Will queue; thanks.
-- -- *** 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 "msysGit" 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.