57392 - LXR doesn't find files with a space in the name.

Status: RESOLVED FIXED

(Webtools Graveyard :: MXR, defect, P3)

Description

[David Krause]

LXR doesn't show this file when doing the above query:
http://lxr.mozilla.org/seamonkey/source/embedding/browser/powerplant/PP%20Browser.mcp

It does however find directories with a space in them.

Comment 1

[Dawn Endico]

files checked in to the tree shouldn't have spaces in their file
names. This file was initially checked in by conrad. Could you
please rename the file?

reassigning to conrad.

Comment 2

[timeless]

Endico, this bug's for you. we already have bugs on the files themselves. This 
one was filed for lxr/bonsai robustness.  Please reaccept and mark wontfix if 
you feel that is the appropriate solution.

Comment 3

[David Krause]

Endico, this bug was for you so that LXR could be fixed to not choke on file
names with spaces.  Bug 57389 was already filed for the specific file in the
tree with a space.  I will assign that bug to Conrad.

Comment 4

[timeless]

if find were patched to use 'directory\ restofdirectory/file\ spaced.extention' 
format? and would that fix the problem w/ some scripts?

Comment 5

[timeless]

*** Bug 92371 has been marked as a duplicate of this bug. ***

Comment 6

[timeless]

Attachment: handle +

Comment 7

[timeless]

Comment on attachment 136547 [details] [diff] [review]
handle +

http://landfill.mozilla.org/mxr-test/seamonkey/find?string=+

Comment 8

[Christian Reis]

Comment on attachment 136547 [details] [diff] [review]
handle +

I wonder if we would be better off urldecoding this thing..

This patch changes the search semantics slightly, since +s were originally
ignored (i.e. searching for "Mozilla+Browser" returns files with
"MozillaBrowser" in them, but I don't see that as a valid or expected use case
so I'm okay with it going away.

Comment 9

[timeless]

mozilla/webtools/lxr/find 	1.8