Bug 222177 - math/cgal: move QT5 as an option and add new math dependencies libraries
Summary: math/cgal: move QT5 as an option and add new math dependencies libraries
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Wen Heping
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-09 21:56 UTC by Loïc Bartoletti
Modified: 2017-09-13 07:40 UTC (History)
3 users (show)

See Also:
bugzilla: maintainer-feedback? (wen)


Attachments
cgal 4.9 move QT5 as an option (3.88 KB, patch)
2017-09-09 21:56 UTC, Loïc Bartoletti
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Loïc Bartoletti freebsd_committer 2017-09-09 21:56:58 UTC
Created attachment 186210 [details]
cgal 4.9 move QT5 as an option

CGAL can be built more quickly by removing the forced dependency on QT5.
No port depends directly on libCGAL_Qt5, moreover it adds to the X11 dependencies that are not required for a postgresql server; CGAL is used as a dependency for two postgresql/postgis plugins and OpenSCAD requires libCGAL(core) not libCGAL_Qt5.

I took the opportunity to update the Makefile.

Warning, I did not update the CGAL version to 4.10 because it does not allow the compilation of SFCGAL (the version 4.11 being developed solves the problem. https://github.com/Oslandia/SFCGAL/issues/142).
Comment 1 commit-hook freebsd_committer 2017-09-13 07:39:31 UTC
A commit references this bug:

Author: wen
Date: Wed Sep 13 07:38:41 UTC 2017
New revision: 449755
URL: https://svnweb.freebsd.org/changeset/ports/449755

Log:
  - Move QT5 as an option
  - Add missing DEPENDS

  PR:		222177
  Submitted by:	lbartoletti@tuxfamily.org

Changes:
  head/math/cgal/Makefile
  head/math/cgal/distinfo
  head/math/cgal/pkg-plist