Bug 207073

Summary: editors/openoffice-devel: fix build with boost 1.60
Product: Ports & Packages Reporter: Dmitry Marakasov <amdmi3>
Component: Individual Port(s)Assignee: FreeBSD Office Team <office>
Status: Closed FIXED    
Severity: Affects Some People CC: truckman
Priority: --- Flags: bugzilla: maintainer-feedback? (office)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch
none
Patch none

Description Dmitry Marakasov freebsd_committer freebsd_triage 2016-02-10 10:37:26 UTC 
Created attachment 166825 [details]
Patch

Fix build with boost 1.60 by linking module which fails with unresolved symbols in libboost_system to libboost_system.

See

https://wiki.freebsd.org/BoostPortingProject/1.55-to-1.60
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=199601
Comment 1 Don Lewis freebsd_committer freebsd_triage 2016-02-10 18:43:13 UTC 
I committed a change yesterday to disable the PDF import plugin so that the openoffice-devel port will build with boost 1.60.  That unblocks 199601.

Longer term my plan is to patch configure to detect when -lboost_system is needed and add that when linking the PDF import module, and then submit this upstream.

Boost will also need to be moved from BUILD_DEPENDS to LIB_DEPENDS.
Comment 2 Dmitry Marakasov freebsd_committer freebsd_triage 2016-02-11 00:07:34 UTC 
Created attachment 166859 [details]
Patch

> Boost will also need to be moved from BUILD_DEPENDS to LIB_DEPENDS.

Fixed.

> I committed a change yesterday to disable the PDF import plugin so that the openoffice-devel port will build with boost 1.60.

This is unneeded pessimization.
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-02-11 15:41:15 UTC 
A commit references this bug:

Author: truckman
Date: Thu Feb 11 15:40:54 UTC 2016
New revision: 408688
URL: https://svnweb.freebsd.org/changeset/ports/408688

Log:
  Unbreak PDF Import extension when building with modern boost.  Similar
  to the patch by amdmi3, link the extension with -lboost_system, but
  only do this in the --with-system-boost case.  Doing this unconditionally
  would break building with the bundled boost because only the boost headers
  are available and the boost libraries are not built.  This is still a bit
  pessimal because -lboost_system may be used when it is not strictly
  necessary (when the system boost is old), but it is likely that this is
  only relevant to FreeBSD and we are in the process of upgrading boost.
  This fix should be acceptable upstream.

  Re-enable the PDF Import extension by default.

  When the PDF Import extension is enabled, promote boost from BUILD_DEPENDS
  to LIB_DEPENDS.

  Tested with PDFIMPORT both on and off.  Also tested with PDFIMPORT on,
  --with-system-boost disabled, and boost removed from *_DEPENDS.

  PR:		207073, 199601

Changes:
  head/editors/openoffice-devel/Makefile
  head/editors/openoffice-devel/files/patch-sdext_source_pdfimport_makefile.mk