Gwenole Beauchesne ([EMAIL PROTECTED]) wrote: > On Thu, 21 Nov 2002, Han Boetes wrote: > > > Please test and comment and submit. > > First we review diffs and I have comments:
Ehm yes. Sorry. > - Why the following change? +export PNG_NO_SETJMP_SUPPORTED=1 Ahem, I forgot to remove it. That was what the png-developper suggested, I tested it for quite some time with vaarious ways to set that variable, but it didn't help. > - I don't know why they have left the "comment" as is. But I would > expect it to be guard in the event the include machinery went wrong. > If you reach that case, we need to know why. Well this is the test-case. http://www.xs4all.nl/~hanb/software/rpms/idesk-0.3.5-2mdk.src.rpm Other apps haven't had this problem with linking to png, so why this one. Good question. I have looked for quite some time at /usr/include/libpng12/pngconf.h and even though I hardly know co- ding these two lines look very suspect. They are clearly different from the rest. Perhaps I ought to patch the idesk-cource so it doesn't try to use that part of the code anymore. //Han -- http://www.xs4all.nl/~hanb/software
