On Mon, Jan 19, 2026 at 01:43:18PM +0100, Jeremie Courreges-Anglas 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 <[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.
ping svt-av1 doesn't *seem* to have any hard arch restriction, except: - message(WARNING "32-bit is not supported") - only amd64 and arm64 has assembly code so is there another reason why svt-av1 is disabled on all 32 bits archs but i386? The only other 32 bits archs which gets bulk builds is arm and isn't really suitable for multimedia, but still, it would be saner to clarify the situation here. > 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 > -- jca
