gauret

Uploaded to Gitorious, the project is here: http://gitorious.org/copycover.

- Mar 31 2010

Hi guys,

I'm the original author of the CopyCover script for Amarok 1.4.

I didn't have a need for it these last years, that's why I never ported it to Amarok 2.

This weekend, I suddendly needed it, so I said to myself "why not use this occasion to port CopyCover to Amarok2, at long last ?".

I just spent the last 5 days learning QTJambi and the Amarok APIs. Now that I have finished porting the script, I come here to update the page (feeling like I did a good deed) and find this...

Not only had someone already done the work, but TWO people had ported the script already !!

Please give me a few minutes while I jump from the nearest building.

Seriously, I'm deeply sorry for not porting the script earlier and causing you guys to do duplicated work.

I was going to upload my version to Gitorious, maybe there's something to salvage nonetheless ? I'll go and do that, you'll tell me if it's worth something.

Thanks to you for doing what I've been too lazy to do for 2 years.

If that doesn't teach me to do my research before picking up any significant piece of work...

By the way, funny thing that we're all french :) - Mar 31 2010

You have to set your DISPLAY variable. Add "DISPLAY=:0" in front of the command call, that should do it (it's a common problem: using cron with graphical apps. I'm sure you favorite search engine will help you) - Sep 26 2007

Ah, that's interesting. Are you using MySQL as a backend ? Do all those who see this "0-as-a-dir" bug use MySQL too ? - Mar 05 2007

Do you really have an artist called "A" in your collection ? Who could that be ?

Could you paste a sample of the following command please ?
dcop amarok collection query "SELECT DISTINCT album.name, devices.lastmountpoint || '/' || tags.dir FROM tags, album, devices WHERE tags.album = album.id AND tags.deviceid = devices.id AND album.name != '' AND tags.artist = A"

Thanks - Feb 21 2007

Yeah, my time machine is still in beta. I'll fix it, thanks for reporting :) - Oct 19 2006

Thanks, I've merged your patch.

The scripts works for my current amarok, which is 1.4.3. If it stops working when dynamic collections are added, I'll update it (but please remind me :) )
In this next version, your patch will be there. - Oct 13 2006

Yep, I'm using amarok 1.4 beta 3 right now and it works fine. - Apr 20 2006

It should only depend on PyQT (which is installed by default on SuSE IIRC), but it will warn you it it is not available.

Remember that the cover will not be copied if there is already an image file in the folder, png, jpg or gif (check for hidden files). - Feb 13 2006

I'm running amarok 1.3.8 and it seems to work fine here. Do you have interesting messages in your ~/.xsession-errors ? - Feb 12 2006

> BTW, one other thing I noticed: I hade some albums of german artists whose
> names containd non english characters. CopyCover wouldn't copy the artwork to
> those folders too.

Hmm, I have albums containing non-ascii characters too (é, ö, ...), and CopyCover works for them. Which artists for example ?

> But this is not the case with my various artists albums.

Hmm, I'll have to look at that. Maybe it's handled differently in the database. - Oct 03 2005

Thanks for the report.
The script does not copy an album to a directory if there is already a picture in this directory. Could that be the reason ? - Oct 03 2005

Right now, this is not possible, mainly because I have no idea how to do that, but also because it would probably bloat the mp3 files (the image only needs to be once in the album dir, not in every song in it) - Aug 23 2005

To reset the directory icon, juste delete the (hidden) .directory file in the directory. If you have to do it for your whole collection, a simple search for these files should do it.

Concerning the album in konqueror's tooltip, I don't know how to do that (yet), but if I have time I'll look into it. - Jul 03 2005

Those bugs should be fixed in version 1.6
Thanks for your report - Jun 05 2005

Thanks for your support !
I'll probably add a check for write permissions in the future, it's a good idea. - Apr 27 2005

There are two ways to deal with that. Either I allow the script to overwrite the existing cover, but some users may not like it at all, or I let the user just delete the old cover, and it will be replaced automatically.

For now, I chose the second option. If I add your feature, there will be a checkbox to enable it anyway.

So for now, if you want to replace an existing cover, just delete it, and it will be copied from the database on the next play. - Apr 26 2005

Fixed in version 1.3, thanks for the report ! - Apr 23 2005

Yes I saw that just after I uploaded it. It's fixed now, sorry. I secretely hoped noone had noticed... ;o) - Apr 08 2005

OK, I've made a script to do what you'd like. It requires amaroK >= 1.2.3 and python >= 2.3. If you have an older python, I could lower the dependency easily, but I think that all the latest distributions have it anyway (well, except Woody of course... ;o) )
amaroK >= 1.2.3 is stricly required however.
The script is available on this page. Run it with --help to see what's available, and run it with -d first to see what actions will be performed. - Apr 07 2005

Yes, I said "now with 1.3" but I meant "now with (amarok) 1.2.3".
I still can't find a way to find the (artist,album) couple when I only have the album filename (which is something like 100@, see in .kde/share/apps/amarok/albumcovers/large). Anyway, I'll figure that out eventually. Thanks Markey - Apr 03 2005

Well, I wanted to do that at first, but amarok had no way to do that. Now, with 1.3 and the new dcop query to the database, it may be possible.
However, I need to figure out a way to do that, since the database does not seem to contain the images downloaded from the web, but only the ones in the music directory. - Apr 02 2005

OK. How about simply removing the front.jpg and back.jpg images if you don't want them ? A simple "find" command would do the trick :
find /my/music/folder -iname 'front.jpg' -o -iname 'back.jpg' | xargs rm -f - Apr 01 2005

No, that's all, you're all set. Play a song for which you have an album cover, and see if the picture is copied to the directory.
I'll look at the PyQt warning problem soon. - Apr 01 2005

Thanks !
If PyQt is not installed, a popup should tell you to install it when you run the script. Is is not the case ? - Apr 01 2005

Well, since it's GPL you can do that if you want of course. It would be pretty easy to do by the way, just remove the checks in the copyCover function. - Apr 01 2005

Well, this was such a good idea that I've just implemented it :)
The .directory file is only created if a cover has been copied in the folder. - Apr 01 2005

Excellent idea ! I'll do that when I have the time (which probably means "this weekend", but no promise) - Apr 01 2005

\_o<~   Ko1n !!

PAN ! - Nov 05 2006

I've installed syncekonnector, but in the configure dialog I can't check Appointment, nor Contact, nor Todo, etc...
I always pops up a box with "No synchronizer found for Appointment" - Feb 23 2006

The build fails with GCC4. I get this error on FC4:
matchmaker.h:91: error: ISO C++ forbids declaration of 'RRA_Matchmaker' with no type
matchmaker.h:91: error: expected ';' before '*' token

Any idea ? After some searching is should be some kind of missing "forward declaration", but I don't know C++... - Nov 26 2005

I'm having the same problem. I'm using a package of qt which does not include .la files, and this is the first time I have a problem. On top of that, a grep for libqt-mt.la in amarok's source doesn't return anything
Here's the output :
==============
/bin/sh ../../libtool --silent --mode=link --tag=CXX g++ -Wnon-virtual-dtor -Wno-long-long -Wundef -ansi -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wcast-align -Wconversion -Wchar-subscripts -Wall -W -Wpointer-arith -Wwrite-strings -O2 -Wformat-security -Wmissing-format-attribute -fno-exceptions -fno-check-new -fno-common -DQT_CLEAN_NAMESPACE -DQT_NO_ASCII_CAST -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -o amarokapp -L/usr/X11R6/lib -L/usr/lib/qt-3.2/lib -L/usr/lib -R /usr/lib -R /usr/lib/qt-3.2/lib -R /usr/X11R6/lib playlistwidget.o playlisttooltip.o playlistloader.o playlistitem.o playlistbrowser.o browserwin.o streambrowser.o expandbutton.o filebrowser.o kbookmarkhandler.o amarokfilelist.o playerwidget.o osd.o effectwidget.o amarokslider.o amaroksystray.o playerapp.o fht.o amarokconfigdialog.o threadweaver.o Options1.o Options2.o Options3.o Options4.o Options5.o ../../amarok/amarok/amarokcore/libamarokcore.a ../../amarok/amarok/engine/libengine.a ../../amarok/amarok/titleproxy/libtitleproxy.a ../../amarok/amarok/analyzers/libanalyzers.a ../../amarok/amarok/amarokarts/libamarokarts.la -lqtmcop -lkmedia2_idl -lartsflow -lsoundserver_idl -lartskde -lartsgui -lartsgui_kde -lartsmodules -lkutils -lkio -lkdeui -lkdecore -L/usr/lib -ltag -lGL

*** Warning: Linking the executable amarokapp against the loadable module
*** libamarokarts.so is not portable!
libtool: link: cannot find the library `/usr/lib/qt-3.2/lib/libqt-mt.la'
============

Any idea ? - Mar 02 2004