Thanks much for the help guys!

WRT the sndio quirks, the latest commit from this project is 7 years old, safe to say that it may be dormant, I'll look into possibly forking and fixing up sndio support later- cos I too, have the same crackling issue on occassion.


On November 28, 2024 4:40:52 AM CST, Stuart Henderson <stu@spacehopper.org> wrote:
>On 2024/11/27 21:48, George Koehler wrote:
>> On Wed, 16 Oct 2024 18:46:14 -0500
>> izzy Meyer <izder456@disroot.org> wrote:
>> 
>> > Hello ports@
>> > 
>> > This port has stayed dormant in the mailing list for quite some time.
>> > 
>> > Any committers willing to look at this?
>> > 
>> > Attached again for ease of looking over.
>> 
>> Import is ok gkoehler@
>> 
>> Maybe drop the pkg/README?  The README only repeats some info from
>> "man nxbelld", and I still needed to read "man nxbelld" to learn the
>> command-line options.
>
>definitely, that is all information which is copied from the manual.
>
>> I attach the port yet again; it's the same .tgz that izzy mailed in
>> October.  I never had a beeper, so I now need nxbelld to hear beeps.
>> 
>> The sndio in nxbelld adds a little noise;
>> 	$ nxbelld -bf quack.wav
>> 	$ echo '\a'
>> didn't sound good, so I switched to
>> 	$ nxbelld -be 'aucat -i quack.wav'
>
>That combined with the manpage text "When using sndio, the beeps are
>slightly shorter than the specified beep duration. Beeps of 120 ms or
>shorter don't get played at all" suggests that the sndio code is not
>quite correct.
>
>diff below with other tweaks (plus rm pkg/README),
>
>the PKGNAME/HOMEPAGE bits are all set by DIST_TUPLE and there's no need
>for a separate V
>
>If it's a C program then setting COMPILER to "base-clang ports-gcc
>base-gcc" can never be right (if it builds with base-gcc and doesn't use
>C++ libraries then there's no need to specify this). As was set here
>it means that on ports-gcc archs there would be add a LIB_DEPENDS on
>ports-gcc's C++ libraries.
>
>This is what Makefile.template has to say:
>
># Standard for C++ ports:
>#COMPILER =             base-clang ports-gcc base-gcc
># Standard for C++11 or newer:
>#COMPILER =             base-clang ports-gcc
># Ports that require a non-default compiler that do *not* use C++ should set this:
>#COMPILER_LANGS =       c
>
>
>diff --git a/x11/nxbelld/Makefile b/x11/nxbelld/Makefile
>index 49ecff1..47bfc56 100644
>--- a/x11/nxbelld/Makefile
>+++ b/x11/nxbelld/Makefile
>@@ -1,14 +1,9 @@
> COMMENT =	fork of xbelld, the X bell daemon for machines w/o a pcspkr
> 
>-V =	0.1.2
>-PKGNAME =	nxbelld-$V
>-
>-DIST_TUPLE +=	github dusxmt nxbelld $V .
>+DIST_TUPLE +=	github dusxmt nxbelld 0.1.2 .
> 
> CATEGORIES =	x11
> 
>-HOMEPAGE =	https://github.com/dusxmt/nxbelld
>-
> MAINTAINER =	izzy Meyer <izder456@disroot.org>
> 
> # GPL v3+
>@@ -16,8 +11,6 @@ PERMIT_PACKAGE =	Yes
> 
> WANTLIB +=	X11 c m sndio
> 
>-COMPILER =	base-clang ports-gcc base-gcc
>-
> USE_GMAKE =	Yes
> 
> CONFIGURE_STYLE =	autoreconf
>diff --git a/x11/nxbelld/pkg/PLIST b/x11/nxbelld/pkg/PLIST
>index 2681ce0..6afaa64 100644
>--- a/x11/nxbelld/pkg/PLIST
>+++ b/x11/nxbelld/pkg/PLIST
>@@ -1,3 +1,2 @@
> @bin bin/nxbelld
> @man man/man1/nxbelld.1
>-share/doc/pkg-readmes/${PKGSTEM}
>