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 eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org