Re: [Bug-tar] tar 1.27.1 regression on darwin12

[Nathan Stratton Treadway]

On Mon, Nov 25, 2013 at 09:26:14 -0500, Jack Howarth wrote:
> On Sun, Nov 24, 2013 at 11:06:44PM -0500, Nathan Stratton Treadway wrote:
> > On Sun, Nov 24, 2013 at 13:35:37 -0500, Jack Howarth wrote:
> > >   I am seeing the following regression in tar 1.27.1 on 
> > > x86_64-apple-darwin12...
> > 
> > [...]
> > > echo 'a' > A/b
> > [...]
> > > mkdir ext/A/B
> > > touch ext/A/B/file
> > [...]
> > > --- -   2013-11-24 13:17:27.000000000 -0500
> > > +++ 
> > > /sw/src/fink.build/tar-1.27.1-1/tar-1.27.1/tests/testsuite.dir/at-groups/92/stderr
> > >   2013-11-24 13:17:27.000000000 -0500
> > > @@ -5,5 +5,7 @@
> > >  E0
> > >  E1
> > >  E2
> > > +mkdir: ext/A/B: File exists
> > > +touch: ext/A/B/file: Not a directory
> > >  E3
> > >  
> > 
> > Are you running this test on a volume where filename case is ignored? 
> > The test assumes that case is significant.
> > 
> > 
> 
> The standard darwin filesystem has always been case-insensitive so this
> test should be enhanced to only enable it on case-sensitive file systems.

I realized that the test used the same letter in upper and lower case
just because they were easy names to type; there wasn't any reason they
actually needed to to use the "same" name.

Attached is a patch that simply changes the names given to the temporary
directories so they don't clash with the names given to the files.  It
works as expected on my Linux (i.e. case-sensitive) filesystem; if you
could test it under Darwin and confirm it works correctly there,
hopefully Paul or Sergey will be willing to push it.

                                                Nathan

----------------------------------------------------------------------------
Nathan Stratton Treadway  -  address@hidden  -  Mid-Atlantic region
Ray Ontko & Co.  -  Software consulting services  -   http://www.ontko.com/
 GPG Key: http://www.ontko.com/~nathanst/gpg_key.txt   ID: 1023D/ECFB6239
 Key fingerprint = 6AD8 485E 20B9 5C71 231C  0C32 15F3 ADCD ECFB 6239

tar_incr07.patch
Description: Text Data