In talking over this with Aaron on IRC, I think a fix like this would be reasonable: === modified file 'bzrlib/repository.py' --- bzrlib/repository.py 2010-05-25 17:27:52 +0000 +++ bzrlib/repository.py 2010-08-06 19:11:55 +0000 @@ -242,7 +242,7 @@
def _gen_revision_id(self): """Return new revision-id.""" - return generate_ids.gen_revision_id(self._config.username(), + return generate_ids.gen_revision_id(self._committer, self._timestamp)
def _generate_revision_if_needed(self):
self._committer is already set to self._config.username() if one is not supplied, and this way if one *is* specified then it gets used.
In talking over this with Aaron on IRC, I think a fix like this would be reasonable: repository. py' repository. py 2010-05-25 17:27:52 +0000 repository. py 2010-08-06 19:11:55 +0000
=== modified file 'bzrlib/
--- bzrlib/
+++ bzrlib/
@@ -242,7 +242,7 @@
def _gen_revision_ id(self) : ids.gen_ revision_ id(self. _config. username( ), ids.gen_ revision_ id(self. _committer,
self. _timestamp)
"""Return new revision-id."""
- return generate_
+ return generate_
def _generate_ revision_ if_needed( self):
self._committer is already set to self._config. username( ) if one is not supplied, and this way if one *is* specified then it gets used.