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: Thu, 2 Jul 2009 09:47:34 +0200
User-agent: Mutt/1.5.18 (2008-05-17)

Hi Chris,

On Thu, Jul 02, 2009 at 07:03:21AM +0200, chris.com wrote:
>
> Hello Henrik,
>
> The behaviour is not "intended" as you so nicely put it.
> It is/was a bug and has been corrected in the 6.4.9-9 version.
> From the ChangeLog:
>
> 2009-03-01  6.4.9-9 Cristy  <address@hidden>
>   * Convert returns MagickFalse for the -version option (reference  
> http://www.imagemagick.org/discourse-server/viewtopic.php?f=2&t=13230).

Thats good news. So I'll let this pass and I'll not invest any time into
working around it.

> I think "conversion -version" is a correct way to check whether  
> ImageMagick is installed.
> autoconf contains a macro AC_PATH_PROG that checks the existence of the  
> executable
> But there is no predefined macro as far as I can see that checks whether  
> a particular version is installed

Every program reports its version in a different format, so there is no
standard way of checking for a version. At least no way that would work 
on different Linux distributions, let alone different Unix versions.

> However I haven't been able to figure out what to do with the IMAGIC  
> variable.
> It seems indeed unused elsewhere.
> I'll try to figure that out next

Don't bother.. it realy isn't used anywhere else. There's only the check
and the output in "configure.ac" which is translated by "autoconf" into the
"configure" script.

But if we ever decide to improve the error handling or even decide to change
the code according to autoconf's findings, it's good to know that we should
be able to rely on "convert -version".

Thank you very much for the research.

cheers
-henrik





reply via email to

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