Mail Index

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [ApacheGallery] Cannot see tiff images

[resending after subscribing another address to the list. I trying to
work out if mailman looks at the from or return-path address.
Hopefully /this/ time it will work.]

On Fri, Oct 15, 2004 at 10:29:15AM +0100, Phil wrote:
> With my limited knowlege in this area I suspect that this is probably an
> Imlib2 issue rather than an A::G issue, I hope you don't mind me asking the
> question here.
This is a good place to ask if you are having trouble with A::G if you
are not sure where the problem might be. So you did the right thing.

The problem is with AG.
Here is a diff relative to the subversion HEAD:
--- Apache/   (revision 280)
+++ Apache/   (working copy)
@@ -127,6 +127,7 @@
                my $image_height = $2;

                $filename =~ s/\/(\d+)x(\d+)\-//;
+               $filename =~ s/\-((?i)PPM|TIFF?|GIF)\.jpg$/\.$1/;

                my ($width, $height, $type) = imgsize($filename);

You can apply this by hand by adding the line marked with '+' (without
the '+') at around line 130 of .

This problem is caused by the substitution in get_scaled_picture_name.
The fix is not perfect because I think $type is calculated from looking
at the file contents, not the filename extension. The fix above to find the
original filename from the path_info in ($r->uri =~ m/\.cache\//i)
makes some rather tame assumptions about the file name. A better fix
would be to apply the same substitution in get_scaled_picture_name
whatever the value of $type.

> Looks good
> Creating new 'Build' script for 'Image-Imlib2' version '1.03'
> t/simple....NOK 6#     Failed test (t/simple.t at line 33)
> #     Structures begin differing at:
> #          $got->[3] = '0'
> #     $expected->[3] = '255'
> t/simple....ok 7/7# Looks like you failed 1 tests of 7.
> t/simple....dubious
>         Test returned status 1 (wstat 256, 0x100)
> DIED. FAILED test 6
>         Failed 1/7 tests, 85.71% okay
I'm didn't look into this failed test, but including the output could
have been handy.

28 70 20 71 2C 65 29 61 9C B1 36 3D D4 69 CE 62 4A 22 8B 0E DC 3E
mailto:[email protected]
users mailing list
[email protected]