Re: [Ids-devel] [PATCH] IDS & PerlMagick 5.30

[asja <as...@se...>]

On Mon, Mar 26, 2001 at 06:49:53PM -0500, Caleb Epstein wrote:
> 	Not sure if anyone else has been bitten by this, or if the
> 	latest code handles it, but I just ran nito an issue w/IDS
> 	when run wiht PerlMagick 5.30 (the debian unstable distro is
> 	using this now).

I ran into trouble three days ago installing the latest perlmagick    
which comes with debian/unstable. It was 5.30-5.      
It was not the first time that perlmagick failed to work in my
system right after an upgrade, but this time i noticed something
different:

> 
> 	In the function getImageDimensions() in idsShared.pm, the
> 	Image::Magick::Ping() method is expected to return a
> 	comma-delimited dtring.  It appears in 5.30 that it now
> 	returns an array in array context and only the image FORMAT
> 	when called in scalar context.  The fix is to re-code the
> 	function as follows:
> 
> sub getImageDimensions {
> 	my $imageName = shift(@_);
> 	my $preview = new Image::Magick;
> 	my ($width, $height, $size, $format) = $preview->Ping ($imageName);
> 	if ($width =~ /,/) {
> 	   ($width, $height, $size, $format) = split /,/, $width;
> 	}
> 	return ($width, $height);
> }
> 
> 	This ought to be backwardly compatible with 5.29's behavior.

Yes, this works and it is an elegant solution, but if i downgrade the
perlmagick 5.30-5 back to 5.30-2 (the only official .deb i managed to find
searching around...), the ids' 0.6 original code works perfectly again,
without any modification!

Note that on the perlmagick's page they still show how to parse the Ping()
method the old fashion...                  

So it could be a build-specific problem, not a version-specific one...
However this is really a boring issue: BTW it's the only trouble i got with
debian/unstable in the last year!

Best Regards
.asja