commit.count: is an instance method now

repo: added head , tag and iter_trees methods for completeness
changes: headlines now sorted chronologically

Author: Byron

CHANGES

@@ -45,6 +45,40 @@ objects Package
   not affect you though unless you explicitly imported individual objects. If you just 
   used the git package, names did not change.
 
+Blob
+----
+* former 'name' member renamed to path as it suits the actual data better
+
+Commit
+------
+* 'count' method is not an instance method to increase its ease of use
+
+Config
+------
+* The git configuration can now be read and manipulated directly from within python
+  using the GitConfigParser
+* Repo.config_reader returns a read-only parser
+* Repo.config_writer returns a read-write parser 
+ 
+Diff
+----
+* Members a a_commit and b_commit renamed to a_blob and b_blob - they are populated
+  with Blob objects if possible
+* Members a_path and b_path removed as this information is kept in the blobs
+* Diffs are now returned as DiffIndex allowing to more quickly find the kind of 
+  diffs you are interested in
+  
+Diffing
+-------
+* Commit and Tree objects now support diffing natively with a common interface to 
+  compare agains other Commits or Trees, against the working tree or against the index.
+
+Refs
+----
+* Will dynmically retrieve their object at the time of query to assure the information 
+  is actual. Recently objects would be cached, hence ref object not be safely kept 
+  persistent.
+  
 Repo
 ----
 * Moved blame method from Blob to repo as it appeared to belong there much more.
@@ -73,24 +107,19 @@ Repo
   - archive_tar_gz and archive_tar and replaced by archive method with different signature
 * 'commits' method has no max-count of returned commits anymore, it now behaves 
  like git-rev-list
-* 'untracked_files' property added, returning all currently untracked files
-
-Diff
-----
-* Members a a_commit and b_commit renamed to a_blob and b_blob - they are populated
-  with Blob objects if possible
-* Members a_path and b_path removed as this information is kept in the blobs
-* Diffs are now returned as DiffIndex allowing to more quickly find the kind of 
-  diffs you are interested in
+* The following methods and properties were added
+  - 'untracked_files' property, returning all currently untracked files
+  - 'head', creates a head object
+  - 'tag', creates a tag object
+  - 'iter_trees' method
+  - 'config_reader' property
+  - 'config_writer' property
 
-Diffing
--------
-* Commit and Tree objects now support diffing natively with a common interface to 
-  compare agains other Commits or Trees, against the working tree or against the index.
-
-Blob
-----
-* former 'name' member renamed to path as it suits the actual data better
+Remote
+------
+* Added Remote object allowing easy access to remotes
+* Repo.remotes lists all remotes
+* Repo.remote returns a remote of the specified name if it exists
 
 Tree
 ----
@@ -102,24 +131,6 @@ Tree
 * now mimics behaviour of a read-only list instead of a dict to maintain order.
 * content_from_string method is now private and not part of the public API anymore
 
-Refs
-----
-* Will dynmically retrieve their object at the time of query to assure the information 
-  is actual. Recently objects would be cached, hence ref object not be safely kept 
-  persistent.
-  
-Remote
-------
-* Added Remote object allowing easy access to remotes
-* Repo.remotes lists all remotes
-* Repo.remote returns a remote of the specified name if it exists
-
-Config
-------
-* The git configuration can now be read and manipulated directly from within python
-  using the GitConfigParser
-* Repo.config_reader returns a read-only parser
-* Repo.config_writer returns a read-write parser
 
 0.1.6
 =====

lib/git/objects/commit.py

@@ -101,16 +101,9 @@ def summary(self):
 		"""
 		return self.message.split('\n', 1)[0]
 
-	@classmethod
-	def count(cls, repo, rev, paths='', **kwargs):
+	def count(self, paths='', **kwargs):
 		"""
-		Count the number of commits reachable from this revision
-
-		``repo``
-			is the Repo
-
-		``rev``
-			revision specifier, see git-rev-parse for viable options
+		Count the number of commits reachable from this commit
 
 		``paths``
 			is an optinal path or a list of paths restricting the return value 
@@ -121,7 +114,7 @@ def count(cls, repo, rev, paths='', **kwargs):
 		Returns
 			int
 		"""
-		return len(repo.git.rev_list(rev, '--', paths, **kwargs).strip().splitlines())
+		return len(self.repo.git.rev_list(self.id, '--', paths, **kwargs).strip().splitlines())
 
 	@classmethod
 	def iter_items(cls, repo, rev, paths='', **kwargs):

lib/git/refs.py

@@ -174,7 +174,7 @@ def commit(self):
 		return self.object
 
 
-class TagRef(Reference):
+class TagReference(Head):
 	"""
 	Class representing a lightweight tag reference which either points to a commit 
 	or to a tag object. In the latter case additional information, like the signature
@@ -219,9 +219,9 @@ def tag(self):
 
 
 # provide an alias
-Tag = TagRef
+Tag = TagReference
 
-class RemoteRef(Head):
+class RemoteReference(Head):
 	"""
 	Represents a reference pointing to a remote head.
 	"""

lib/git/remote.py

@@ -8,7 +8,7 @@
 """
 
 from git.utils import LazyMixin, Iterable
-from refs import RemoteRef
+from refs import RemoteReference
 
 class _SectionConstraint(object):
 	"""
@@ -105,7 +105,7 @@ def iter_items(cls, repo):
 		# parse them using refs, as their query can be faster as it is 
 		# purely based on the file system
 		seen_remotes = set()
-		for ref in RemoteRef.iter_items(repo):
+		for ref in RemoteReference.iter_items(repo):
 			remote_name = ref.remote_name
 			if remote_name in seen_remotes:
 				continue
@@ -121,7 +121,7 @@ def refs(self):
 			List of RemoteRef objects
 		"""
 		out_refs = list()
-		for ref in RemoteRef.list_items(self.repo):
+		for ref in RemoteReference.list_items(self.repo):
 			if ref.remote_name == self.name:
 				out_refs.append(ref)
 			# END if names match

lib/git/repo.py

@@ -108,6 +108,20 @@ def heads(self):
 			``git.Head[]``
 		"""
 		return Head.list_items(self)
+
+	# alias heads
+	branches = heads
+
+	@property
+	def head(self,path="HEAD"):
+		"""
+		Return
+			Head Object, reference pointing to commit
+			
+		``path``
+			path to the head or its name, i.e. master or heads/master
+		"""
+		return Head(self,path)
 
 	@property
 	def remotes(self):
@@ -130,18 +144,6 @@ def remote(self, name='origin'):
 		# END for each existing remote
 		raise ValueError( "Remote named %s does not exist" % name )
 
-	# alias heads
-	branches = heads
-
-	@property
-	def head(self):
-		"""
-		Return
-			Head Object, reference pointing to the current head of the repository
-		"""
-		return Head(self,'HEAD')
-			
-
 	@property
 	def tags(self):
 		"""
@@ -150,7 +152,17 @@ def tags(self):
 		Returns
 			``git.Tag[]``
 		"""
-		return Tag.list_items(self)
+		return TagReference.list_items(self)
+		
+	def tag(self,path):
+		"""
+		Return
+			TagReference Object, reference pointing to a Commit or Tag
+		
+		``path``
+			path to the tag reference, i.e. 0.1.5 or tags/0.1.5
+		"""
+		return TagReference(self, path)
 
 	def _get_config_path(self, config_level ):
 		# we do not support an absolute path of the gitconfig on windows , 
@@ -214,6 +226,15 @@ def commit(self, rev=None):
 		c = Object.new(self, rev)
 		assert c.type == "commit", "Revision %s did not point to a commit, but to %s" % (rev, c)
 		return c
+		
+	def iter_trees(self, *args, **kwargs):
+		"""
+		Returns
+			Iterator yielding Tree objects
+			
+		Note: Takes all arguments known to iter_commits method
+		"""
+		return ( c.tree for c in self.iter_commits(*args, **kwargs) )
 
 	def tree(self, ref=None):
 		"""

test/git/test_commit.py

@@ -66,7 +66,7 @@ def test_rev_list_bisect_all(self, git):
 			assert_equal(sha1, commit.id)
 
 	def test_count(self):
-		assert Commit.count( self.repo, '0.1.5' ) == 141
+		assert self.repo.tag('0.1.5').commit.count( ) == 141
 
 	def test_str(self):
 		commit = Commit(self.repo, id='abc')

test/git/test_repo.py

@@ -67,6 +67,16 @@ def test_commits(self, git):
 
 		assert_true(git.called)
 
+	def test_trees(self):
+		mc = 30
+		num_trees = 0
+		for tree in self.repo.iter_trees('0.1.5', max_count=mc):
+			num_trees += 1
+			assert isinstance(tree, Tree)
+		# END for each tree
+		assert num_trees == mc
+			
+
 	@patch_object(Repo, '__init__')
 	@patch_object(Git, '_call_process')
 	def test_init(self, git, repo):
@@ -170,6 +180,9 @@ def test_active_branch(self, git):
 	def test_head(self):
 		assert self.repo.head.object == self.repo.active_branch.object
 
+	def test_tag(self):
+		assert self.repo.tag('0.1.5').commit
+		
 	@patch_object(Git, '_call_process')
 	def test_should_display_blame_information(self, git):
 		git.return_value = fixture('blame')