Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-31 Thread Reuben Thomas
On 31 January 2011 11:47, Julien Cristau wrote: > On Mon, Jan 31, 2011 at 11:34:57 +, Reuben Thomas wrote: >> 1. Remove psiconv just for squeeze. >> > We've done 1 for now, the rest is not up to the release team. In that case, would you be kind enough to build a backport for squeeze? Who know

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-31 Thread Julien Cristau
On Mon, Jan 31, 2011 at 11:34:57 +, Reuben Thomas wrote: > On 25 January 2011 20:43, Adam D. Barratt wrote: > > user release.debian@packages.debian.org > > usertag 609535 + squeeze-will-remove > > thanks > > Please see bug #611260: the major part of this is in fact a bug in > graphicsmag

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-31 Thread Reuben Thomas
On 25 January 2011 20:43, Adam D. Barratt wrote: > user release.debian@packages.debian.org > usertag 609535 + squeeze-will-remove > thanks Please see bug #611260: the major part of this is in fact a bug in graphicsmagick, not in psiconv. With my patch, and a fixed graphicsmagick, psiconv wor

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-25 Thread Adam D. Barratt
user release.debian@packages.debian.org usertag 609535 + squeeze-will-remove thanks On Fri, 2011-01-21 at 15:11 +0100, Jakub Wilk wrote: > * Reuben Thomas , 2011-01-19, 19:05: > >you can always build against ImageMagick instead. > > I played a little bit more and unfortunately psiconv rebuilt

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-21 Thread Jakub Wilk
* Reuben Thomas , 2011-01-19, 19:05: you can always build against ImageMagick instead. I played a little bit more and unfortunately psiconv rebuilt against ImageMagick sometimes produces different images that the one from lenny. For examples/Clipart, the latter produces 71-page TIFF, while t

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-19 Thread Reuben Thomas
On 18 January 2011 12:15, Jakub Wilk wrote: > * Julien Cristau , 2011-01-18, 12:50: >>> >>> This doesn't look good. GetMagickFileList is called in a loop and >>> InitializeMagick is supposed to be run only once. >>> >> That sounds like broken API for a library.  How is a library using >> graphicsm

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-18 Thread Jakub Wilk
* Julien Cristau , 2011-01-18, 12:50: This doesn't look good. GetMagickFileList is called in a loop and InitializeMagick is supposed to be run only once. That sounds like broken API for a library. How is a library using graphicsmagick to know whether it needs to call InitializeMagick or it's a

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-18 Thread Julien Cristau
On Tue, Jan 11, 2011 at 14:55:21 +0100, Jakub Wilk wrote: > This doesn't look good. GetMagickFileList is called in a loop and > InitializeMagick is supposed to be run only once. > That sounds like broken API for a library. How is a library using graphicsmagick to know whether it needs to call In

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-14 Thread Jakub Wilk
* Reuben Thomas , 2011-01-10, 12:48: --- psiconv-0.9.8/configure.in 2005-11-15 15:43:19.0 + +++ psiconv-0.9.8-rrt/configure.in 2011-01-10 12:35:58.0 + @@ -112,6 +112,7 @@ #include #include int main(void) { Exce

Processed: Re: Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org: > retitle 609535 psiconv: magick/semaphore.c:526: LockSemaphoreInfo: Assertion > `semaphore_info->signature == 0xabacadabUL' failed. Bug #609535 [psiconv] psiconv: Patch for GraphicsMagick API change causing crash on startup Changed Bug title to '

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Jakub Wilk
retitle 609535 psiconv: magick/semaphore.c:526: LockSemaphoreInfo: Assertion `semaphore_info->signature == 0xabacadabUL' failed. tags 609535 - patch thanks Thanks for your bugreport! (I'm not maintainer of this package, but I'll comment anyway.) * Reuben Thomas , 2011-01-10, 12:48: --- psico

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Jakub Wilk
* Reuben Thomas , 2011-01-11, 15:15: The traceback you give is identical with that obtained if the patch is not applied. At least on my machine, they are different. When I run "psiconv examples/Clipart" I get: #0 0xf76dc537 in raise () from /lib/libc.so.6 #1 0xf76df922 in abort () from /li

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Reuben Thomas
On 11 January 2011 16:16, Jakub Wilk wrote: > * Reuben Thomas , 2011-01-11, 15:15: >> >> GetMagickFileList is only called in the loop initializer, and the loop is >> only run once. > > You are right, thanks for correction. Still, it's quite a surprising place > to put InitializeMagick into. Well,

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Jakub Wilk
* Reuben Thomas , 2011-01-11, 15:15: GetMagickFileList is only called in the loop initializer, and the loop is only run once. You are right, thanks for correction. Still, it's quite a surprising place to put InitializeMagick into. And in fact, after applying your patch, psiconv still crashes

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-11 Thread Reuben Thomas
On 11 January 2011 13:55, Jakub Wilk wrote: > retitle 609535 psiconv: magick/semaphore.c:526: LockSemaphoreInfo: Assertion > `semaphore_info->signature == 0xabacadabUL' failed. > tags 609535 - patch > thanks > > Thanks for your bugreport! (I'm not maintainer of this package, but I'll > comment any

Bug#609535: psiconv: Patch for GraphicsMagick API change causing crash on startup

2011-01-10 Thread Reuben Thomas
Package: psiconv Version: 0.9.8-4.1 Severity: grave Tags: patch Justification: renders package unusable Recent versions of GraphicsMagick require the InitializeMagick API to be called: 1.3.8 (January 21, 2010) […] Behavior Changes: * InitializeMagick() MUST be invoked