On Mon, 19 Jan 2026 13:43:18 +0100,
Jeremie Courreges-Anglas <jca@wxcvbn.org> wrote:
> 
> On Mon, Jan 19, 2026 at 01:36:46PM +0100, Kirill A. Korinsky wrote:
> > On Mon, 19 Jan 2026 13:18:08 +0100,
> > Jeremie Courreges-Anglas <jca@wxcvbn.org> wrote:
> > > 
> > > On Mon, Jan 19, 2026 at 12:53:36PM +0100, Kirill A. Korinsky wrote:
> > > > On Mon, 19 Jan 2026 12:12:58 +0100,
> > > > Stuart Henderson <stu@spacehopper.org> wrote:
> > > > > 
> > > > > On 2026/01/19 17:33, Pontus Stenetorp wrote:
> > > > > >  # keep it synced with x11/mplayer
> > > > > >  V=		8.0.1
> > > > > > +REVISION=       1
> > > > > 
> > > > > please use tabs not spaces and REVISION should start at 0
> > > > > 
> > > > > > +.include <bsd.port.arch.mk>
> > > > > > +
> > > > > > +.if ${PROPERTIES:Mlp64} || ${MACHINE_ARCH} == i386
> > > > > 
> > > > > why the arch restriction?
> > > > >
> > > > 
> > > > Probably because multimedia/svt-av1 has:
> > > > 
> > > > ONLY_FOR_ARCHS=	${LP64_ARCHS} i386
> > > 
> > > That sucks because it means wantlib vs deps breakage on non-lp64
> > > non-i386 archs.
> > >
> > 
> > I had fast look to https://gitlab.com/AOMediaCodec/SVT-AV1/-/tree/master/Source/Lib
> > and I don't see anything that should technically prevent it from at least
> > build on non LP64 architectures.
> > 
> > Also, it seems that it was imported with ONLY_FOR_ARCHS=${LP64_ARCHS}
> > 
> > Maybe try to remove it and see how it goes?
> 
> That, and/or ask Brad (cc) why ONLY_FOR_ARCHS was added in the first
> place.
> 
> IIRC this restriction was already discussed in the past, if there's
> a good reason we should at least add a comment in svt-av1/Makefile.
> 

I can't find any trace of discussion before import it.

-- 
wbr, Kirill