[BUG] bzr upgrade: ERROR: exceptions.TypeError: sequence item 0: expected string, NoneType found

[John Arbash Meinel]

Jean-François Veillette wrote:
> Here is some more info :
> 

Thanks for sending me the contents off list. I was pretty confused until
I did an 'ls' to see what was going on.

The basic result is that you added CVS control directories to all of the
directories inside .bzr/ which confused the upgrade code. It basically
looks in .bzr/revision-store/* to find revision ids that it needs to
upgrade. and CVS/Root made it look like 'Root' was a revision that
needed to be converted.

The latest format should be more resilient to this sort of thing,
because we don't depend on listing the directories as much. (The old
format only listed the directories at certain times, it was something
we've tried to get away from).

So if you do:

find .bzr/ -name CVS | xargs rm -r

Then 'bzr uprade' works fine.

We can add this as a bug if you would like... Though modifying stuff
underneath .bzr/ is one of those things that we say "Don't do that",
because you might break something.

John
=:->

PS> This patch might be useful to prevent the specific traceback that we
were getting. It doesn't really help figuring out what is going on, but
it is still a worthwhile fix.

=== modified file 'bzrlib/errors.py'
--- bzrlib/errors.py    2006-11-02 07:29:02 +0000
+++ bzrlib/errors.py    2006-11-02 19:37:25 +0000
@@ -661,7 +661,8 @@

 class InstallFailed(BzrError):
     def __init__(self, revisions):
-        msg = "Could not install revisions:\n%s" % " ,".join(revisions)
+        revision_str = ", ".join(str(r) for r in revisions)
+        msg = "Could not install revisions:\n%s" % revision_str
         BzrError.__init__(self, msg)
         self.revisions = revisions

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061102/38a12780/attachment.pgp