Bug 735182 - libreoffice doesn't build with poppler-0.17.3
Summary: libreoffice doesn't build with poppler-0.17.3
Alias: None
Product: Fedora
Classification: Fedora
Component: libreoffice
Version: 16
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Caolan McNamara
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 735175
TreeView+ depends on / blocked
Reported: 2011-09-01 17:42 UTC by Marek Kašík
Modified: 2011-09-09 17:06 UTC (History)
3 users (show)

Fixed In Version: libreoffice-
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2011-09-02 12:21:52 UTC

Attachments (Terms of Use)

Description Marek Kašík 2011-09-01 17:42:57 UTC
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 10:27:37 UTC

Comment 2 Caolan McNamara 2011-09-02 11:47:24 UTC
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 12:21:52 UTC
fix checked in for F-16/master

Comment 4 Marek Kašík 2011-09-02 12:28:25 UTC
Thank you very much.

Comment 5 Fedora Update System 2011-09-07 10:02:08 UTC
libreoffice- has been submitted as an update for Fedora 16.

Comment 6 Fedora Update System 2011-09-09 17:05:41 UTC
libreoffice- has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.

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