Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 417951 - dev-lang/R - bash completion fails with paths containing spaces
Summary: dev-lang/R - bash completion fails with paths containing spaces
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Denis Dupeyron (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: bashcomp-spacepath
  Show dependency tree
 
Reported: 2012-05-28 10:14 UTC by poletti.marco
Modified: 2013-01-29 18:28 UTC (History)
1 user (show)

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


Attachments
Patch for the R.bash_completion file in R-2.10.1 (R.bash_completion.patch,5.08 KB, text/plain)
2012-06-21 21:46 UTC, poletti.marco
Details

Note You need to log in before you can comment on or make changes to this bug.
Description poletti.marco 2012-05-28 10:14:17 UTC
The bash completion installed by dev-lang/R doesn't work for files/directories with spaces.

See Bug 417717 for more details.
Comment 1 Sébastien Fabbro (RETIRED) gentoo-dev 2012-06-19 16:59:22 UTC
hi, i just commited a new version of bash completion on R-2.15. it does not fix the issues mentioned, but if you have a patch or suggestions, please submit them to upstream at:

http://code.google.com/p/rcompletion/

thanks
Comment 2 poletti.marco 2012-06-21 21:46:19 UTC
Created attachment 315943 [details]
Patch for the R.bash_completion file in R-2.10.1

I attach a patch for the bash completion of R-2.10.1.
I looked a bit at the SVN trunk, and the relevant parts of the file seem unchanged.
Tell me if you have trouble applying the patch to the current version.
Comment 3 Denis Dupeyron (RETIRED) gentoo-dev 2012-06-22 16:48:32 UTC
(In reply to comment #2)
> Created attachment 315943 [details]
> Patch for the R.bash_completion file in R-2.10.1
> 
> I attach a patch for the bash completion of R-2.10.1.
> I looked a bit at the SVN trunk, and the relevant parts of the file seem
> unchanged.
> Tell me if you have trouble applying the patch to the current version.

Thanks for the patch. Could you please confirm the issue is not only with R-2.10.1, but also also with other versions? Could you please give us a hint on how to test wether the issue is fixed or still there?

Sébastien, one thing we may want to do is not touch any of the old and/or stable versions, fix R-2.15.0 if necessary, stabilize it, and get rid of all versions older than that. What do you think?

Assuming we go for that I'll adapt Marco's patch for 2.15 and I'll push it.

Thanks,
Denis.
Comment 4 poletti.marco 2012-06-22 16:53:13 UTC
Type

mkdir 'folder with spaces'
R CMD COMPILE folder<TAB>

and it should work with the patch and not otherwise.

I don't know with which versions of R this issue appears, but you can try the above to check.
Comment 5 Sébastien Fabbro (RETIRED) gentoo-dev 2012-06-22 16:59:46 UTC
(In reply to comment #3)

> Sébastien, one thing we may want to do is not touch any of the old and/or
> stable versions, fix R-2.15.0 if necessary, stabilize it, and get rid of all
> versions older than that. What do you think?


although i doubt many users are still on R-2.10.1, i would not touch it and aim at stabilizing 2.15.1 since fixing all test issues on 2.15.0 might not be worth it.

marco, make sure you submit your patch to the url in comment #2.
thanks.
Comment 6 poletti.marco 2012-06-22 18:46:35 UTC
I filed a bugreport upstream:

http://code.google.com/p/rcompletion/issues/detail?id=1&thanks=1&ts=1340390740

I included the patch there.
Comment 7 Sébastien Fabbro (RETIRED) gentoo-dev 2013-01-29 18:28:21 UTC
+  29 Jan 2013; Sébastien Fabbro <bicatali@gentoo.org> R-2.15.2-r1.ebuild:
+  Fixed bash completion spaces (bug #417951, thanks Marco Poletti) and parallel
+  build (bug #395403)
+