Bug 1163476 - gwenview FTBFS against latest libjpeg-turbo
Summary: gwenview FTBFS against latest libjpeg-turbo
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gwenview
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1164815
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-11-12 19:05 UTC by Rex Dieter
Modified: 2014-11-17 15:18 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-11-17 14:58:41 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Rex Dieter 2014-11-12 19:05:45 UTC
gwenview FTBFS against latest libjpeg-turbo, see
https://koji.fedoraproject.org/koji/taskinfo?taskID=8074343

Excerpt:

In file included from ../../lib/cms/iccjpeg.c:42:0:
../../lib/cms/iccjpeg.h:55:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'JPP'
 extern void write_icc_profile JPP((j_compress_ptr cinfo,
                               ^
../../lib/cms/iccjpeg.h:76:36: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'JPP'
 extern void setup_read_icc_profile JPP((j_decompress_ptr cinfo));
                                    ^
../../lib/cms/iccjpeg.h:95:33: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'JPP'
 extern boolean read_icc_profile JPP((j_decompress_ptr cinfo,
                                 ^
../../lib/cms/iccjpeg.c: In function 'read_icc_profile':
../../lib/cms/iccjpeg.c:253:18: error: expected '=', ',', ';', 'asm' or '__attribute__' before '*' token
       JOCTET FAR *src_ptr;
                  ^
../../lib/cms/iccjpeg.c:253:19: error: 'src_ptr' undeclared (first use in this function)
       JOCTET FAR *src_ptr;
                   ^
../../lib/cms/iccjpeg.c:253:19: note: each undeclared identifier is reported only once for each function it appears in
lib/CMakeFiles/gwenviewlib.dir/build.make:113: recipe for target 'lib/CMakeFiles/gwenviewlib.dir/cms/iccjpeg.o' failed

Comment 1 nucleo 2014-11-14 06:05:53 UTC
Similar problems in digikam https://bugs.kde.org/show_bug.cgi?id=340944
Maybe this is libjpeg-turbo bug?

Comment 2 Kevin Kofler 2014-11-14 16:05:57 UTC
The JPP macro was removed. That's a macro that enabled or disabled modern-day C prototypes. They probably figured that people long stopped caring about ancient pre-standard K&R C. The easiest workaround is to just remove the JPP(…) wrappers, or to:
#ifndef JPP
#define JPP(x) x
#endif

That said, seeing how many packages are affected, IMHO, this backwards-incompatible change in libjpeg-turbo should be reverted.

Comment 3 Rex Dieter 2014-11-17 14:58:41 UTC
I found this upstream ml thread,
http://sourceforge.net/p/libjpeg-turbo/mailman/message/32677570/

I included a workaround patch in the meantime:

%changelog
* Mon Nov 17 2014 Rex Dieter <rdieter> - 4.14.3-2
- fix/workaround FTBFS against new libjpeg-turbo (#1163476)
- use %%find_lang


Note You need to log in before you can comment on or make changes to this bug.