On Thu, Mar 05, 2020 at 08:59:45PM +0000, Stuart Henderson wrote:
> On 2020/03/01 20:12, Sebastien Marie wrote:
> > Hi,
> > 
> > The following diff adds few magic to devel/cargo module in order to use 
> > system
> > libraries instead of building embedded version in crates in an automatic 
> > way.
> > 
> > It works by removing source library from a defined set of crates. So it 
> > will be
> > not possible to build the embedded library (the build will fail instead of
> > silenciously build embedded version).
> > 
> > It also provides magic to help configuring crates to use system library.
> > 
> > Please note it is still the responsability of the maintainer to add 
> > LIB_DEPENDS
> > and WANTLIB in the port.
> 
> I think this makes sense, normally I think it would be "too much magic" but
> in the case of rust ports, things are already so complicated I think they
> need all the help we can give them ;)

I finally commited it. I am still considering another way to deal with such
problem, in a more generic way (and to be able to deal with crates vendored as
in firefox or librsvg), but it will take time to properly design it.

> > file + security/sn0int/Makefile
> > --- security/sn0int/Makefile
> > +++ security/sn0int/Makefile
> 
> This one should use something like:
> 
> MODULES=      lang/lua
> MODLUA_VERSION=       5.2
> MODLUA_SA=    Yes
> LIB_DEPENDS=  ... ${MODLUA_LIB_DEPENDS}
> WANTLIB+=     ... ${MODLUA_WANTLIB}
> 

Thanks:  I changed my diff for sn0int to match your.

-- 
Sebastien Marie

Reply via email to