shithub: sox

Download patch

ref: 0a60ae593cc8a64bb3d6a6c5f0f5df684f64a993
parent: 07de8a77a862e6800b95a8d3a61c6b4e41362755
author: Mans Rullgard <mans@mansr.com>
date: Wed Aug 12 18:42:19 EDT 2020

sndfile: use separate variable names for library and format handler

Use LIBSNDFILE when referring to the library and SNDFILE for the format
handler.

--- a/configure.ac
+++ b/configure.ac
@@ -521,7 +521,7 @@
 
 dnl When enable_dl_sndfile, do not let SOX_PATH_SNDFILE add libraries
 dnl to be linked in (since they will be dlopen()'ed instead).
-ac_sox_save_SNDFILE_LIBS="$SNDFILE_LIBS"
+ac_sox_save_SNDFILE_LIBS="$LIBSNDFILE_LIBS"
 SOX_PATH_SNDFILE(tmp_using_sndfile=$opt_default, tmp_using_sndfile=no)
 AC_ARG_ENABLE(dl_sndfile,
 	      AS_HELP_STRING([--enable-dl-sndfile], 
@@ -529,10 +529,10 @@
 			     enable_dl_sndfile=$enableval, enable_dl_sndfile=no)
 if test "x$tmp_using_sndfile" = "xyes"; then
   if test "x$using_libltdl" = "xyes" -a "x$enable_dl_sndfile" = "xyes"; then
-    AC_DEFINE(DL_SNDFILE, 1, [Define to dlopen() sndfile.]) 
+    AC_DEFINE(DL_LIBSNDFILE, 1, [Define to dlopen() sndfile.])
     dnl When enable_dl_sndfile, do not let SOX_PATH_SNDFILE add libraries
     dnl to be linked in (since they will be dlopen()'ed instead).
-    SNDFILE_LIBS="$ac_sox_save_SNDFILE_LIBS"
+    LIBSNDFILE_LIBS="$ac_sox_save_SNDFILE_LIBS"
   else
     enable_dl_sndfile="no"
   fi
--- a/m4/sndfile.m4
+++ b/m4/sndfile.m4
@@ -8,20 +8,20 @@
 dnl Brendan Cully <brendan@xiph.org> 20030612
 dnl
 # SOX_PATH_SNDFILE([ACTION-IF-FOUND [, ACTION-IF-NOT-FOUND]])
-# Test for libsndfile, and define SNDFILE_CFLAGS and SNDFILE_LIBS
+# Test for libsndfile, and define LIBSNDFILE_CFLAGS and LIBSNDFILE_LIBS
 AC_DEFUN([SOX_PATH_SNDFILE],
 [dnl
 # Step 1: Use pkg-config if available
 m4_ifdef([PKG_CHECK_MODULES],
   [# PKG_CHECK_MODULES available
-  PKG_CHECK_MODULES(SNDFILE, sndfile, have_sndfile="maybe", 
+  PKG_CHECK_MODULES(LIBSNDFILE, sndfile, have_sndfile="maybe",
                     have_sndfile="no")],
   [# Step 2: use pkg-config manually if available
   AC_PATH_PROG([PKGCONFIG], [pkg-config], [none])
   if test "$PKGCONFIG" != "none" && `$PKGCONFIG --exists sndfile`
   then
-    SNDFILE_CFLAGS=`$PKGCONFIG --cflags sndfile`" $ac_save_SNDFILE_CFLAGS $SNDFILE_CFLAGS"
-    SNDFILE_LIBS=`$PKGCONFIG --libs sndfile`" $ac_save_SNDFILE_LIBS $SNDFILE_LIBS"
+    LIBSNDFILE_CFLAGS=`$PKGCONFIG --cflags sndfile`" $ac_save_SNDFILE_CFLAGS $LIBSNDFILE_CFLAGS"
+    LIBSNDFILE_LIBS=`$PKGCONFIG --libs sndfile`" $ac_save_SNDFILE_LIBS $LIBSNDFILE_LIBS"
     have_sndfile="maybe"
   else
     have_sndfile="no"
@@ -35,7 +35,7 @@
   # As a last resort, just hope that header and ilbrary can
   # be found in default paths and that it doesn't need
   # to link against any other libraries.
-  SNDFILE_LIBS="-lsndfile $SNDFILE_LIBS"
+  LIBSNDFILE_LIBS="-lsndfile $LIBSNDFILE_LIBS"
   have_sndfile="maybe"
 fi
 
@@ -45,8 +45,8 @@
 then
   ac_save_CFLAGS="$CFLAGS"
   ac_save_LIBS="$LIBS"
-  CFLAGS="$CFLAGS $SNDFILE_CFLAGS"
-  LIBS="$LIBS $SNDFILE_LIBS"
+  CFLAGS="$CFLAGS $LIBSNDFILE_CFLAGS"
+  LIBS="$LIBS $LIBSNDFILE_LIBS"
   AC_CHECK_HEADER([sndfile.h], [
     AC_DEFINE([HAVE_SNDFILE_H], 1, [Define if you have <sndfile.h>])
     AC_CHECK_FUNC([sf_open_virtual], [
--- a/src/optional-fmts.am
+++ b/src/optional-fmts.am
@@ -252,43 +252,43 @@
 if HAVE_SNDFILE
 if STATIC_SNDFILE
   libsox_la_SOURCES += sndfile.c caf.c mat4.c mat5.c paf.c fap.c w64.c xi.c pvf.c sd2.c
-  libsox_la_CFLAGS += @SNDFILE_CFLAGS@
+  libsox_la_CFLAGS += @LIBSNDFILE_CFLAGS@
 if STATIC_LIBSOX_ONLY
-  sox_LDADD += @SNDFILE_LIBS@
+  sox_LDADD += @LIBSNDFILE_LIBS@
 else
-  libsox_la_LIBADD += @SNDFILE_LIBS@
+  libsox_la_LIBADD += @LIBSNDFILE_LIBS@
 endif
 else
 libsox_fmt_sndfile_la_SOURCES = sndfile.c
-libsox_fmt_sndfile_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_sndfile_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_sndfile_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_sndfile_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_caf_la_SOURCES = caf.c sndfile.c
-libsox_fmt_caf_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_caf_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_caf_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_caf_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_mat4_la_SOURCES = mat4.c sndfile.c
-libsox_fmt_mat4_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_mat4_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_mat4_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_mat4_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_mat5_la_SOURCES = mat5.c sndfile.c
-libsox_fmt_mat5_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_mat5_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_mat5_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_mat5_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_paf_la_SOURCES = paf.c sndfile.c
-libsox_fmt_paf_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_paf_la_LIBADD = libsox.la  @SNDFILE_LIBS@
+libsox_fmt_paf_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_paf_la_LIBADD = libsox.la  @LIBSNDFILE_LIBS@
 libsox_fmt_fap_la_SOURCES = fap.c sndfile.c
-libsox_fmt_fap_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_fap_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_fap_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_fap_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_w64_la_SOURCES = w64.c sndfile.c
-libsox_fmt_w64_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_w64_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_w64_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_w64_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_xi_la_SOURCES = xi.c sndfile.c
-libsox_fmt_wi_la_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_xi_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_wi_la_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_xi_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_pvf_la_SOURCES = pvf.c sndfile.c
-libsox_fmt_pvf_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_pvf_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_pvf_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_pvf_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 libsox_fmt_sd2_la_SOURCES = sd2.c sndfile.c
-libsox_fmt_sd2_CFLAGS = @SNDFILE_CFLAGS@
-libsox_fmt_sd2_la_LIBADD = libsox.la @SNDFILE_LIBS@
+libsox_fmt_sd2_CFLAGS = @LIBSNDFILE_CFLAGS@
+libsox_fmt_sd2_la_LIBADD = libsox.la @LIBSNDFILE_LIBS@
 pkglib_LTLIBRARIES += libsox_fmt_sndfile.la libsox_fmt_caf.la libsox_fmt_mat4.la libsox_fmt_mat5.la libsox_fmt_paf.la libsox_fmt_fap.la libsox_fmt_w64.la libsox_fmt_xi.la libsox_fmt_pvf.la libsox_fmt_sd2.la
 endif
 endif
--- a/src/sndfile.c
+++ b/src/sndfile.c
@@ -30,7 +30,7 @@
 
 static const char* const sndfile_library_names[] =
 {
-#ifdef DL_SNDFILE
+#ifdef DL_LIBSNDFILE
   "libsndfile",
   "libsndfile-1",
   "cygsndfile-1",
@@ -38,7 +38,7 @@
   NULL
 };
 
-#ifdef DL_SNDFILE
+#ifdef DL_LIBSNDFILE
   #define SNDFILE_FUNC      LSX_DLENTRY_DYNAMIC
   #define SNDFILE_FUNC_STOP LSX_DLENTRY_STUB
 #else
@@ -48,7 +48,7 @@
 #else
   #define SNDFILE_FUNC_STOP LSX_DLENTRY_STUB
 #endif
-#endif /* DL_SNDFILE */
+#endif /* DL_LIBSNDFILE */
 
 #define SNDFILE_FUNC_OPEN(f,x) \
   SNDFILE_FUNC(f,x, SNDFILE*, sf_open_virtual, (SF_VIRTUAL_IO *sfvirtual, int mode, SF_INFO *sfinfo, void *user_data))