Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 21174

Summary: ebuild for mp3mover.pl Perl script
Product: Gentoo Linux Reporter: Nick Fish <ennui>
Component: New packagesAssignee: Gentoo Sound Team <sound>
Status: RESOLVED INVALID    
Severity: enhancement CC: perl, rac
Priority: High Keywords: EBUILD
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 21117    
Bug Blocks:    
Attachments: the ebuild
md5 digest and stuff

Description Nick Fish 2003-05-17 17:18:32 UTC
mp3mover.pl is a Perl script which will rename or move MP3 files into different
directories/names per their ID3 tag.
Comment 1 Nick Fish 2003-05-17 17:19:12 UTC
Created attachment 12102 [details]
the ebuild
Comment 2 Nick Fish 2003-05-17 17:19:32 UTC
Created attachment 12103 [details]
md5 digest and stuff
Comment 3 rob holland (RETIRED) gentoo-dev 2003-06-03 08:12:55 UTC
Thanks for submitting this. It needs some work before it can be committed though.

Your IUSE line lists oggvorbis but doesn't change dependancies or the way the packages is built when USE=oggvorbis is set (or not). It also implies a flac USE flag in DEPEND but doesn't list flac in IUSE.

If you'd like to try and fix it, please do so. Otherwise I'll come back to this and fix it up in a few days.

Thanks again for submitting it :)
Comment 4 rob holland (RETIRED) gentoo-dev 2003-06-03 08:35:33 UTC
please ignore my comments, not sure how I managed it but it was meant for a different bug :)
Comment 5 rob holland (RETIRED) gentoo-dev 2003-10-17 02:10:14 UTC
No time for gentoo dev stuff right now :(
Comment 6 Brian Harring (RETIRED) gentoo-dev 2004-02-06 00:41:15 UTC
What version is this ebuild for?
Comment 7 Nick Fish 2004-02-10 17:07:10 UTC
mp3mover-2.1.1 .  Might not be pertinent to add to portage now since no one else has bumped this since I originally posted it.
Comment 8 Michael Cummings (RETIRED) gentoo-dev 2004-09-04 05:45:02 UTC
I'm going to close this out as INVALID, but only because that's the closes I can get to the real situation. I worked with the ebuild this morning, massaging the authors default work into something more conducive to a distribution. My edits to the ebuild included:
src_compile() {
        sed -i -e "s|MP3 /usr/bin|MP3 ${D}/usr/local/lib/mp3mover|g" install.sh
        sed -i -e "s|/usr/bin|${D}/usr/local/bin|g" install.sh
        sed -i -e 's|push @INC,dirname($0);|use lib "/usr/local/lib/mp3mover";|'
 mp3mover.pl
}
src_install() {
        dodir "/usr/local/bin"
        dodir "/usr/local/lib/mp3mover"
        ./install.sh


This was to overcome having the author's custom modules placed in the heart of /usr/bin, an altogether unacceptable action. Even so, the tk module that he employs is error-prone (anchor on line 68 needs to be removed - invalid syntax), and even once launched - it doesn't work. I've tried the various options for getting it to read a directory, and ended up with nothing. If someone else later down the line wants to try and pick this up again, or if the author comes out with a more coherent version in the future, we can reopen this bug, but for now I'm going to have to close it out because the app really doesn't work. Sorry to close this out for you sound team, but since no one else has touched this bug in 7 months, I'm not thinking you will protest.