[MERGE] Consistent SHA1 for testaments, bundle version bump
Aaron Bentley
aaron.bentley at utoronto.ca
Wed Jun 21 17:47:02 BST 2006
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
John Arbash Meinel wrote:
> I actually already merged this as part of my win32 fixes (since I needed
> it to track down the problems).
> But I don't think it will conflict, since I simply applied and committed
> your patch.
Minor conflict. I used different help text in this version.
> So why are you making the StrictTestament use v2.1 rather than v1.1?
It had previously been 2, so I bumped it to 2.1
> It seems like it should either be 'version 2' or it should be version
> '1.1' for .1 meaning 'strict'.
That makes it sound like an ehancement of 1.1, when it's really an
alternative to it. But '2' wasn't something I was completely happy
with, either.
> def _entry_to_line(self, path, ie):
> - l = ie.revision.decode('utf-8') + ' '
> - l += {True: 'yes', False: 'no'}[ie.executable] + ' '
> - l += Testament._entry_to_line(self, path, ie)
> + l = Testament._entry_to_line(self, path, ie)[:-1]
> + l += ' ' + ie.revision.decode('utf-8')
> + l += {True: ' yes\n', False: ' no\n'}[ie.executable]
> return l
> -
> - def as_text_lines(self):
> - lines = ['bazaar-ng testament version 2']
> - lines.extend(Testament.as_text_lines(self)[1:])
> - return lines
>
> Is this the whitespace stuff that you wanted to fix?
The entry_to_line bit was. The as_text_lines was mostly refactoring.
Format 2 had really ugly spacing when I finally saw it using bzr
testament --strict.
> Is it possible to have a STRICT testament test which includes the
> executable bit? This test checks last_modified, and 'executable:no', but
> it doesn't seem to check executable:yes.
Don't see why not.
> Generally +1 from me.
Cool. I'll add that test and send it in.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFEmXgG0F+nu1YWqI0RAtBlAJ99p2W9W6rz2PACvuPW6xr8ASb1cgCfWc8z
8YR+sPam/J8GYn+F72iHXEU=
=3Fvq
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list