Rev 1544: Support bzr sign-my-commits. in http://people.samba.org/bzr/jelmer/bzr-svn/trunk

[Jelmer Vernooij]

At http://people.samba.org/bzr/jelmer/bzr-svn/trunk

------------------------------------------------------------
revno: 1544
revision-id: jelmer at samba.org-20080803000200-1eo725olflcx2b9c
parent: jelmer at samba.org-20080802234133-84ivpqdwrmqootat
committer: Jelmer Vernooij <jelmer at samba.org>
branch nick: 0.4
timestamp: Sun 2008-08-03 02:02:00 +0200
message:
  Support bzr sign-my-commits.
modified:
  NEWS                           news-20061231030336-h9fhq245ie0de8bs-1
  repository.py                  repository.py-20060306123302-1f8c5069b3fe0265
=== modified file 'NEWS'
--- a/NEWS	2008-08-02 23:41:33 +0000
+++ b/NEWS	2008-08-03 00:02:00 +0000
@@ -48,6 +48,9 @@
 
    * Support bzr uncommit.
 
+   * Support bzr sign-my-commits. (Requires revision properties in the 
+	 Subversion repository to be mutable)
+
   BUG FIXES
 
    * Now uses absolute imports and no longer adds plugin directory to the 

=== modified file 'repository.py'
--- a/repository.py	2008-08-02 21:19:27 +0000
+++ b/repository.py	2008-08-03 00:02:00 +0000
@@ -243,7 +243,6 @@
 
     def lock_write(self):
         """See Branch.lock_write()."""
-        # TODO: Obtain lock on the remote server?
         if self._lock_mode:
             assert self._lock_mode == 'w'
             self._lock_count += 1
@@ -251,12 +250,24 @@
             self._lock_mode = 'w'
             self._lock_count = 1
 
+    def is_write_locked(self):
+        return (self._lock_mode == 'w')
+
     def get_latest_revnum(self):
         if self._lock_mode in ('r','w') and self._cached_revnum:
             return self._cached_revnum
         self._cached_revnum = self.transport.get_latest_revnum()
         return self._cached_revnum
 
+    def start_write_group(self):
+        pass
+
+    def abort_write_group(self):
+        pass
+
+    def commit_write_group(self):
+        pass
+
     def item_keys_introduced_by(self, revision_ids, _files_pb=None):
         fileids = {}
 
@@ -749,7 +760,10 @@
 
     def add_signature_text(self, revision_id, signature):
         (path, revnum, mapping) = self.lookup_revision_id(revision_id)
-        self.transport.change_rev_prop(revnum, SVN_REVPROP_BZR_SIGNATURE, signature)
+        try:
+            self.transport.change_rev_prop(revnum, SVN_REVPROP_BZR_SIGNATURE, signature)
+        except SubversionException, (_, errors.ERR_REPOS_DISABLED_FEATURE):
+            raise errors.RevpropChangeFailed(SVN_REVPROP_BZR_SIGNATURE)
 
     @needs_read_lock
     def find_branches(self, using=False, layout=None):