Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 148038 - darcs-1.0.8 installs bashcomp script in /etc/bash-completion.d
Summary: darcs-1.0.8 installs bashcomp script in /etc/bash-completion.d
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Haskell Language team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-18 03:39 UTC by Henrique Ferreiro
Modified: 2006-10-16 14:11 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Henrique Ferreiro 2006-09-18 03:39:22 UTC
It must be installed in /usr/share/bash-completion.
Comment 1 Duncan Coutts (RETIRED) gentoo-dev 2006-09-18 04:12:57 UTC
Fix pushed to the haskell overlay. I'll get tested and make it into portage from there.
Comment 2 Henrique Ferreiro 2006-09-18 08:30:19 UTC
I tried the ebuild in the overlay and your fix is wrong:

        mkdir "${D}/usr/share/bash-completion/" \
                mv "${D}/etc/bash_completion.d/darcs" \
                   "${D}/usr/share/bash-completion/" \
                && rmdir "${D}/etc/bash_completion.d" && "${D}/etc" \
                || die "fixing location of darcs bash completion failed"

It has to be:

        mkdir "${D}/usr/share/bash-completion/"; \
                mv "${D}/etc/bash_completion.d/darcs" \
                   "${D}/usr/share/bash-completion/" \
                && rmdir "${D}/etc/bash_completion.d" && rmdir "${D}/etc" \
                || die "fixing location of darcs bash completion failed"
Comment 3 Luis Araujo (RETIRED) gentoo-dev 2006-09-18 13:57:12 UTC
I have already fixed this on a new revision bump.

Please test.
Comment 4 Duncan Coutts (RETIRED) gentoo-dev 2006-10-04 09:12:48 UTC
(In reply to comment #3)
> I have already fixed this on a new revision bump.

Araujo, I think the fix is fine. Please commit the new revision to portage (package.masked if you think necessary) so it's easier for other people to test.
Comment 5 Duncan Coutts (RETIRED) gentoo-dev 2006-10-16 14:11:33 UTC
Now in cvs.