Created attachment 245499 [details] Update to 4.2.2
Update to 4.2.2 Enable Singlepass and LLVM compilers. Limit a number of threads to the number of CPU threads because of runtime issues.
Thank you for your submission. I see that your patch hardcodes /usr/local as a search path in a patch. This is incorrect. The user can install the package into any prefix he/she likes. You must obey LOCALBASE here. I recommend as follows: in the patch, use %%LOCALBASE%% for the directory. Then, add a pre-configure target like pre-configure: ${REINPLACE_CMD} -e s,%%LOCALBASE%%,${LOCALBASE}, ${WRKSRC}/Makefile to patch in the true LOCALBASE prefix. Please check this issue and resubmit as appropriate.
Rust compiler can't find libffi and libzstd, so I hardcoded search path to /usr/local/lib. It must be hardcoded because rust path search obviously doesn't work well on FreeBSD.
(In reply to Krešimir Jozić from comment #3) Yes, and /usr/local/lib is wrong. While this is indeed the default LOCALBASE, the user can built package for whatever prefix he/she likes. Please adapt the patch as I indicated previously so it patches in whatever LOCALBASE says instead of hard-coding /usr/local.
Created attachment 245507 [details] Update to 4.2.2
Thanks for the fix.
One final issue I just noted: You have USES=llvm, which adds llvm as a build-time dependency. Are you sure you won't need LLVM at runtime?
As I am aware, everything is compiled inside of executable.
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=6ab67bad1dcc637c580ec720ed13c614a005034b commit 6ab67bad1dcc637c580ec720ed13c614a005034b Author: Krešimir Jozić <kjozic@gmail.com> AuthorDate: 2023-10-08 14:40:31 +0000 Commit: Robert Clausecker <fuz@FreeBSD.org> CommitDate: 2023-10-09 07:40:32 +0000 devel/wasmer: Update to 4.2.2 Changelog: https://github.com/wasmerio/wasmer/blob/master/CHANGELOG.md#422---04102023 PR: 274355 devel/wasmer/Makefile | 17 ++- devel/wasmer/Makefile.crates | 74 +++++----- devel/wasmer/distinfo | 154 +++++++++++---------- devel/wasmer/files/patch-Makefile (new) | 11 ++ ..._wasix_src_runtime_task__manager_tokio.rs (new) | 11 ++ 5 files changed, 154 insertions(+), 113 deletions(-)
Thank you for your contribution.