On 2013-02-11 20:43:08, Julien Cristau wrote:
> On Mon, Feb 11, 2013 at 20:15:21 +0100, Anton Gladky wrote:
> 
> > Package: release.debian.org
> > Severity: normal
> > User: release.debian....@packages.debian.org
> > Usertags: unblock
> > 
> > Please unblock package imview
> > 
> > the version fixes RC-Bug #699820 (security issue)
> > and FTBFS on kFreeBSD-systems (it was detected after
> > the version 1.1.9c-10 uploaded).
> > 
> > unblock imview/1.1.9c-11
> 
> The patch from -10 is horrible...  I was told it also wasn't tested on
> real ics images.  Do we actually need to keep this package?

It's also wrong. I looked over it once again and noticed too many
mistakes. In my opinion it should be reverted for now until there is a
working fix for that issue.

In the meanwhile I've also found some ics images in the libics source
package. There is test/testim_c.ics and the corresponding .ids file.
The thing is that I can't even open the file with the imview version
from wheezy. With the patch imview fails differenctly but still doesn't
display the image.

So instead of fixing the ICS support I'd just drop it and if there is
real interest in imview and the capability to read ICS files, the code
should be rewritten to use libics instead. But that's clearly something
for jessie.

I'm sorry for this horrible patch.

Regards
-- 
Sebastian Ramacher

Attachment: signature.asc
Description: Digital signature

Reply via email to