FS#19305 - [xdg-utils] Spaces in filename

Attached to Project: Arch Linux
Opened by João Vieira (Vieira) - Saturday, 01 May 2010, 00:16 GMT
Last edited by Pierre Schmitz (Pierre) - Friday, 18 June 2010, 14:02 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jan de Groot (JGC)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:
If you try to run xdg-open with a filename which contains spaces an error will occur.

A patch is available on launchpad https://bugs.launchpad.net/xdg-utils/+bug/220750 (attached here for convenience).

Additional info:
* xdg-utils 1.0.2.20100303-1

Steps to reproduce:
$ xdg-open 1\ 2.jpg
feh WARNING: 1 does not exist - skipping
feh WARNING: 2.jpg does not exist - skipping
feh - No loadable images specified.
Use feh --help for detailed usage information.
This task depends upon

Closed by  Pierre Schmitz (Pierre)
Friday, 18 June 2010, 14:02 GMT
Reason for closing:  Fixed
Comment by João Vieira (Vieira) - Saturday, 01 May 2010, 01:18 GMT
The patch I previously attached (i.e., the one from launchpad) will not run against the current version of xdg-open (1.0.2) because some things changed in the implementation of open_generic(). I made a small change that fixed the problem. Here is my patch.
The *0.5KiB version is the right one*. (The 0.8KiB works but includes useless changes, (white lines, etc) but flyspray won't let me delete it)
Comment by Pierre Schmitz (Pierre) - Monday, 03 May 2010, 17:34 GMT
This shouldn't be patched on our site (or in ubuntu) but fixed upstream. We can then use a new snapshot.
Comment by João Vieira (Vieira) - Tuesday, 04 May 2010, 12:23 GMT
I also submitted the bug and patch upstream, but thought that was a good idea also opening a bug here in case someone facing the same problem want to use the patch while the new version doesn't come out.
Comment by Thomas Dziedzic (tomd123) - Saturday, 05 June 2010, 14:31 GMT
supposedly fixed in 1.0.2, status?
Comment by João Vieira (Vieira) - Thursday, 17 June 2010, 12:01 GMT Comment by Pierre Schmitz (Pierre) - Friday, 18 June 2010, 05:24 GMT
I have put a new snapshot into extra. Please have a look (I cannot test it atm).
Comment by João Vieira (Vieira) - Friday, 18 June 2010, 11:08 GMT
Yes, it's solved in 1.0.2.20100618-1.
Thanks.

Loading...