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 <[email protected]> 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 <[email protected]> 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. -- jca
