Bug 735182

Summary: libreoffice doesn't build with poppler-0.17.3
Product: [Fedora] Fedora Reporter: Marek Kašík <mkasik>
Component: libreofficeAssignee: Caolan McNamara <caolanm>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 16CC: caolanm, dtardon, ltinkl
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: libreoffice- Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-02 08:21:52 EDT Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
Bug Depends On:    
Bug Blocks: 735175    

Description Marek Kašík 2011-09-01 13:42:57 EDT
Description of problem:
Current version of libreoffice needs poppler's class DCTStream, which is not
present now.

Log snippet:
Entering /builddir/build/BUILD/libreoffice-
Compiling: sdext/source/pdfimport/xpdfwrapper/wrapper_gpl.cxx
Compiling: sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx
/builddir/build/BUILD/libreoffice- In function 'void pdfi::writeJpeg_(OutputBuffer&, Stream*, bool)':
/builddir/build/BUILD/libreoffice- error: 'DCTStream' was not declared in this scope
/builddir/build/BUILD/libreoffice- error: expected primary-expression before ')' token
/builddir/build/BUILD/libreoffice- error: expected ')' before 'str'

Version-Release number of selected component (if applicable):

Actual results:
Build fails.

Expected results:
Build succeed.

Additional info:
See the bug #735175 for more info and scratch-build of poppler-0.17.3.
This was introduced by upstream commit "Sanitize headers a big" (mainly by
correction of definition of ENABLE_LIBJPEG - see definition of DCTStream class
in poppler/Stream.h) (see http://cgit.freedesktop.org/poppler/poppler/commit/?id=a8fca630d592941c033ca6a380bf46b6e733a748).
Comment 1 Caolan McNamara 2011-09-02 06:27:37 EDT
Comment 2 Caolan McNamara 2011-09-02 07:47:24 EDT
I see that for poppler itself, when its ENABLE_LIBJPEG its own utils include 

#include "DCTStream.h"

and regain access to DCTStream, they (like us) use it to call getRawStream.

DCTStream.h however isn't installed into the -devel package, so external apps can't use the same hack as poppler itself
Comment 3 Caolan McNamara 2011-09-02 08:21:52 EDT
fix checked in for F-16/master
Comment 4 Marek Kašík 2011-09-02 08:28:25 EDT
Thank you very much.
Comment 5 Fedora Update System 2011-09-07 06:02:08 EDT
libreoffice- has been submitted as an update for Fedora 16.
Comment 6 Fedora Update System 2011-09-09 13:05:41 EDT
libreoffice- has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.