Can't open .bzr.log in explorer if path contains spaces

Does your login name (losolkowski) has any non-ascii characters in it?

Might just be a quoting problem, if it's really trying to open '"filename"' (with double quotes) rather than 'filename'.

"Bazaar - Explore - System Log" works for me with ascii-only login on Windows XP.

Possible dupe https://bugs.launchpad.net/bzr/+bug/887163

Martin Packman пишет:
> Might just be a quoting problem, if it's really trying to open
> '"filename"' (with double quotes) rather than 'filename'.

Yes, but it works for me with explorer trunk. I should check with
bundled version of explorer.

--
All the dude wanted was his rug back

My login name does not have any non-ascii characters in it. I am running Windows XP. Regarding the possible quoting problem, I don't know how the .bzr.log file path name is generated when the Bazaar Explorer System Log menu item is selected.

Larry, can you attach the section from your .bzr.log that includes the traceback for this error? You'll need to open it from a different program, obviously. :)

OK, I can reproduce it now with changing path to .bzr.log to the path that contains spaces. I think the problem really in the extra quoting. That's issue could be reported before, I don't remember.

Seems that bug 504842 and bug 828803 are of peripheral interest here, though not directly useful for fixing this issue.

Just reverting r449.1.17 from bug 391334 is probably enough to fix this, the code in bzrlib that was working around was fixed several major versions ago. However, not doing argument interpolating in strings like that would be best.

Martin, here's the section from my .bzr.log file, although it sounds like you probably don't need it anymore:

Mon 2011-11-07 10:33:18 -0500
0.141 bazaar version: 2.4.1
0.141 bzr arguments: [u'qviewer', u'"C:\\Documents and Settings\\losolkowski\\My Documents\\.bzr.log"']
0.172 looking for plugins in C:/Documents and Settings/losolkowski/Application Data/bazaar/2.0/plugins
0.172 looking for plugins in C:/Program Files/Bazaar/plugins
0.234 encoding stdout as osutils.get_user_encoding() 'cp1252'
0.453 None

0.453 Traceback (most recent call last):
  File "C:/Program Files/Bazaar/plugins\qbzr\lib\cat.py", line 304, in load
  File "bzrlib\osutils.pyo", line 2184, in file_kind
WindowsError: [Error 123] The filename, directory name, or volume label syntax is incorrect: u'"C:\\Documents and Settings\\losolkowski\\My Documents\\.bzr.log"'

0.453 No Apport available to Bazaar
54.706 return code 0

That's helpful Larry, thanks. It looks like bzrlib should treat ERROR_INVALID_NAME from stat as NoSuchFile in file_kind which would at least give a more useful error message.