Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 100953 - abcde editing CDDB data fails with spaces in path
Summary: abcde editing CDDB data fails with spaces in path
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Ryan Phillips (RETIRED)
URL:
Whiteboard:
Keywords: Inclusion
Depends on:
Blocks:
 
Reported: 2005-07-31 18:35 UTC by chris-gentoo
Modified: 2005-08-27 13:37 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to fix problem (abcde-spaces.patch,418 bytes, patch)
2005-07-31 18:37 UTC, chris-gentoo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description chris-gentoo 2005-07-31 18:35:19 UTC
When choosing to edit the CDDB data, the path is not correctly passed to the 
editor if the path contains a space. 

Reproducible: Always
Steps to Reproduce:
1.mkdir "/tmp/some cd" 
2.cd "/tmp/some cd" 
3.abcde 
4.Edit selected CDDB data? [y/n] (n): y 
 
Actual Results:  
The editor specified by $EDITOR is launched with the path 
/tmp/some 

Expected Results:  
The editor specified by $EDITOR is launched with the path 
/tmp/some cd
Comment 1 chris-gentoo 2005-07-31 18:37:01 UTC
Created attachment 64840 [details, diff]
Patch to fix problem

Patched against the /usr/bin/abcde provided by
media-sound/abcde-2.1.19
Comment 2 Ryan Phillips (RETIRED) gentoo-dev 2005-08-01 17:00:11 UTC
Chris: have you pushed this patch upstream?
Comment 3 chris-gentoo 2005-08-02 00:30:55 UTC
I wasn't sure what the policy was with pushing these up-stream. But since you 
asked, I've sent an email to the maintainer with the patch. 
Comment 4 Joshua Jackson (RETIRED) gentoo-dev 2005-08-20 06:30:25 UTC
it appears the patch got in on version 2.3.0. Once its in the tree this should
be able to be closed
Comment 5 Ryan Phillips (RETIRED) gentoo-dev 2005-08-27 13:37:24 UTC
bumped the abcde ebuild