Bug 245085 - math/flint2: adding NTL support
Summary: math/flint2: adding NTL support
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Thierry Thomas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-03-26 21:01 UTC by Thierry Thomas
Modified: 2020-03-26 21:38 UTC (History)
0 users

See Also:
yuri: maintainer-feedback+

Attachments
Adding NTL support and switch to configure + gmake (2.17 KB, text/plain)
2020-03-26 21:01 UTC, Thierry Thomas 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thierry Thomas freebsd_committer freebsd_triage 2020-03-26 21:01:50 UTC 
Created attachment 212738 [details]
Adding NTL support and switch to configure + gmake

Add an optional support of NTL, and set this option by default because it is needed by SageMath.

Notes:

- the current port defines a mix of cmake and configure, but only cmake is actually used;

- the NTL option is not available with cmake => switch to completely to configure;

- for some reason, the generated Makefile now requires gmake, and it installs more headers;

- the shared library is built by default and it is usable.

Since I'm there, add a test target (all tests passed).
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2020-03-26 21:18:03 UTC 
Thank you for the patch!

Please commit it.
Comment 2 Thierry Thomas freebsd_committer freebsd_triage 2020-03-26 21:37:46 UTC 
Committed, thanks!
Comment 3 commit-hook freebsd_committer freebsd_triage 2020-03-26 21:38:28 UTC 
A commit references this bug:

Author: thierry
Date: Thu Mar 26 21:37:25 UTC 2020
New revision: 529225
URL: https://svnweb.freebsd.org/changeset/ports/529225

Log:
  Add an option to build the support of NTL.

  Set the option by default, because it is used by SageMath.

  PR:		245085
  Approved by:	yuri (maintainer)

Changes:
  head/math/flint2/Makefile
  head/math/flint2/pkg-plist