Bug 443235 - gpk-install-file does not handle files where the path has a space
Summary: gpk-install-file does not handle files where the path has a space
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: gnome-packagekit
Version: 9
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Robin Norwood
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-04-19 15:11 UTC by Patrick C. F. Ernzer
Modified: 2008-05-19 08:27 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2008-05-19 08:27:52 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
what i'm about to commit (13.73 KB, patch)
2008-04-20 20:50 UTC, Richard Hughes
no flags Details | Diff

Description Patrick C. F. Ernzer 2008-04-19 15:11:31 UTC
Description of problem:
When a user double clicks an RPM in nautilus, the path of which has a space,
package installation fails

Version-Release number of selected component (if applicable):
gnome-packagekit-0.1.12-5.20080416git.fc9.i386

How reproducible:
always

Steps to Reproduce:
1. as user, double-click a file to install which has a space in the path (e.g.:
/media/VMware Tools/VMwareTools-5.5.6-80404.i386.rpm)
2. authenticate as root
  
Actual results:
package installation fails with "The package that is trying to be installed is
already installed."
clicking on "More details" shows "Can't install /media/VMware"

Expected results:
twofold:
  1) an error message that is not misleading
  2) handling of spaces in filenames

Additional info:
file-roller-2.22.1-1.fc9 handles the same file & path just fine (i.e. right
clicking the file and selecting Open with "Archive Manager"). So there must be a
way to deal with the spaces.

Comment 1 Richard Hughes 2008-04-20 19:14:27 UTC
>an error message that is not misleading

Fixed in 8c4a5efbf356a9886cc7dea4893b07dae4bd94c6

I'm now trying to work out where the space issue is. The daemon does the right
thing, so I'm erring on the backend.

Comment 2 Richard Hughes 2008-04-20 20:46:16 UTC
Nope, it's my fault, I'm assuming a space is a delimiter in pk_spawn_command.
I'll fix this now.

Comment 3 Richard Hughes 2008-04-20 20:50:05 UTC
Created attachment 303072 [details]
what i'm about to commit

This is the proper fix.

Comment 4 Richard Hughes 2008-04-20 21:19:37 UTC
Fixed in master and stable, many thanks.

Comment 5 Bug Zapper 2008-05-14 09:44:22 UTC
Changing version to '9' as part of upcoming Fedora 9 GA.
More information and reason for this action is here:
http://fedoraproject.org/wiki/BugZappers/HouseKeeping


Note You need to log in before you can comment on or make changes to this bug.