Bug 257993 - math/nauty: Install missing headers nausparse.h, etc; Enable TLS
Summary: math/nauty: Install missing headers nausparse.h, etc; Enable TLS
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: Yuri Victorovich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-22 06:09 UTC by Yuri Victorovich
Modified: 2021-08-23 20:17 UTC (History)
1 user (show)

See Also:
moritz: maintainer-feedback+

Attachments
patch (1.04 KB, patch)
2021-08-22 06:09 UTC, Yuri Victorovich 		no flags Details | Diff
patch (1.15 KB, patch)
2021-08-22 07:33 UTC, Yuri Victorovich 		no flags Details | Diff
patch (1.35 KB, patch)
2021-08-23 06:06 UTC, Yuri Victorovich 		no flags Details | Diff
Show Obsolete (2) 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 Yuri Victorovich freebsd_committer freebsd_triage 2021-08-22 06:09:39 UTC 
Created attachment 227363 [details]
patch

math/cbc expects nausparse.h when built with nauty.
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2021-08-22 07:33:04 UTC 
Created attachment 227364 [details]
patch
Comment 2 Yuri Victorovich freebsd_committer freebsd_triage 2021-08-23 02:48:01 UTC 
All headers mentioned in the documentation.
Comment 3 Yuri Victorovich freebsd_committer freebsd_triage 2021-08-23 06:06:07 UTC 
Created attachment 227373 [details]
patch

Enable TL because some other ports require TLS.
Comment 4 Moritz Schmitt 2021-08-23 18:40:25 UTC 
Thanks for adding the missing headers, your patch seems fine.

Relevant documentation can be found at https://pallini.di.uniroma1.it/nug27.pdf.
Comment 5 Yuri Victorovich freebsd_committer freebsd_triage 2021-08-23 20:14:49 UTC 
Committed, thanks!
Comment 6 commit-hook freebsd_committer freebsd_triage 2021-08-23 20:17:58 UTC 
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=51c8f4db5f27af53e787a0616d009dec29b259f5

commit 51c8f4db5f27af53e787a0616d009dec29b259f5
Author:     Yuri Victorovich <yuri@FreeBSD.org>
AuthorDate: 2021-08-23 19:24:27 +0000
Commit:     Yuri Victorovich <yuri@FreeBSD.org>
CommitDate: 2021-08-23 20:14:44 +0000

    math/nauty: Install missing headers (that were not installed before); Add the TLS option

    * all headers mentioned in the documentation are now installed
      (docs https://pallini.di.uniroma1.it/nug27.pdf)
    * TLS option is required at least by math/libnormaliz

    PR:             257993
    Approved by:    moritz@schmi.tt (maintainer)

 math/nauty/Makefile  | 9 +++++++--
 math/nauty/pkg-plist | 3 +++
 2 files changed, 10 insertions(+), 2 deletions(-)