10 янв. 2025 г. 02:07:44 Miguel Landaeta <mig...@miguel.cc>:

> Hi,
>
> lang/mruby is an embeddable ruby interpreter, but currently is not supporting 
> all use
> cases described in its official docs [1] because is not shipping libmruby.a 
> static library.
> For reference, see also a typical linux package example shipping those files 
> [2].
>
> 1. https://mruby.org/docs/articles/executing-ruby-code-with-mruby.html
> 2. https://packages.debian.org/sid/amd64/libmruby-dev/filelist
>
> ok?

Makes sense, but...

>
>
> Index: Makefile
> ===================================================================
> RCS file: /home/cvs/ports/lang/mruby/Makefile,v
> diff -u -p -u -p -r1.26 Makefile
> --- Makefile    20 Jun 2024 22:41:25 -0000  1.26
> +++ Makefile    9 Jan 2025 22:20:26 -0000
> @@ -6,7 +6,7 @@ GH_ACCOUNT =        mruby
> GH_PROJECT =       mruby
> GH_TAGNAME =       3.3.0
>
> -REVISION =     0
> +REVISION =     1
>
> CATEGORIES =       lang
> HOMEPAGE =     https://mruby.org/
> @@ -32,6 +32,7 @@ do-build:
> do-install:
>     ${INSTALL_PROGRAM} ${WRKSRC}/bin/* ${PREFIX}/bin/
>     ${INSTALL_DATA_DIR} ${PREFIX}/{lib,include,share/doc}/mruby/
> +   cp -R ${WRKSRC}/build/host/lib/libmruby*.a ${PREFIX}/lib/

These are files, so recursive cp is not needed.
Why not INSTALL_DATA?

>     cp -R ${WRKSRC}/mrb{lib,gems} ${PREFIX}/lib/mruby/
>     cp -R ${WRKSRC}/include/* ${PREFIX}/include
>     ${INSTALL_DATA} ${WRKSRC}/doc/guides/* ${PREFIX}/share/doc/mruby/
> Index: pkg/PLIST
> ===================================================================
> RCS file: /home/cvs/ports/lang/mruby/pkg/PLIST,v
> diff -u -p -u -p -r1.10 PLIST
> --- pkg/PLIST   15 Apr 2024 15:10:07 -0000  1.10
> +++ pkg/PLIST   9 Jan 2025 22:23:01 -0000
> @@ -42,6 +42,8 @@ include/mruby/throw.h
> include/mruby/value.h
> include/mruby/variable.h
> include/mruby/version.h
> +@static-lib lib/libmruby.a
> +@static-lib lib/libmruby_core.a
> lib/mruby/
> lib/mruby/mrbgems/
> lib/mruby/mrbgems/default-no-fpu.gembox

Reply via email to