Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 65234 - shompatches ebuild typo?
Summary: shompatches ebuild typo?
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High major
Assignee: Jeremy Huddleston (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-24 11:34 UTC by John Richard Moser
Modified: 2004-09-26 08:28 UTC (History)
1 user (show)

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 John Richard Moser 2004-09-24 11:34:57 UTC
src_install() {
    insinto /usr/share/timidity/shampatches  # <--- BORKEN
    doins *.cfg

    # Install patches from subdirectories
    for d in `find . -type f -name \*.pat | sed 's,/[^/]*$,,' | sort -u`; do
        insinto ${instdir}/shompatches/${d}
        doins ${d}/*.pat
    done
}

Does that say 'shampatches' up there?  Should it?  Thankfully this is ~arch right now.

Reproducible: Always
Steps to Reproduce:
1. Look at the shompatches ebuild for timidity-shompatches-19960409-r1.ebuild
Actual Results:  
I found it putting something in /usr/share/timidity/shampatches

Expected Results:  
Everything should go into /usr/share/timidity/shompatches

This is a typo.  Please stab the typodemon.
Comment 1 Jeremy Huddleston (RETIRED) gentoo-dev 2004-09-24 11:43:42 UTC
in portage now, thanks.
Comment 2 Santiago Gala 2004-09-26 08:28:28 UTC
It is still broken.

Now it installs a part of the files in /shompatches

It looks like insinto ${instdir}/shompatches -> /shompatches

I tried substituting ${instdir} -> /usr/share/timidity/shompatches and it seems to work better.