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

Author: Byron

git/index/base.py

@@ -535,6 +535,7 @@ def _to_relative_path(self, path):
         if it is not within our git direcotory"""
         if not os.path.isabs(path):
             return path
+        path = os.path.realpath(path)
         relative_path = path.replace(self.repo.working_tree_dir + os.sep, "")
         if relative_path == path:
             raise ValueError("Absolute path %r is not in git repository at %r" % (path, self.repo.working_tree_dir))

git/repo/base.py

@@ -128,14 +128,14 @@ def __init__(self, path=None, odbt=DefaultDBType, search_parent_directories=Fals
         # walk up the path to find the .git dir
         while curpath:
             if is_git_dir(curpath):
-                self.git_dir = curpath
-                self._working_tree_dir = os.path.dirname(curpath)
+                self.git_dir = os.path.realpath(curpath)
+                self._working_tree_dir = os.path.dirname(self.git_dir)
                 break
 
             gitpath = find_git_dir(join(curpath, '.git'))
             if gitpath is not None:
-                self.git_dir = gitpath
-                self._working_tree_dir = curpath
+                self.git_dir = os.path.realpath(gitpath)
+                self._working_tree_dir = os.path.realpath(curpath)
                 break
 
             if not search_parent_directories:

git/test/lib/helper.py

@@ -62,10 +62,14 @@ def wait(self):
 
 
 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