Created attachment 197170 [details]
ntl requires C11-aware compiler. This patch adds USES for it and reorders existing USES.
Tested on CURRENT / powerpc64.
I’m gonna test it on other jails to make sure it’s not needed in others. I rather have it when it’s needed on certain archs as c11 will break mips because need of gcc
(In reply to Nathan from comment #1)
Seems not gonna work on mips anyway either. Probably will need to bump the revision on the port
Created attachment 197176 [details]
Fix build for PowerPC
(In reply to Nathan from comment #3)
IMO bumping PORTREVISION isn't necessary. If it builds, you already use C11-aware compiler. If it isn't, adding USES fixes the build.
Set maintainer-approval to ? on patch, so I can + it
A commit references this bug:
Date: Thu Sep 20 00:00:29 UTC 2018
New revision: 480130
math/ntl: fix build with powerpc64, reorder USES
Submitted by: Piotr Kubaj <firstname.lastname@example.org>
Approved by: Nathan <email@example.com> (maintainer)