Volker Schlecht wrote (2023-09-15 23:21 CEST):
> On my amd64 systems espeak-ng does not produce audio output.
> 
> Interestingly it seems to have broken with the last commit:
> 
> https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/audio/espeak/Makefile?rev=1.27&content-type=text/x-cvsweb-markup
> 
> because when I add CPPFLAGS to CONFIGURE_ENV, it does actually
> speak, but it seems to be somehow important, that there are no
> line breaks in CONFIGURE_ENV.
> 
> i.e. with the below patch it produces audio output. Does that
> make sense to anyone?

Hi,

I think your mail program has wrapped the lines in the patch. It doesn't 
apply like this.

I don't understand what you mean with the lines break in CONFIGURE_ENV.

This works for me:
CONFIGURE_ENV=  LDFLAGS="-L${LOCALBASE}/lib -L${X11BASE}/lib" \
                CPPFLAGS="-I${LOCALBASE}/include"


The reason for why CPPFLAGS are needed is in the configure output.

Without setting CPPFLAGS, pcaudiolib support will be disabled, because
configure is unable to find the header:
    PCAudioLib:                    no

Once CPPFLAGS is set correctly:
    PCAudioLib:                    yes

OK sdk@ for the fixed patch below.


Index: audio/espeak/Makefile
===================================================================
RCS file: /cvs/ports/audio/espeak/Makefile,v
retrieving revision 1.27
diff -u -p -u -p -r1.27 Makefile
--- audio/espeak/Makefile       1 Jun 2023 20:58:21 -0000       1.27
+++ audio/espeak/Makefile       16 Sep 2023 09:23:59 -0000
@@ -4,7 +4,7 @@ GH_ACCOUNT=             espeak-ng
 GH_PROJECT=            espeak-ng
 GH_TAGNAME=            1.51
 PKGNAME=               ${DISTNAME:S/-ng//}
-REVISION=              1
+REVISION=              2
 
 SHARED_LIBS += espeak-ng                 0.0 # 2.51
 
@@ -29,7 +29,9 @@ AUTORECONF=           ./autogen.sh
 
 CONFIGURE_STYLE=       autoreconf
 # upstream is doing something odd and doesn't add LDFLAGS/CXXFLAGS if CPPFLAGS 
is set
-CONFIGURE_ENV=         LDFLAGS="-L${LOCALBASE}/lib -L${X11BASE}/lib"
+
+CONFIGURE_ENV=         LDFLAGS="-L${LOCALBASE}/lib -L${X11BASE}/lib" \
+                       CPPFLAGS="-I${LOCALBASE}/include"
 
 DEBUG_PACKAGES=                ${BUILD_PACKAGES}
 

Reply via email to