On 07/19/2010 10:48 AM, Paolo Bonzini wrote:
> On 07/19/2010 06:33 PM, Eric Blake wrote:
>> Yuck - that means if /tmp is mounted differently than ., then using
>> mkostemp(,0) will force the wrong line endings (converting binary to
>> text, or converting text to binary, depending on which direction the
>> mismatch is between the mount modes).  If you aren't creating the temp
>> file in the same mount point as the target, then you cannot blindly rely
>> on automatic mount point file modes to do the right thing.
> 
> It's buggy anyway in all released versions of sed.  I'll probably make
> the change---just, not yet.

As a compromise, it is faster to at least document that the choice of
TMPDIR=. can help fix line ending issues with 'sed -i' on cygwin, until
such time as you actually do make some change on either the default
in-place location or more work on explicit mode matching when creating
the temp file.

-- 
Eric Blake   [email protected]    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to