On 2018/07/15 10:27, Leonid Bobrov wrote:
> On Sun, Jul 15, 2018 at 03:11:06AM -0400, Brian Callahan wrote:
> > Your threads are impossible to follow because you post tons of extraneous
> > information that has nothing to do with the original purpose of the thread.
> > I don't know what this dpb listing has to do with updating openal, other
> > than to show us that you may not entirely understand how dpb works. If you
> > are having a problem with dpb, please start a new thread about that.
> > 
> 
> David CARLIER asked me for dpb(1) output to make sure after updating
> this port other ports will keep building. Yes, I can't understand its
> manual page. Maybe I better manually build all those ports via make(1)?

If you can't figure out dpb, just use make with SUBDIRLIST.

> > All this extraneous information makes it very difficult to figure out
> > exactly what I'm supposed to be looking at, and consequently I have no idea
> > what I'm supposed to review or commit. Which is why I've asked in each of my
> > mails to you, to please send a new diff with what you want looked at. This
> > is the third time I'm asking you now.
> > 
> 
> The first diff I've sent is perfect, should I resend it at every mail
> if I didn't change anything else? Very well:

Switching to portaudio isn't perfect though. You are making things worse
for many use cases while fixing one which you're about the only person
who has complained about.

> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/audio/openal/Makefile,v
> retrieving revision 1.50
> diff -u -p -u -p -r1.50 Makefile
> --- Makefile  31 Dec 2017 18:46:26 -0000      1.50
> +++ Makefile  4 Jun 2018 01:16:29 -0000
> @@ -4,13 +4,12 @@ BROKEN-hppa =       undefined reference to __s
>  
>  COMMENT =    cross-platform 3D audio API
>  
> -V =          1.17.2
> +V =          1.18.2
>  EPOCH =              0
>  DISTNAME =   openal-soft-$V
>  PKGNAME =    openal-$V
>  CATEGORIES = audio
> -SHARED_LIBS =        openal  3.0
> -REVISION =   0
> +SHARED_LIBS =        openal  4.0
>  
>  HOMEPAGE =   http://kcat.strangesoft.net/openal.html
>  
> @@ -19,7 +18,7 @@ MAINTAINER =        David Carlier <devnexen@gma
>  # LGPLv2+
>  PERMIT_PACKAGE_CDROM =       Yes
>  
> -WANTLIB +=   c m pthread sndio ${COMPILER_LIBCXX}
> +WANTLIB +=   c m portaudio pthread
>  
>  MASTER_SITES =       ${HOMEPAGE:S,.html,-releases/,}
>  EXTRACT_SUFX =       .tar.bz2
> @@ -29,7 +28,7 @@ CONFIGURE_ARGS =-DCMAKE_BUILD_TYPE=OpenB
>               -DALSOFT_DLOPEN=Off \
>               -DALSOFT_UTILS=On \
>               -DALSOFT_EXAMPLES=Off \
> -             -DALSOFT_REQUIRE_SNDIO=On \
> +             -DALSOFT_REQUIRE_PORTAUDIO=On \
>               -DALSOFT_BACKEND_WAVE=Off \
>               -DALSOFT_CPUEXT_SSE=Off \
>               -DALSOFT_CPUEXT_SSE2=Off \
> @@ -39,12 +38,13 @@ CONFIGURE_ARGS =-DCMAKE_BUILD_TYPE=OpenB
>               -DCMAKE_DISABLE_FIND_PACKAGE_AudioIO:Bool=Yes \
>               -DCMAKE_DISABLE_FIND_PACKAGE_JACK:Bool=Yes \
>               -DCMAKE_DISABLE_FIND_PACKAGE_OSS:Bool=Yes \
> -             -DCMAKE_DISABLE_FIND_PACKAGE_PortAudio:Bool=Yes \
>               -DCMAKE_DISABLE_FIND_PACKAGE_PulseAudio:Bool=Yes \
>               -DCMAKE_DISABLE_FIND_PACKAGE_QSA:Bool=Yes \
> +             -DCMAKE_DISABLE_FIND_PACKAGE_SoundIO:Bool=Yes \
>               -DALSOFT_NO_CONFIG_UTIL=On
>  
>  MODULES =    devel/cmake
> +LIB_DEPENDS =        audio/portaudio-svn
>  
>  COMPILER =           base-clang ports-clang ports-gcc
>  
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/audio/openal/distinfo,v
> retrieving revision 1.10
> diff -u -p -u -p -r1.10 distinfo
> --- distinfo  15 Nov 2017 13:02:42 -0000      1.10
> +++ distinfo  4 Jun 2018 01:16:29 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (openal-soft-1.17.2.tar.bz2) = 
> o0H4VC8fC4xlJBoX2hPQc/GOwGZY4aFgao7Mi7wrMxQ=
> -SIZE (openal-soft-1.17.2.tar.bz2) = 486934
> +SHA256 (openal-soft-1.18.2.tar.bz2) = 
> n4rB4n+6FaWXWKE/DH9lQKBgW2w6aR3vnUIFcFBtfoI=
> +SIZE (openal-soft-1.18.2.tar.bz2) = 639420
> Index: patches/patch-Alc_ALc_c
> ===================================================================
> RCS file: patches/patch-Alc_ALc_c
> diff -N patches/patch-Alc_ALc_c
> --- patches/patch-Alc_ALc_c   31 Dec 2017 18:46:26 -0000      1.1
> +++ /dev/null 1 Jan 1970 00:00:00 -0000
> @@ -1,35 +0,0 @@
> -$OpenBSD: patch-Alc_ALc_c,v 1.1 2017/12/31 18:46:26 jca Exp $
> -
> -commit d1e98c36d375433cb11a7a74ce20c968491773aa
> -Author: Chris Robinson <chris.k...@gmail.com>
> -Date:   Sat Apr 30 17:14:55 2016 -0700
> -
> -    Don't crash when there's no backend to probe
> -
> -Index: Alc/ALc.c
> ---- Alc/ALc.c.orig
> -+++ Alc/ALc.c
> -@@ -1117,6 +1117,11 @@ static void alc_initconfig(void)
> -         V0(factory,init)();
> -     }
> - 
> -+    if(!PlaybackBackend.name)
> -+        WARN("No playback backend available!\n");
> -+    if(!CaptureBackend.name)
> -+        WARN("No capture backend available!\n");
> -+
> -     if(ConfigValueStr(NULL, NULL, "excludefx", &str))
> -     {
> -         size_t len;
> -@@ -1230,9 +1235,9 @@ static void ProbeDevices(al_string *list, struct Backe
> -     LockLists();
> -     al_string_clear(list);
> - 
> --    if(!backendinfo->getFactory)
> -+    if(backendinfo->Probe)
> -         backendinfo->Probe(type);
> --    else
> -+    else if(backendinfo->getFactory)
> -     {
> -         ALCbackendFactory *factory = backendinfo->getFactory();
> -         V(factory,probe)(type);
> Index: patches/patch-Alc_alcConfig_c
> ===================================================================
> RCS file: /cvs/ports/audio/openal/patches/patch-Alc_alcConfig_c,v
> retrieving revision 1.4
> diff -u -p -u -p -r1.4 patch-Alc_alcConfig_c
> --- patches/patch-Alc_alcConfig_c     15 Nov 2017 13:02:42 -0000      1.4
> +++ patches/patch-Alc_alcConfig_c     4 Jun 2018 01:16:29 -0000
> @@ -2,8 +2,8 @@ $OpenBSD: patch-Alc_alcConfig_c,v 1.4 20
>  Index: Alc/alcConfig.c
>  --- Alc/alcConfig.c.orig
>  +++ Alc/alcConfig.c
> -@@ -353,7 +353,7 @@ void ReadALConfig(void)
> -     const char *str;
> +@@ -424,7 +424,7 @@ void ReadALConfig(void)
> +     al_string ppath;
>       FILE *f;
>   
>  -    str = "/etc/openal/alsoft.conf";
> Index: patches/patch-CMakeLists_txt
> ===================================================================
> RCS file: /cvs/ports/audio/openal/patches/patch-CMakeLists_txt,v
> retrieving revision 1.7
> diff -u -p -u -p -r1.7 patch-CMakeLists_txt
> --- patches/patch-CMakeLists_txt      15 Nov 2017 13:02:42 -0000      1.7
> +++ patches/patch-CMakeLists_txt      4 Jun 2018 01:16:29 -0000
> @@ -2,33 +2,12 @@ $OpenBSD: patch-CMakeLists_txt,v 1.7 201
>  Index: CMakeLists.txt
>  --- CMakeLists.txt.orig
>  +++ CMakeLists.txt
> -@@ -843,17 +843,14 @@ ENDIF()
> - 
> - # Check SndIO backend
> - OPTION(ALSOFT_REQUIRE_SNDIO "Require SndIO backend" OFF)
> --FIND_PACKAGE(SoundIO)
> --IF(SOUNDIO_FOUND)
> -+FIND_LIBRARY(SNDIOLIB NAMES sndio)
> -+IF(NOT SNDIOLIB-NOTFOUND)
> -     OPTION(ALSOFT_BACKEND_SNDIO "Enable SndIO backend" ON)
> -     IF(ALSOFT_BACKEND_SNDIO)
> -         SET(HAVE_SNDIO 1)
> -         SET(BACKENDS  "${BACKENDS} SndIO (linked),")
> -         SET(ALC_OBJS  ${ALC_OBJS} Alc/backends/sndio.c)
> --        SET(EXTRA_LIBS ${SOUNDIO_LIBRARIES} ${EXTRA_LIBS})
> --        IF(CMAKE_VERSION VERSION_LESS "2.8.8")
> --            INCLUDE_DIRECTORIES(${SOUNDIO_INCLUDE_DIRS})
> --        ENDIF()
> -+        SET(EXTRA_LIBS ${SNDIOLIB} ${EXTRA_LIBS})
> -     ENDIF()
> - ENDIF()
> - IF(ALSOFT_REQUIRE_SNDIO AND NOT HAVE_SNDIO)
> -@@ -1207,7 +1204,7 @@ ENDIF()
> +@@ -1381,7 +1381,7 @@ endif()
>   # Install alsoft.conf configuration file
>   IF(ALSOFT_CONFIG)
>       INSTALL(FILES alsoftrc.sample
> --            DESTINATION ${SHARE_INSTALL_DIR}/openal
> -+            DESTINATION ${SHARE_INSTALL_DIR}/examples/openal
> +-            DESTINATION ${CMAKE_INSTALL_DATADIR}/openal
> ++            DESTINATION ${CMAKE_INSTALL_DATADIR}/examples/openal
>       )
>       MESSAGE(STATUS "Installing sample configuration")
>       MESSAGE(STATUS "")
> Index: patches/patch-OpenAL32_alBuffer_c
> ===================================================================
> RCS file: /cvs/ports/audio/openal/patches/patch-OpenAL32_alBuffer_c,v
> retrieving revision 1.2
> diff -u -p -u -p -r1.2 patch-OpenAL32_alBuffer_c
> --- patches/patch-OpenAL32_alBuffer_c 15 Nov 2017 13:02:42 -0000      1.2
> +++ patches/patch-OpenAL32_alBuffer_c 4 Jun 2018 01:16:29 -0000
> @@ -2,9 +2,9 @@ $OpenBSD: patch-OpenAL32_alBuffer_c,v 1.
>  Index: OpenAL32/alBuffer.c
>  --- OpenAL32/alBuffer.c.orig
>  +++ OpenAL32/alBuffer.c
> -@@ -1209,7 +1209,7 @@ static ALboolean DecomposeFormat(ALenum format, enum F
> -         { AL_FORMAT_BFORMAT3D_16,      FmtBFormat3D, FmtShort },
> -         { AL_FORMAT_BFORMAT3D_FLOAT32, FmtBFormat3D, FmtFloat },
> +@@ -1171,7 +1171,7 @@ static ALboolean DecomposeUserFormat(ALenum format, en
> +         { AL_FORMAT_BFORMAT3D_FLOAT32, UserFmtBFormat3D, UserFmtFloat },
> +         { AL_FORMAT_BFORMAT3D_MULAW,   UserFmtBFormat3D, UserFmtMulaw },
>       };
>  -    ALuint i;
>  +    volatile ALuint i;
> Index: patches/patch-utils_makehrtf_c
> ===================================================================
> RCS file: /cvs/ports/audio/openal/patches/patch-utils_makehrtf_c,v
> retrieving revision 1.3
> diff -u -p -u -p -r1.3 patch-utils_makehrtf_c
> --- patches/patch-utils_makehrtf_c    15 Nov 2017 13:02:42 -0000      1.3
> +++ patches/patch-utils_makehrtf_c    4 Jun 2018 01:16:29 -0000
> @@ -2,12 +2,12 @@ $OpenBSD: patch-utils_makehrtf_c,v 1.3 2
>  Index: utils/makehrtf.c
>  --- utils/makehrtf.c.orig
>  +++ utils/makehrtf.c
> -@@ -2070,7 +2070,7 @@ static int StoreMhr (const HrirDataT * hData, const ch
> -   step = hData -> mIrSize;
> -   end = hData -> mIrCount * step;
> -   n = hData -> mIrPoints;
> --  srand (0x31DF840C);
> -+  srand_deterministic (0x31DF840C);
> -   for (j = 0; j < end; j += step) {
> -       hpHist = 0;
> -       for (i = 0; i < n; i ++) {
> +@@ -1891,7 +1891,7 @@ static int StoreMhr(const HrirDataT *hData, const char
> +     step = hData->mIrSize;
> +     end = hData->mIrCount * step;
> +     n = hData->mIrPoints;
> +-    srand(0x31DF840C);
> ++    srand_deterministic(0x31DF840C);
> +     for(j = 0;j < end;j += step)
> +     {
> +         hpHist = 0;
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/audio/openal/pkg/PLIST,v
> retrieving revision 1.9
> diff -u -p -u -p -r1.9 PLIST
> --- pkg/PLIST 15 Nov 2017 13:02:42 -0000      1.9
> +++ pkg/PLIST 4 Jun 2018 01:16:29 -0000
> @@ -10,6 +10,10 @@ include/AL/alext.h
>  include/AL/efx-creative.h
>  include/AL/efx-presets.h
>  include/AL/efx.h
> +lib/cmake/
> +lib/cmake/OpenAL/
> +lib/cmake/OpenAL/OpenALConfig-openbsd.cmake
> +lib/cmake/OpenAL/OpenALConfig.cmake
>  @lib lib/libopenal.so.${LIBopenal_VERSION}
>  lib/pkgconfig/openal.pc
>  share/examples/openal/
> @@ -20,3 +24,10 @@ share/openal/
>  share/openal/hrtf/
>  share/openal/hrtf/default-44100.mhr
>  share/openal/hrtf/default-48000.mhr
> +share/openal/presets/
> +share/openal/presets/3D7.1.ambdec
> +share/openal/presets/hexagon.ambdec
> +share/openal/presets/itu5.1.ambdec
> +share/openal/presets/presets.txt
> +share/openal/presets/rectangle.ambdec
> +share/openal/presets/square.ambdec

Reply via email to