[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