OS X tests status update

Vincent Ladeuil v.ladeuil+lp at free.fr
Tue Aug 26 15:04:48 BST 2008


>>>>> "Guillermo" == Guillermo Gonzalez <guillo.gonzo at gmail.com> writes:

    >> And *I* am using a case-sensitive HFS+ and they fail too. I
    >> suspect the 'preserving-case' HFS property (even if it shouldn't
    >> show up for me, weird).

    Guillermo> indeed..
    Guillermo> after a little google search[1], I learned that by default HFS+ is
    Guillermo> case-preserving AND case-insensitive.

case-preserving implies case-insensitive

The only difference is that case-preserving retains the file name
used at creation time. After that point, the file can be accessed
with any case but will always appear with its original name
(except when it doesn't :-/ ).

vila:~/tmp :) $ mkdir osx
vila:~/tmp :) $ cd osx
vila:~/tmp/osx :) $ touch Toto
vila:~/tmp/osx :) $ ls 
Toto
vila:~/tmp/osx :) $ ls toto
toto
vila:~/tmp/osx :) $ 

    Guillermo> So, I wondering if we should to teach bazaar this
    Guillermo> special OS X/HFS+ behaviour or we can reuse some
    Guillermo> of the win32 fs magic?

Reusing the win32 fs magic was the intent of
CaseInsensitiveFilesystemFeature.

Since the tests are currently failing, I think we should try
harder.

Given that only a few devs fully understand the difference
between case-preserving and case-insensitive (and even fewer
*care*), I don't think anybody will *require* bzr to handle
them. So treating HFS+ as case-insensitive sounds reasonable.

    >> 
    >> Can you publish your branch on launchpad in the ~bzr namespace so
    >> that we can both commit on it ?
    >> 

    Guillermo> done: lp:~bzr/bzr/osx_tests_fix

Thanks, I've pushed an update to your first patch to:

- consistently use osutils.mkdtemp instead of tempfile.mkdtemp
  (but I wonder if it's really the right approach since *one*
  test
  (bzrlib.tests.branch_implementations.test_permissions.TestPermissions.test_new_branch)
  really want to set the group sticky bit... Is it really worth
  it to add a chown call for every temp directory (as you do in
  _mac_mkdtemp) ?

- fix a bug in workingtree fixing 3 more tests

Down to 4 failures, 9 errors :-)

  Vincent



More information about the bazaar mailing list