Created attachment 252765 [details] Patch for simdutf * Add USES= pathfix to fix install path of .pc files * Add USES= python:build,test as python is executed during build and test build * Remove BENCHMARKS as it depends on external library/header from base64 (https://github.com/aklomp/base64 ) which isn't ported * Define USES= compiler:c++17-lang for tools * Integrate TEST option into ports framework * Some minor layout changes for better readability Build and run-time tested on 14.1-RELEASE (amd64)
Thank you for the patch. I had a bit of a busy time and will get back to updating my ports soon. I am in touch with upstream as this is a project I contributed to in the past and will see that it is updated shortly. The BENCHMARKS option was mainly a convenience for me; it can indeed be removed. Python used to not be needed to build and test this port. Did this change? Feel free to commit this, I trust that it'll do the right thing.
It uses Python for amalgamation
Committed, thanks
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=64e1eee3c8120d77c53c2d496c6df3540dfaa502 commit 64e1eee3c8120d77c53c2d496c6df3540dfaa502 Author: Daniel Engberg <diizzy@FreeBSD.org> AuthorDate: 2024-08-24 10:45:56 +0000 Commit: Daniel Engberg <diizzy@FreeBSD.org> CommitDate: 2024-08-24 10:45:59 +0000 converters/simdutf: Update to 5.3.7 * Add USES= pathfix to fix install path of .pc files * Add USES= python:build,test as python is executed during build and test build * Remove BENCHMARKS as it depends on external library/header from base64 (https://github.com/aklomp/base64 ) that isn't ported * Define USES= compiler:c++17-lang * Integrate TEST option into ports framework * Some minor layout changes for better readability Changelog: https://github.com/simdutf/simdutf/compare/v5.2.8...v5.3.7 PR: 280821 Reviewed by: fuz (maintainer) converters/simdutf/Makefile | 19 ++++++++----------- converters/simdutf/distinfo | 6 +++--- converters/simdutf/pkg-plist | 4 ++-- 3 files changed, 13 insertions(+), 16 deletions(-)