diff options
| author | Sebastian Thiel <byronimo@gmail.com> | 2015-01-12 11:14:06 +0100 |
|---|---|---|
| committer | Sebastian Thiel <byronimo@gmail.com> | 2015-01-12 11:14:06 +0100 |
| commit | 43e430d7fa5298f6db6b1649c1a77c208bacf2fc (patch) | |
| tree | ed554025ee4366c76a318f50dfe48ac0783f28f1 /git/test/lib/helper.py | |
| parent | dfb0a9c4bca590efaa86f8edc3fdb62bd536bce7 (diff) | |
| download | gitpython-43e430d7fa5298f6db6b1649c1a77c208bacf2fc.tar.gz | |
Repo paths are now converted to real paths, as well as all paths involved in index related work.
That way, we don't try to compare a real-path to a non-real one, which would make the implementation think
a file is not actually part of the repository.
Fixes #224
Diffstat (limited to 'git/test/lib/helper.py')
| -rw-r--r-- | git/test/lib/helper.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/git/test/lib/helper.py b/git/test/lib/helper.py index 6c9f33c6..a0b8dc15 100644 --- a/git/test/lib/helper.py +++ b/git/test/lib/helper.py @@ -62,10 +62,14 @@ class StringProcessAdapter(object): def _mktemp(*args): - """Wrapper around default tempfile.mktemp to fix an osx issue""" + """Wrapper around default tempfile.mktemp to fix an osx issue + :note: the OSX special case was removed as it was unclear why that was needed in the first place. It seems + to be just fine without it. However, if we leave this special case, and if TMPDIR is set to something custom, + prefixing /private/ will lead to incorrect paths on OSX.""" tdir = tempfile.mktemp(*args) - if sys.platform == 'darwin': - tdir = '/private' + tdir + # See :note: above to learn why this is comented out. + # if sys.platform == 'darwin': + # tdir = '/private' + tdir return tdir |