Bug 272937 - math/givaro: install headers incompatible with clang 16
Summary: math/givaro: install headers incompatible with clang 16
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Kurt Jaeger
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-08-04 15:00 UTC by Thierry Thomas
Modified: 2023-08-04 16:34 UTC (History)
1 user (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Thierry Thomas freebsd_committer freebsd_triage 2023-08-04 15:00:20 UTC
Commit 4409f4891a8fe2f7c547961ca57c00b2fb578c59, which upgraded Givaro to 4.2.0, broke math/linbox.

To unbreak linbox, it must be upgraded to 1.7.0 Sablet, but it does not compile on -CURRENT, because Givaro installs headers containing incompatible code.

See <https://github.com/linbox-team/givaro/issues/225>.
Comment 1 Kurt Jaeger freebsd_committer freebsd_triage 2023-08-04 15:47:21 UTC
I'm not sure I understand the details enough to be able to provide a fix. If you have a suggestion, please provide a patch.
Comment 2 Thierry Thomas freebsd_committer freebsd_triage 2023-08-04 16:34:47 UTC
In the GH issue for Mac OS X <https://github.com/linbox-team/givaro/issues/129>, the OP suggested to rename bool_constant, and it seems a good idea also for me, but it has been rejected by upstream, without an explicit reason.