+ def _get_revno(self):
+ # FIXME Can't use this until we move to using bzrlib
+ # directly as it causes to much hassle with FakeExpect
+ revno = self.expect.run("bzr revno %s" % self.working_tree,
+ logfile=file,
+ timeout=None)
+ print revno
+ return revno
I would remove this. We can add it when its necessary. Thanks for
adding the TODO about it in _get_version.
[9]
+ def _get_revno(self): run("bzr revno %s" % self.working_tree,
+ # FIXME Can't use this until we move to using bzrlib
+ # directly as it causes to much hassle with FakeExpect
+ revno = self.expect.
+ logfile=file,
+ timeout=None)
+ print revno
+ return revno
I would remove this. We can add it when its necessary. Thanks for
adding the TODO about it in _get_version.
[10]
+ "now": time.strftime( "%Y%m%d% H%M%S") , date.today( ).strftime( "%Y%m%d" ),
+ "today": datetime.
These should be changed to:
now = datetime.utcnow() "%Y%m%d% H%M%S") ,
"today" : now.strftime( "%Y%m%d" ),
...
"now": now.strftime(
[11]
I have a test failure:
[FAIL]: autoppa. tests.test_ target. BuildTargetTest .test_prepare_ custom_ files_with_ custom_ version_ content
Traceback (most recent call last): python2. 6/unittest. py", line 279, in run jkakar/ src/johnf- inodes/ autoppa/ version- template/ autoppa/ tests/test_ target. py", line 280, in test_prepare_ custom_ files_with_ custom_ version_ content assertEquals( file.read( ), expected) AssertionError: "version = 'AUTOPPA_ VERSION( 1.3.11- upside1~ dapper~ 6.06~20091027~ user_from_ env)'" != "version = 'AUTOPPA_ VERSION( 1.3.11- upside1~ dapper~ 6.06~20091024~ user_from_ env)'"
File "/usr/lib/
testMethod()
File "/home/
self.
exceptions.
Looking great, +1 considering these comments. Have you had a chance
to sign the Canonical contributor agreement yet?