[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-gnupod] artwork support apparently (but not really) broken on u
From: |
H. Langos |
Subject: |
Re: [Bug-gnupod] artwork support apparently (but not really) broken on ubuntu |
Date: |
Wed, 1 Jul 2009 00:09:36 +0200 |
User-agent: |
Mutt/1.5.18 (2008-05-17) |
Hi Chris,
On Tue, Jun 30, 2009 at 11:08:58PM +0200, chris.com wrote:
> H. Langos wrote:
>> GNUpod's configure script tries to run "convert --version" and checks
>> the returncode to determine if convert from imagemagick is installed.
>>
>> This (accidentially) works as convert accepts "--version" eventhough
>> the correct option would be "-version".
>>
>> On Debian both commands return with 0, which for configure means OK.
>>
>> Apparently imagemagick 6.4 on ubuntu 9.04 prefers not to be detected
>> that way. Eventhough this version also accepts --version and -version,
>> the return code in both cases is 1.
>>
>> This leads configure to the assumption that something is broken and
>> therefore artwork would not work. But as far as I've looked into the
>> code, this finding is not used anywhere in the code. So artwork
>> operations will still be tried and will work as long as imagemagick is
>> installed.
>>
>> Somebody should get in touch with the imagemagic developers and try to
>> find out if that is intended behavior or just an oversight.
>
> I'll contact them
> When you say 'this finding is not used anywhere' what doe you mean with
> "this finding"? The return code or the version?
With "this finding" I ment configure's assumption that imagemagick was not
installed and that gnupod's artwork support would not work, based on the
different return code of convert --version.
As far as I've traced the gnupod configure script, it only sets the variable
IMAGIC to "No (ImageMagick is not installed)" and the varible IMAGIC is
printed at the end of the configure script. No changes in the actual gnupod
code are made. No matter if IMAGIC is set to :-)
If you are curious here's the configure.ac code:
http://git.savannah.gnu.org/cgit/gnupod.git/tree/configure.ac
cheers
-henrik