Support author and committer options to commit.

dulwich/porcelain.py

@@ -74,16 +74,20 @@ def update_server_info(repo="."):
     server_update_server_info(r)
 
 
-def commit(repo=".", message=None):
+def commit(repo=".", message=None, author=None, committer=None):
     """Create a new commit.
 
     :param repo: Path to repository
     :param message: Optional commit message
+    :param author: Optional author name and email
+    :param committer: Optional committer name and email
+    :return: SHA1 of the new commit
     """
     # FIXME: Support --all argument
     # FIXME: Support --signoff argument
     r = open_repo(repo)
-    r.do_commit(message=message)
+    return r.do_commit(message=message, author=author,
+        committer=committer)
 
 
 def init(path=".", bare=False):
@@ -91,14 +95,15 @@ def init(path=".", bare=False):
 
     :param path: Path to repository.
     :param bare: Whether to create a bare repository.
+    :return: A Repo instance
     """
     if not os.path.exists(path):
         os.mkdir(path)
 
     if bare:
-        Repo.init_bare(path)
+        return Repo.init_bare(path)
     else:
-        Repo.init(path)
+        return Repo.init(path)
 
 
 def clone(source, target=None, bare=False, outstream=sys.stdout):

dulwich/tests/test_porcelain.py

@@ -26,10 +26,6 @@ import tempfile
 
 from dulwich import porcelain
 from dulwich.repo import Repo
-from dulwich.tests.utils import (
-    make_object,
-    make_commit,
-    )
 from dulwich.tests import (
     TestCase,
     )
@@ -80,7 +76,18 @@ class CommitTests(PorcelainTestCase):
         c1, c2, c3 = build_commit_graph(self.repo.object_store, [[1], [2, 1],
             [3, 1, 2]])
         self.repo.refs["refs/heads/foo"] = c3.id
-        porcelain.commit(self.repo.path, message="Some message")
+        sha = porcelain.commit(self.repo.path, message="Some message")
+        self.assertTrue(type(sha) is str)
+        self.assertEquals(len(sha), 40)
+
+    def test_custom_author(self):
+        c1, c2, c3 = build_commit_graph(self.repo.object_store, [[1], [2, 1],
+            [3, 1, 2]])
+        self.repo.refs["refs/heads/foo"] = c3.id
+        sha = porcelain.commit(self.repo.path, message="Some message",
+                author="Joe <joe@example.com>", committer="Bob <bob@example.com>")
+        self.assertTrue(type(sha) is str)
+        self.assertEquals(len(sha), 40)
 
 
 class CloneTests(PorcelainTestCase):