Re: [update] net/synapse 1.68.0 | devel/matrix-common-1.3.0

On Wed, Sep 28, 2022 at 02:43:27PM +0100, Stuart Henderson wrote:
> On 2022/09/28 13:37, Landry Breuil wrote:
> > Le Wed, Sep 28, 2022 at 01:26:29PM +0200, Renaud Allard a écrit :
> > >
> > >
> > > On 9/28/22 13:07, Stuart Henderson wrote:
> > > > As things stand, by having this RDEP you're pulling in the rust compiler
> > > > as a run dependency, as well as setuptools_rust itself, setuptools, and
> > > > a few others. (Maybe I can move setuptools_rust's RDEP on lang/rust
> > > > now that we're not trying to build cryptography on !rust archs, I'll
> > > > take a look).
> > > >
> > > > Anyway I would try patching out that line in pyproject.toml, it shouldn't
> > > > affect anything else and will cut the run deps down by a reasonable amount.
> > > >
> > >
> > > Here it is with setuptools_rust removed from the runtime deps.
> > > It runs fine here so far.
> >
> > since there's some binary being built WANTLIB needs updating:
> >
> > synapse-1.68.0(net/synapse):
> > Missing: c++abi.6 (/usr/local/lib/python3.9/site-packages/synapse/synapse_rust.abi3.so) (system lib)
> > Missing: c.96 (/usr/local/lib/python3.9/site-packages/synapse/synapse_rust.abi3.so) (system lib)
> > WANTLIB += c c++abi
> >
> > so WANTLIB += ${COMPILER_LIBCXX} ?
> >
> > Landry
> >
>
> I think it specifically needs "c c++abi", on sparc64 COMPILER_LIBCXX
> will point at GCC libs and we needs the clang ones here. Same for other
> rust ports (bat, etc)
>

hum, it seems COMPILER_LIBCXX is right even for sparc64.

on sparc64, devel/llvm is linked against estdc++. as rustc uses the devel/llvm
libraries, it is using also estdc++. so COMPILER_LIBCXX is the right thing to
use here.

--
Sebastien Marie