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 a...@adam-barratt.org.uk 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

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 a...@adam-barratt.org.uk 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

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 jcris...@debian.org 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

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 r...@sc3d.org, 2011-01-19, 19:05: you can always build against ImageMagick instead. I played a little bit more and unfortunately

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

2011-01-21 Thread Jakub Wilk
* Reuben Thomas r...@sc3d.org, 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

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 jw...@debian.org wrote: * Julien Cristau jcris...@debian.org, 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

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

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

2011-01-18 Thread Jakub Wilk
* Julien Cristau jcris...@debian.org, 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

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

2011-01-14 Thread Jakub Wilk
* Reuben Thomas r...@sc3d.org, 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 time.h #include magick/api.h

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 jw...@debian.org 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

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

2011-01-11 Thread Jakub Wilk
* Reuben Thomas r...@sc3d.org, 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

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 jw...@debian.org wrote: * Reuben Thomas r...@sc3d.org, 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

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

2011-01-11 Thread Jakub Wilk
* Reuben Thomas r...@sc3d.org, 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

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 r...@sc3d.org, 2011-01-10,

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-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