pcmanfm incorrectly escapes spaces in "Extract Here" with file-roller

Bug #686526 reported by Eliah Kagan
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
PCManFM
Unknown
Unknown
pcmanfm (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: pcmanfm

On a Maverick amd64 system, I created a "/home/ek/dir name with spaces" directory and put a single file, test.tar, inside it. I navigated to this directory in pcmanfm (package version 0.9.7-1ubuntu1), right-clicked on test.tar, and clicked "Extract Here" in the contextual menu. I got a message box from file-roller, saying:

Destination folder "file:///home/ek/dir0name0with0spaces" does not exist.

Do you want to create it?

To double-check that it was pcmanfm, and not file-roller, that was incorrectly escaping spaces in the URI (replacing them with '0' characters instead of with the sequence "%20"), I opened a terminal and ran the command "ps x | grep file-roller | grep -v grep", and got:

29196 ? Sl 0:00 file-roller --extract-to file:///home/ek/dir0name0with0spaces /home/ek/dir name with spaces/test.tar

In Edit > Preferences > Advanced, changing "Archiver integration" from file-roller to xarchiver is a successful workaround to the problem, presumably since pcmanfm doesn't construct a URI when invoking xarchiver.

I am able to reproduce this bug on an i386 Maverick system (same pcmanfm package version) and an i386 Natty system (pcmanfm package version 0.9.8-1).

ProblemType: Bug
DistroRelease: Ubuntu 10.10
Package: pcmanfm 0.9.7-1ubuntu1
ProcVersionSignature: Ubuntu 2.6.35-23.41-generic 2.6.35.7
Uname: Linux 2.6.35-23-generic x86_64
Architecture: amd64
Date: Tue Dec 7 06:43:12 2010
ExecutablePath: /usr/bin/pcmanfm
InstallationMedia: Xubuntu 10.04 "Lucid Lynx" - Beta amd64 (20100406)
Pref_Config_System_Lubuntu:
 [Preferred Applications]
 WebBrowser=firefox.desktop
 MailClient=
ProcEnviron:
 LANGUAGE=en_US.utf8
 LANG=en_US.utf8
 LC_MESSAGES=en_US.utf8
 SHELL=/bin/bash
RelatedPackageVersions:
 libmenu-cache1 0.3.2-2
 pcmanfm 0.9.7-1ubuntu1
 udisks 1.0.1+git20100614-3
 gvfs 1.6.4-0ubuntu1.1
SourcePackage: pcmanfm

Related branches

Revision history for this message
Eliah Kagan (degeneracypressure) wrote :
tags: added: i386 natty
Revision history for this message
jpxsat (jpxsat) wrote :

Confirmed in lubuntu 10.04

Revision history for this message
SteVe Cook (yorvyk) wrote :

Also in Natty with PCmanFM 9.8

Changed in pcmanfm (Ubuntu):
status: New → Confirmed
Revision history for this message
Julien Lavergne (gilir) wrote :
tags: added: lucid
Revision history for this message
Eliah Kagan (degeneracypressure) wrote :

I am able to reproduce this bug from the latest git sources (cloned less than a half hour ago from the posting of this comment) on a Maverick i386 system, so this bug does not appear to be fixed upstream.

This bug seems to be in the way URI's are created to represent filenames that contain spaces. That bug (ID 3096318 upstream on SourceForge) is similar in that it only affects certain applications or file types, but I don't think that bug is the same as this bug. (If it is, then either that bug is not really fixed, or there is some very peculiar about my system. I've installed the git sources on a system that does not otherwise have libfm or pcmanfm installed, so I can be sure that I am really running the latest git version.)

Revision history for this message
Julien Lavergne (gilir) wrote :

Confirmed with lastest git, it doesn't hable space and encoding caracters (like éè...)

Revision history for this message
Julien Lavergne (gilir) wrote :

Forwarding upstream.

Changed in pcmanfm (Ubuntu):
status: Confirmed → Triaged
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pcmanfm - 0.9.8+git-6240436419-0ubuntu1

---------------
pcmanfm (0.9.8+git-6240436419-0ubuntu1) natty; urgency=low

  * New upstream snapshot (2011-02-15)
   - Correctly handle spaces in path and filenames (LP: #686526, #693990).
  * debian/patches/
   - 90-enable-fake-transparency.patch: Remove, merged upstream.
   - 01-libfm-0.1.14-API-changes.patch: Refresh.
  * debian/control
   - Build-depends on libfm-dev (>= 0.1.15).
 -- Julien Lavergne <email address hidden> Mon, 21 Feb 2011 23:18:32 +0100

Changed in pcmanfm (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.