libreoffice pdfimport w/ poppler-0.72.0: build failure

classic Classic list List threaded Threaded
3 messages Options
Andreas Sturmlechner-2 Andreas Sturmlechner-2
Reply | Threaded
Open this post in threaded view
|

libreoffice pdfimport w/ poppler-0.72.0: build failure

In git master, libreoffice fails to build with:

sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx:558:70: error: ‘const
class GooString’ has no member named ‘getCString’

The relevant change in poppler 0.72 release notes:
* Rename GooString::getCString to GooString::c_str

The attached patch made it build successfully for me.

Regards,
Andreas
_______________________________________________
LibreOffice mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/libreoffice

libreoffice-6.2.0.1-poppler-0.72.patch (1K) Download Attachment
Mike Kaganski Mike Kaganski
Reply | Threaded
Open this post in threaded view
|

Re: libreoffice pdfimport w/ poppler-0.72.0: build failure

Hi Andreas!

On 08.01.2019 3:12, Andreas Sturmlechner wrote:
> In git master, libreoffice fails to build with:
>
> sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.cxx:558:70: error: ‘const
> class GooString’ has no member named ‘getCString’
>
> The relevant change in poppler 0.72 release notes:
> * Rename GooString::getCString to GooString::c_str
>
> The attached patch made it build successfully for me.

Could you please post the patch to gerrit [1] [2], and also post your
license statement [3]?

[1] https://gerrit.libreoffice.org
[2] https://wiki.documentfoundation.org/Development
[3]
https://wiki.documentfoundation.org/Development/Developers#License_Statements

--
Best regards,
Mike Kaganski
_______________________________________________
LibreOffice mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/libreoffice
Andreas Sturmlechner-2 Andreas Sturmlechner-2
Reply | Threaded
Open this post in threaded view
|

Re: libreoffice pdfimport w/ poppler-0.72.0: build failure

On Dienstag, 8. Jänner 2019 06:17:59 CET Kaganski Mike wrote:
> Could you please post the patch to gerrit [1] [2], and also post your
> license statement [3]?
>
> [1] https://gerrit.libreoffice.org

https://gerrit.libreoffice.org/#/c/65960/

> [3]
> https://wiki.documentfoundation.org/Development/Developers#License_Statement
> s

All of my past & future contributions to LibreOffice may be
licensed under the MPLv2/LGPLv3+ dual license.

Regards,
Andreas


_______________________________________________
LibreOffice mailing list
[hidden email]
https://lists.freedesktop.org/mailman/listinfo/libreoffice