If you're going to tack on a character at the end, shorten the length from 9 to 8. I believe the goal is to keep the resulting string to 14 characters. Most likely this is to be compliant with _POSIX_NAME_MAX, which in limits.h is 14.
- doug -----Original Message----- From: cygwin-ow...@cygwin.com [mailto:cygwin-ow...@cygwin.com] On Behalf Of Charles Wilson Sent: Monday, April 05, 2010 10:50 PM To: cygwin@cygwin.com Subject: Re: [ATTN: cvs maintainer] trailing dot issue with cygwin 1.7 On 4/4/2010 11:01 AM, crosmun_will...@emc.com wrote: > The _POSIX_NO_TRUNC variable is defined in cygwin 1.7, and not defined in cygwin 1.5, so in cygwin 1.7, the code goes down the path where the temp_filename is the first 9 characters of the original filename preceded by '.new.' The problem occurs when the 9th character in the filename is a dot. Eg, 'abcdefgh.txt'. The temp_filename becomes '.new.abcdefgh.' Unfortunately, when checking out to a Samba share, or other network shares that try to be CIFS compatible, a trailing . is invalid in a filename. The temporary file either doesn't get created, or gets created but with the trailing dot truncated. The checkout then fails with 'file not found' errors. > > A simple fix would be to add a valid character after the .9s, eg > > sprintf (temp_filename, ".new.%.9s~", filename); > > Can something like this be done and a new package released? Thanks for the report and suggestion. I'll include a suitable change in the next release, but it may be a few weeks before I can get to that. -- Chuck -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
