Created attachment 244302 [details] math/polymake: remove NATIVE option Remove NATIVE option, bump PORTREVISION due to change in available options.
NATIVE is not enabled by default, and packages are built without it. Why do you want to remove this option?
There have been instances where -march=native makes mistakes and produces broken binaries and on top of that it only works on amd64/i386. People should make use of CPUTYPE instead which is supported be the framework if they want optimized binaries.
Created attachment 244571 [details] Additional fixes to port Add GitHub as primary download site and polymake.org as fallback Respect CFLAGS, O3 optimization should be a menu toggle (if it makes any difference in performance at all)
Reference: https://docs.freebsd.org/en/books/porters-handbook/book/#dads-cflags Sorry for the late reply
Comment on attachment 244571 [details] Additional fixes to port Looks good. Thanks for the additional updates!
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=96e194fafea582015e3d14ec48653a4734a1a15c commit 96e194fafea582015e3d14ec48653a4734a1a15c Author: Daniel Engberg <diizzy@FreeBSD.org> AuthorDate: 2023-10-08 09:27:59 +0000 Commit: Daniel Engberg <diizzy@FreeBSD.org> CommitDate: 2023-10-08 09:40:40 +0000 math/polymake: Remove NATIVE option and add GitHub to MASTER_SITES * Add GitHub as primary download site and upstream website as backup * Remove NATIVE option, users wanting to apply CPU specific optimization should use CPUTYPE instead using ports framework * Use OPTIMIZED_CFLAGS option to define O3 optimization PR: 273316 Reviewed by: Philipp Ost <bsd@philippost.de> (maintainer) Sponsored by: Blinkinblox math/polymake/Makefile | 14 ++++++++------ math/polymake/files/patch-support_configure.pl (new) | 11 +++++++++++ 2 files changed, 19 insertions(+), 6 deletions(-)
Committed, thanks!