SymbolicReferences can now be at any path within the repository, there is no restriction anymore.

Added a test to assure the git commands can handle it

3 files changed, 21 insertions, 24 deletions

diff --git a/lib/git/cmd.py b/lib/git/cmd.py
index bccfb611..6ee8f355 100644
--- a/lib/git/cmd.py
+++ b/lib/git/cmd.py
@@ -344,7 +344,7 @@ class Git(object):
 		"""
 		tokens = header_line.split()
 		if len(tokens) != 3:
-			raise ValueError("SHA named %s could not be resolved" % tokens[0] )
+			raise ValueError("SHA named %s could not be resolved, git returned: %r" % (tokens[0], header_line.strip()) )
 		if len(tokens[0]) != 40:
 			raise ValueError("Failed to parse header: %r" % header_line) 
 		return (tokens[0], tokens[1], int(tokens[2]))
diff --git a/lib/git/index.py b/lib/git/index.py
index 61307bb8..f4aa26c7 100644
--- a/lib/git/index.py
+++ b/lib/git/index.py
@@ -1039,10 +1039,6 @@ class IndexFile(LazyMixin, diff.Diffable):
 			If None, all paths in the index will be checked out. Otherwise an iterable
 			of relative or absolute paths or a single path pointing to files or directories
 			in the index is expected.
-			The command will raise of files or directories do not exist in the index 
-			( as opposed to the  original git command who ignores them ). Additionally 
-			this command allows to checkout directories which is an extension to git-update-index.
-			
 			
 		``force``
 			If True, existing files will be overwritten even if they contain local modifications. 
@@ -1064,6 +1060,8 @@ class IndexFile(LazyMixin, diff.Diffable):
 		Raise CheckoutError
 			If at least one file failed to be checked out. This is a summary, 
 			hence it will checkout as many files as it can anyway.
+			If one of files or directories do not exist in the index 
+			( as opposed to the  original git command who ignores them ). 
 			Raise GitCommandError if error lines could not be parsed - this truly is 
 			an exceptional state
 		"""
diff --git a/lib/git/refs.py b/lib/git/refs.py
index b0996878..3d9fa226 100644
--- a/lib/git/refs.py
+++ b/lib/git/refs.py
@@ -43,7 +43,7 @@ class Reference(LazyMixin, Iterable):
 		return '<git.%s "%s">' % (self.__class__.__name__, self.path)
 		
 	def __eq__(self, other):
-		return self.path == other.path and self.object == other.object
+		return self.path == other.path
 		
 	def __ne__(self, other):
 		return not ( self == other )
@@ -203,34 +203,29 @@ class SymbolicReference(object):
 	
 	A typical example for a symbolic reference is HEAD.
 	"""
-	__slots__ = ("repo", "name")
+	__slots__ = ("repo", "path")
 	
-	def __init__(self, repo, name):
-		if '/' in name:
-			# NOTE: Actually they can be looking like ordinary refs. Theoretically we handle this
-			# case incorrectly
-			raise ValueError("SymbolicReferences are not located within a directory, got %s" % name)
-		# END error handling 
+	def __init__(self, repo, path):
 		self.repo = repo
-		self.name = name
+		self.path = path
 		
 	def __str__(self):
-		return self.name
+		return self.path
 		
 	def __repr__(self):
-		return '<git.%s "%s">' % (self.__class__.__name__, self.name)
+		return '<git.%s "%s">' % (self.__class__.__name__, self.path)
 		
 	def __eq__(self, other):
-		return self.name == other.name
+		return self.path == other.path
 		
 	def __ne__(self, other):
 		return not ( self == other )
 		
 	def __hash__(self):
-		return hash(self.name)
+		return hash(self.path)
 		
 	def _get_path(self):
-		return join_path_native(self.repo.path, self.name)
+		return join_path_native(self.repo.path, self.path)
 		
 	def _get_commit(self):
 		"""
@@ -311,7 +306,7 @@ class SymbolicReference(object):
 		# checking
 		# Otherwise we detach it and have to do it manually
 		if write_value.startswith('ref:'):
-			self.repo.git.symbolic_ref(self.name, write_value[5:])
+			self.repo.git.symbolic_ref(self.path, write_value[5:])
 			return 
 		# END non-detached handling 
 		
@@ -346,6 +341,10 @@ class SymbolicReference(object):
 		Return
 			Instance of SymbolicReference or HEAD
 			depending on the given path
+			
+		Note
+			It enforces that symbolic refs in git are only found in the 
+			root of the .git repository, never within a folder.
 		"""
 		if not path:
 			raise ValueError("Cannot create Symbolic Reference from %r" % path)
@@ -366,10 +365,10 @@ class HEAD(SymbolicReference):
 	_HEAD_NAME = 'HEAD'
 	__slots__ = tuple()
 	
-	def __init__(self, repo, name=_HEAD_NAME):
-		if name != self._HEAD_NAME:
-			raise ValueError("HEAD instance must point to %r, got %r" % (self._HEAD_NAME, name))
-		super(HEAD, self).__init__(repo, name)
+	def __init__(self, repo, path=_HEAD_NAME):
+		if path != self._HEAD_NAME:
+			raise ValueError("HEAD instance must point to %r, got %r" % (self._HEAD_NAME, path))
+		super(HEAD, self).__init__(repo, path)
 	
 	
 	def reset(self, commit='HEAD', index=True, working_tree = False,