Re: [Monotone-devel] Time for a release

[Thomas Keller]

Am 05.06.10 18:26, schrieb Zack Weinberg:
> On Sat, Jun 5, 2010 at 4:59 AM, Thomas Keller <address@hidden> wrote:
>>> I assume you meant "/dev/nul instead of /dev/null" there?
>>
>> No, I meant /dev/nul - try GNU patch and tamper the patch file to
>> "+++ /dev/nul" (which is of course wrong) - then you'll see that empty
>> files are kept and are only removed explicitely with -E. This is what I
>> mean - we'd tamper the test if we add -E, because then we just test GNU
>> patch to correctly interpret the -E option, but not to automatically
>> remove the file if it is empty _and_ has the target /dev/null.
> 
> Oh, I see what you mean.  Well, is the exact behavior of the system
> 'patch' utility the focus of the test?  I'm kinda inclined to doubt
> it, but I have lost the message that says which test we're talking
> about here...

The test is diff_patch_drop and IMHO it exactly tests the proper removal
of the files at the end. If not we could / should probably just remove
the last two checks.

Thomas.

-- 
GPG-Key 0x160D1092 | address@hidden | http://thomaskeller.biz
Please note that according to the EU law on data retention, information
on every electronic information exchange might be retained for a period
of six months or longer: http://www.vorratsdatenspeicherung.de/?lang=en

signature.asc
Description: OpenPGP digital signature