[Asterisk-code-review] pjproject-bundled: Cleanups for reported issues (asterisk[13])
Anonymous Coward
asteriskteam at digium.com
Fri Mar 25 18:11:38 CDT 2016
Anonymous Coward #1000019 has submitted this change and it was merged.
Change subject: pjproject-bundled: Cleanups for reported issues
......................................................................
pjproject-bundled: Cleanups for reported issues
PortAudio should no longer be required
PJSIP_MAX_PKT_LEN is now 6000
Older autoconf issue fixed. (CentOS 6)
Change-Id: I463fa9586cbe7c6b3b603289f535bd8e361611dd
---
M bootstrap.sh
M configure
M configure.ac
M third-party/pjproject/Makefile.rules
A third-party/pjproject/patches/0002-aconfigure.ac-Fix-autoconf-issue-with-opencore-amrnb.patch
M third-party/pjproject/patches/config_site.h
6 files changed, 57 insertions(+), 10 deletions(-)
Approvals:
Mark Michelson: Looks good to me, approved
Anonymous Coward #1000019: Verified
Joshua Colp: Looks good to me, but someone else must approve
diff --git a/bootstrap.sh b/bootstrap.sh
index 3497e24..051ee3e 100755
--- a/bootstrap.sh
+++ b/bootstrap.sh
@@ -46,7 +46,7 @@
echo "Generating the configure script ..."
-aclocal${MY_AM_VER} -I autoconf
+aclocal${MY_AM_VER} -I autoconf `find third-party/ -maxdepth 1 -type d -printf "-I %p "`
autoconf${MY_AC_VER}
autoheader${MY_AC_VER}
automake${MY_AM_VER} --add-missing --copy 2>/dev/null
diff --git a/configure b/configure
index f2f0f78..14c5a0e 100755
--- a/configure
+++ b/configure
@@ -13602,7 +13602,7 @@
We can't simply define LARGE_OFF_T to be 9223372036854775807,
since some C++ compilers masquerading as C compilers
incorrectly reject 9223372036854775807. */
-#define LARGE_OFF_T ((((off_t) 1 << 31) << 31) - 1 + (((off_t) 1 << 31) << 31))
+#define LARGE_OFF_T (((off_t) 1 << 62) - 1 + ((off_t) 1 << 62))
int off_t_is_large[(LARGE_OFF_T % 2147483629 == 721
&& LARGE_OFF_T % 2147483647 == 1)
? 1 : -1];
@@ -13648,7 +13648,7 @@
We can't simply define LARGE_OFF_T to be 9223372036854775807,
since some C++ compilers masquerading as C compilers
incorrectly reject 9223372036854775807. */
-#define LARGE_OFF_T ((((off_t) 1 << 31) << 31) - 1 + (((off_t) 1 << 31) << 31))
+#define LARGE_OFF_T (((off_t) 1 << 62) - 1 + ((off_t) 1 << 62))
int off_t_is_large[(LARGE_OFF_T % 2147483629 == 721
&& LARGE_OFF_T % 2147483647 == 1)
? 1 : -1];
@@ -13672,7 +13672,7 @@
We can't simply define LARGE_OFF_T to be 9223372036854775807,
since some C++ compilers masquerading as C compilers
incorrectly reject 9223372036854775807. */
-#define LARGE_OFF_T ((((off_t) 1 << 31) << 31) - 1 + (((off_t) 1 << 31) << 31))
+#define LARGE_OFF_T (((off_t) 1 << 62) - 1 + ((off_t) 1 << 62))
int off_t_is_large[(LARGE_OFF_T % 2147483629 == 721
&& LARGE_OFF_T % 2147483647 == 1)
? 1 : -1];
@@ -13717,7 +13717,7 @@
We can't simply define LARGE_OFF_T to be 9223372036854775807,
since some C++ compilers masquerading as C compilers
incorrectly reject 9223372036854775807. */
-#define LARGE_OFF_T ((((off_t) 1 << 31) << 31) - 1 + (((off_t) 1 << 31) << 31))
+#define LARGE_OFF_T (((off_t) 1 << 62) - 1 + ((off_t) 1 << 62))
int off_t_is_large[(LARGE_OFF_T % 2147483629 == 721
&& LARGE_OFF_T % 2147483647 == 1)
? 1 : -1];
@@ -13741,7 +13741,7 @@
We can't simply define LARGE_OFF_T to be 9223372036854775807,
since some C++ compilers masquerading as C compilers
incorrectly reject 9223372036854775807. */
-#define LARGE_OFF_T ((((off_t) 1 << 31) << 31) - 1 + (((off_t) 1 << 31) << 31))
+#define LARGE_OFF_T (((off_t) 1 << 62) - 1 + ((off_t) 1 << 62))
int off_t_is_large[(LARGE_OFF_T % 2147483629 == 721
&& LARGE_OFF_T % 2147483647 == 1)
? 1 : -1];
@@ -24216,7 +24216,6 @@
if test "$USE_PJPROJECT" != "no" ; then
if test "$PJPROJECT_BUNDLED" = "yes" ; then
-
{ $as_echo "$as_me:${as_lineno-$LINENO}: checking for embedded pjproject (may have to download)" >&5
$as_echo_n "checking for embedded pjproject (may have to download)... " >&6; }
diff --git a/configure.ac b/configure.ac
index d0a1dec..7571f7f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2168,7 +2168,6 @@
if test "$USE_PJPROJECT" != "no" ; then
if test "$PJPROJECT_BUNDLED" = "yes" ; then
- AC_CONFIG_MACRO_DIR(third-party/pjproject)
PJPROJECT_CONFIGURE([$PJPROJECT_DIR])
else
AST_PKG_CONFIG_CHECK([PJPROJECT], [libpjproject])
diff --git a/third-party/pjproject/Makefile.rules b/third-party/pjproject/Makefile.rules
index f39629b..062793f 100644
--- a/third-party/pjproject/Makefile.rules
+++ b/third-party/pjproject/Makefile.rules
@@ -2,6 +2,6 @@
# Even though we're not installing pjproject, we're setting prefix to /opt/pjproject to be safe
PJPROJECT_CONFIG_OPTS = --prefix=/opt/pjproject --with-external-speex --with-external-gsm --with-external-srtp \
- --with-external-pa --disable-video --disable-v4l2 --disable-sound \
- --disable-opencore-amr --disable-ilbc-codec --without-libyuv --disable-g7221-codec \
+ --disable-video --disable-v4l2 --disable-sound --disable-opencore-amr --disable-ilbc-codec \
+ --without-libyuv --disable-g7221-codec \
--enable-epoll
diff --git a/third-party/pjproject/patches/0002-aconfigure.ac-Fix-autoconf-issue-with-opencore-amrnb.patch b/third-party/pjproject/patches/0002-aconfigure.ac-Fix-autoconf-issue-with-opencore-amrnb.patch
new file mode 100644
index 0000000..04d1d97
--- /dev/null
+++ b/third-party/pjproject/patches/0002-aconfigure.ac-Fix-autoconf-issue-with-opencore-amrnb.patch
@@ -0,0 +1,48 @@
+From 1281b60a1807d1285b101b6eb61c6478f29785fe Mon Sep 17 00:00:00 2001
+From: George Joseph <george.joseph at fairview5.com>
+Date: Wed, 23 Mar 2016 07:48:52 -0600
+Subject: [PATCH] aconfigure.ac: Fix autoconf issue with opencore-amrnb on
+ older systems
+
+autoconf 2.63 on CentOS6 produces a bad ./aconfigure file related to
+opencore-amrnb.
+
+./aconfigure: line 15158: syntax error near unexpected token `fi'
+
+To get around this, a 'true;' needed to be added to the Ok case of
+AC_ARG_WITH(opencore-amrnb)
+---
+ aconfigure | 3 +++
+ aconfigure.ac | 2 +-
+ 2 files changed, 4 insertions(+), 1 deletion(-)
+
+diff --git a/aconfigure b/aconfigure
+index 33a08f5..4c122c2 100755
+--- a/aconfigure
++++ b/aconfigure
+@@ -7908,6 +7908,9 @@ fi
+ # Check whether --with-opencore-amrnb was given.
+ if test "${with_opencore_amrnb+set}" = set; then :
+ withval=$with_opencore_amrnb; as_fn_error $? "This option is obsolete and replaced by --with-opencore-amr=DIR" "$LINENO" 5
++else
++ true;
++
+ fi
+
+
+diff --git a/aconfigure.ac b/aconfigure.ac
+index 3e88124..5d3e833 100644
+--- a/aconfigure.ac
++++ b/aconfigure.ac
+@@ -1631,7 +1631,7 @@ AC_ARG_WITH(opencore-amrnb,
+ AC_HELP_STRING([--with-opencore-amrnb=DIR],
+ [This option is obsolete and replaced by --with-opencore-amr=DIR]),
+ [AC_MSG_ERROR(This option is obsolete and replaced by --with-opencore-amr=DIR)],
+- []
++ [true;]
+ )
+
+ dnl # opencore-amr alt prefix
+--
+2.5.0
+
diff --git a/third-party/pjproject/patches/config_site.h b/third-party/pjproject/patches/config_site.h
index 544c1e8..840d8b2 100644
--- a/third-party/pjproject/patches/config_site.h
+++ b/third-party/pjproject/patches/config_site.h
@@ -29,6 +29,7 @@
#define PJ_HAS_STRICMP_ALNUM 0
#define PJ_HASH_USE_OWN_TOLOWER 1
#define PJSIP_UNESCAPE_IN_PLACE 1
+#define PJSIP_MAX_PKT_LEN 6000
#undef PJ_TODO
#define PJ_TODO(x)
--
To view, visit https://gerrit.asterisk.org/2449
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I463fa9586cbe7c6b3b603289f535bd8e361611dd
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
More information about the asterisk-code-review
mailing list