Can't view branch/repository in the root of drive/filesystem

[Adam Glauser]

Steps to recreate:

bzr init-repo C:/
bzr init C:/devel
bzr explore C:/devel

Using BZR_PDB I can see that explorer crashed because it tried to work with URL like this: "file:///C:" which is incorrect because it lacks trailing slash.

So, locations.py: display_name() uses .rstrip('/') for all URLs which is wrong thing is branch or repo resides in the root of the drive, either on Windows or Linux.

[Alexander Belchenko (bialix)]

What URL exactly are you using when get this error?

What it means: "Map a network drive to a shared repository directory (can be on \\localhost)"?

[Adam Glauser (adamglauser)]

I am not using a URL directly, but browsing to the mapped drive.

To be more clear, "Map a network drive to a shared repository directory (can be on \\localhost)" means to set a drive letter to refer to a network location, as in:
net use Z: \\servername\sharename

See attachment 'mapDriveSteps.txt' for a full example.

[Adam Glauser (adamglauser)]

FWIW, this is still happening in bzrExplorer 0.11.0dev (revision 351)
see attachment for updated stack trace.

[Alexander Belchenko (bialix)]

It's unrelated to mapped drive, and can be reproduced even on C: drive.

To reproduce:

bzr init-repo C:/
bzr init C:/devel
bzr explore C:/devel

Using BZR_PDB I can see that explorer crashed because it tried to work with URL like this: "file:///C:" which is incorrect because it lacks trailing slash.

[Alexander Belchenko (bialix)]

So, locations.py: display_name() uses .rstrip('/') for all URLs which is wrong thing is branch or repo resides in the root of the drive, either on Windows or Linux.

[Alexander Belchenko (bialix)]

Workaround: don't put the repo/branch in the root of the filesystem until this bug will be fixed.

[Ian Clatworthy (ian-clatworthy)]

Please test rev 416 and close this bug if everything is working now as expected now

[Alexander Belchenko (bialix)]

It does not crash now, but branch opened has the tab with label: "devel []". I think it should be at least "devel [/]" if not "devel [C:/]". But the original bug is fixed. It's just UI thing.