[opus] [PATCH] opusfile configury fixes.

Ozkan Sezer sezeroz at gmail.com
Wed May 8 11:46:31 PDT 2013


[this time compeled the mail].

The autogen.sh of opusfile doesn't work for me:

Updating build configuration files for opusfile, please wait....
configure.ac:8: installing `./install-sh'
configure.ac:8: installing `./missing'
Makefile.am:10: Libtool library used but `LIBTOOL' is undefined
Makefile.am:10:   The usual way to define `LIBTOOL' is to add `AC_PROG_LIBTOOL'
Makefile.am:10:   to `configure.ac' and run `aclocal' and `autoconf' again.
Makefile.am:10:   If `AC_PROG_LIBTOOL' is in `configure.ac', make sure
Makefile.am:10:   its definition is in aclocal's search path.
Makefile.am: installing `./depcomp'
autoreconf: automake failed with exit status: 1

So I made configure.ac of opusfile to look a little bit more like that
of opus as seen in the first two hunks of the patch inlined below. The
last hunk touching the AC_CHECK_HEADER use is needed for me, otherwise
the generated configure is broken (using autoconf-2.63, newer versions
are _possibly_ OK with the existing syntax).

Patch is below.

--- configure.ac~
+++ configure.ac
@@ -2,16 +2,15 @@

 AC_INIT([opusfile], m4_esyscmd([doc/git-version.sh]))

+m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
+
+AC_CONFIG_MACRO_DIR([m4])
+
 AC_USE_SYSTEM_EXTENSIONS
 AC_SYS_LARGEFILE

 AM_INIT_AUTOMAKE([1.11 foreign])
 AM_MAINTAINER_MODE([enable])
-LT_INIT
-
-m4_ifdef([AM_SILENT_RULES], [AM_SILENT_RULES([yes])])
-
-AC_CONFIG_MACRO_DIR([m4])

 dnl Library versioning for libtool.
 dnl Please update these for releases.
@@ -29,6 +28,10 @@
 AC_SUBST(OP_LT_REVISION)
 AC_SUBST(OP_LT_AGE)

+AC_CANONICAL_HOST
+AM_PROG_LIBTOOL
+AM_PROG_CC_C_O
+
 CC_CHECK_CFLAGS_APPEND(
   [-std=c89 -pedantic -Wall -Wextra -Wno-parentheses -Wno-long-long])

@@ -62,15 +65,15 @@

 AM_COND_IF(OP_WIN32,
   AS_IF([test "x$enable_http" != "xno"],
-    AC_CHECK_HEADER([winsock2.h],,
+    AC_CHECK_HEADER([winsock2.h],[],
       AC_MSG_WARN([HTTP support requires a Winsock socket library.])
-      enable_http=no
+      enable_http=no, []
     )
   ),
   AS_IF([test "x$enable_http" != "xno"],
-    AC_CHECK_HEADER([sys/socket.h],,
+    AC_CHECK_HEADER([sys/socket.h],[],
       AC_MSG_WARN([HTTP support requires a POSIX socket library.])
-      enable_http=no
+      enable_http=no, []
     )
   )
 )

--
O.S.


More information about the opus mailing list