On Mon 19 Jan 2026, Stuart Henderson 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

My bad, will fix.

> > +.include <bsd.port.arch.mk>
> > +
> > +.if ${PROPERTIES:Mlp64} || ${MACHINE_ARCH} == i386
> 
> why the arch restriction?

The rest of the chain is correct, I simply followed multimedia/svt-av1. Will 
adapt the patch and send it over once we have some sort of agreement on how to 
proceed. I can not add a comment to multimedia/svt-av1 as I am unsure why the 
restriction is there.

> > +WANTLIB+=          SvtAv1Enc
> > +LIB_DEPENDS+=              multimedia/svt-av1
> > +LIBavcodec_EXTRALIBS+=     -lSvtAv1Enc
> > +CONFIGURE_ARGS+=   --enable-libsvtav1
> > +.endif
> > +
> >  .if ${MACHINE_ARCH} == i386
> >  LDLIBFLAGS+=       -Wl,-znotext
> >  .endif

Reply via email to