On Sun, Mar 02, 2025 at 11:52:02AM +0100, Antoine Jacoutot wrote:
> On Sun, Mar 02, 2025 at 11:14:32AM +0100, Landry Breuil wrote:
> > Le Sun, Mar 02, 2025 at 10:21:06AM +0100, Antoine Jacoutot a écrit :
> > > On Sat, Mar 01, 2025 at 10:24:22AM -0700, Tobias Heider wrote:
> > > > CVSROOT:        /cvs
> > > > Module name:    ports
> > > > Changes by:     [email protected]   2025/03/01 10:24:22
> > > > 
> > > > Log message:
> > > >     import ports/wayland/wf-recorder, ok landry@
> > > >     
> > > >     wf-recorder is a utility program for screen recording of wayland 
> > > > compositors
> > > >     that support the wlr-screencopy-v1 and xdg-output protocols.
> > > >     
> > > >     Audio recording is currently disabled on OpenBSD.
> > > >     
> > > >     Status:
> > > >     
> > > >     Vendor Tag: tobhe
> > > >     Release Tags:       tobhe_20250301
> > > >     
> > > >     N ports/wayland/wf-recorder/Makefile
> > > >     N ports/wayland/wf-recorder/distinfo
> > > >     N ports/wayland/wf-recorder/patches/patch-meson_build
> > > >     N ports/wayland/wf-recorder/patches/patch-meson_options_txt
> > > >     N ports/wayland/wf-recorder/patches/patch-src_main_cpp
> > > >     N ports/wayland/wf-recorder/pkg/DESCR
> > > >     N ports/wayland/wf-recorder/pkg/PLIST
> > > >     
> > > >     No conflicts created by this import
> > > 
> > > Am I missing something or the patch-meson_options_txt patch does not make 
> > > sense?
> > > Why not use CONFIGURE_ARGS?
> > 
> > right, that's something i totally overlooked. seems the diff below
> > achieves the same (after dropping the meson_options.txt patch)
> > 
> > --- a/wayland/wf-recorder/Makefile
> > +++ b/wayland/wf-recorder/Makefile
> > @@ -16,7 +16,10 @@ COMPILER =   base-clang ports-gcc
> >  
> >  MODULES =      devel/meson
> >  
> > -LIB_DEPENDS = graphics/ffmpeg wayland/wayland
> > +LIB_DEPENDS =  graphics/ffmpeg \
> > +               wayland/wayland
> > +CONFIGURE_ARGS =       -Dpulse=disabled \
> > +                       -Dpipewire=disabled
> > 
> > Landry
> 
> 
> OK for me, thanks.

Thanks, fixed!

> 
> -- 
> Antoine
> 

Reply via email to