bug-gnupod
[Top][All Lists]
Advanced

[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





reply via email to

[Prev in Thread] Current Thread [Next in Thread]