Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Custom Fedora patch to mingw32-libjpeg breaks API
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Summary: Custom Fedora patch to mingw32-libjpeg breaks API Product: Fedora Version: 13 Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: mingw32-libjpeg AssignedTo: rjones@redhat.com ReportedBy: adam@spicenitz.org QAContact: extras-qa@fedoraproject.org CC: lfarkas@lfarkas.org, berrange@redhat.com, rjones@redhat.com, erik-fedora@vanpienbroek.nl, fedora-mingw@lists.fedoraproject.org Classification: Fedora
Description of problem: The resolution of bug #497492 creates a source incompatible fork of libjpeg that is not even compatible with the native libjpeg!
Version-Release number of selected component (if applicable): mingw32-libjpeg-7-2.fc12.noarch
Steps to Reproduce: 1. Try to compile libtiff with standard Fedora configure, make. 2. Now try to compile libtiff with mingw32-configure, mingw32-make.
Actual results: tif_jpeg.c:289: error: expected declaration specifiers or '...' before 'boolean'
etc.
I don't think we should fork libjpeg in Fedora. Bug #497492 should be reopened and the programs that crash should be fixed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #1 from Adam Goode adam@spicenitz.org 2010-06-16 10:38:51 EDT --- In fact, here is the code from libtiff that works around the problem without breaking libjpeg:
/* The windows RPCNDR.H file defines boolean, but defines it with the unsigned char size. You should compile JPEG library using appropriate definitions in jconfig.h header, but many users compile library in wrong way. That causes errors of the following type:
"JPEGLib: JPEG parameter struct mismatch: library thinks size is 432, caller expects 464"
For such users we wil fix the problem here. See install.doc file from the JPEG library distribution for details. */
/* Define "boolean" as unsigned char, not int, per Windows custom. */ #if defined(__WIN32__) && !defined(__MINGW32__) # ifndef __RPCNDR_H__ /* don't conflict if rpcndr.h already read */ typedef unsigned char boolean; # endif # define HAVE_BOOLEAN /* prevent jmorecfg.h from redefining it */ #endif
#include "jpeglib.h" #include "jerror.h"
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #2 from Bug Zapper triage@lists.fedoraproject.org 2011-06-02 06:36:56 EDT ---
This message is a reminder that Fedora 13 is nearing its end of life. Approximately 30 (thirty) days from now Fedora will stop maintaining and issuing updates for Fedora 13. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as WONTFIX if it remains open with a Fedora 'version' of '13'.
Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version prior to Fedora 13's end of life.
Bug Reporter: Thank you for reporting this issue and we are sorry that we may not be able to fix it before Fedora 13 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora please change the 'version' of this bug to the applicable version. If you are unable to change the version, please add a comment here and someone will do it for you.
Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete.
The process we are following is described here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Kalev Lember kalev@smartlink.ee changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kalev@smartlink.ee Version|13 |rawhide
--- Comment #3 from Kalev Lember kalev@smartlink.ee 2011-06-02 07:01:35 EDT --- Setting version to rawhide.
I would like to switch from libjpeg7 to libjpeg-turbo like was done in native Fedora packages. libjpeg-turbo upstream is much more responsive and I believe we should be able to fix the boolean problem in an upstreamable manner.
Erik, you said you had mingw-libjpeg-turbo packaged up locally. Can you post it for review? I would be happy to review it and help with rebuilds.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #4 from Levente Farkas lfarkas@lfarkas.org 2011-06-02 07:12:29 EDT --- unfortunately even libjpeg-turbo seems to be api and binary compatible with libjpeg it's not (we learn it from rhel-6.1 extra packages). so either it's required a large massrebuild or please keep mingw32-libjpeg along with mingw-libjpeg-turbo.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #5 from Kalev Lember kalev@smartlink.ee 2011-06-02 07:27:52 EDT --- Yes, we would need a large scare rebuild to switch over. We would also be breaking binary compatibility with this change, because libjpeg7 and libjpeg-turbo have different DLL names. (libjpeg-turbo and libjpeg-6b would have been compatible, but we are already using libjpeg7).
libjpeg 6b: libjpeg-62.dll libjpeg 7: libjpeg-7.dll libjpeg-turbo: libjpeg-62.dll
So, considering that we'll need a rebuild to switch over, it would also make sense to fix the boolean definition in one go.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #6 from Levente Farkas lfarkas@lfarkas.org 2011-06-02 07:35:08 EDT --- in the mean time may be it'd be useful to apply libtiff's patch to the current mingw32-libjpeg...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #7 from Erik van Pienbroek erik-fedora@vanpienbroek.nl 2011-06-02 07:44:56 EDT --- @Kalev, comment 3: I have a mingw-libjpeg-turbo package ready, but it's already based on the new guidelines (with win64 support). I could put it up for review, but it won't build successfully at the moment because the mingw-w64 toolchain isn't approved in Fedora yet
@Levente, comment 4: It's strange that you encountered an API/ABI difference between libjpeg and libjpeg-turbo. When libjpeg-turbo was introduced in Fedora 14 no rebuilds were necessary at all as can be seen at https://fedoraproject.org/wiki/Features/libjpeg-turbo
@Kalev, comment 5: That was also the plan I was having with the mingw-libjpeg-turbo package once the mingw-w64 toolchain gets approved for inclusion in Fedora. A mass rebuild will be required at that time anyway
@Levente, comment 6: The patch mentioned in comment 1 only applies to libtiff. It isn't supposed to work in libjpeg. All packages using libjpeg would have to apply a variant to this patch as it all depends on the headers which get included in c/c++ files which want to use libjpeg headers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Erik van Pienbroek erik-fedora@vanpienbroek.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |710101
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
--- Comment #8 from Erik van Pienbroek erik-fedora@vanpienbroek.nl 2011-06-02 09:13:49 EDT --- The mingw-libjpeg-turbo package has just been put up for review at bug 710101
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Bug 604702 depends on bug 710101, which changed state.
Bug 710101 Summary: Review Request: mingw-libjpeg-turbo - MinGW Windows Libjpeg-turbo library https://bugzilla.redhat.com/show_bug.cgi?id=710101
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Erik van Pienbroek erik-fedora@vanpienbroek.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ON_QA Flag| |needinfo?(adam@spicenitz.or | |g)
--- Comment #9 from Erik van Pienbroek erik-fedora@vanpienbroek.nl 2011-06-10 13:39:21 EDT --- The mingw-libjpeg-turbo package is now in Fedora and doesn't contain any ugly hacks any more.
Are you okay that we close this bug?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Adam Goode adam@spicenitz.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(adam@spicenitz.or |needinfo?(adam@spicenitz.or |g), |g) |needinfo?(adam@spicenitz.or | |g) |
--- Comment #10 from Adam Goode adam@spicenitz.org 2011-06-11 14:43:07 EDT --- I will test it out using OpenSlide, which is what had problems before. But looking promising so far.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=604702
Erik van Pienbroek erik-fedora@vanpienbroek.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution| |RAWHIDE Flag|needinfo?(adam@spicenitz.or | |g) | Last Closed| |2012-02-28 11:44:53
--- Comment #11 from Erik van Pienbroek erik-fedora@vanpienbroek.nl 2012-02-28 11:44:53 EST --- No more issues reported, closing bug
-
Red Hat Bugzilla