summaryrefslogtreecommitdiffstats
path: root/package/libpjsip
diff options
context:
space:
mode:
authorLuca Ceresoli <luca@lucaceresoli.net>2016-03-14 23:18:02 +0100
committerPeter Korsgaard <peter@korsgaard.com>2016-03-15 20:58:36 +0100
commit4507e261c0c3c5f6726a7951c055c6d0c8b86510 (patch)
treef843011a37153414a52e5e73f46ea3ad03695f00 /package/libpjsip
parentf35f54585f9e63400bdbd9c9ba2b9fd67be538ea (diff)
downloadbuildroot-4507e261c0c3c5f6726a7951c055c6d0c8b86510.tar.gz
buildroot-4507e261c0c3c5f6726a7951c055c6d0c8b86510.zip
libpjsip: fix musl build
Fix two different build issues when building against musl. Fixes: http://autobuild.buildroot.net/results/e5c/e5c85272b5561549dfc478264ee6507bb9ad8c9f/ Based on an Alpine Linux patch [0] as suggested by Thomas [1]. [0] http://git.alpinelinux.org/cgit/aports/plain/main/pjproject/musl-fixes.patch [1] http://lists.busybox.net/pipermail/buildroot/2016-March/155056.html Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Diffstat (limited to 'package/libpjsip')
-rw-r--r--package/libpjsip/0001-Use-mutex-types-compatible-with-musl-fixes-musl-buil.patch53
-rw-r--r--package/libpjsip/0002-Replace-__sched_priority-with-sched_priority-fixes-m.patch82
2 files changed, 135 insertions, 0 deletions
diff --git a/package/libpjsip/0001-Use-mutex-types-compatible-with-musl-fixes-musl-buil.patch b/package/libpjsip/0001-Use-mutex-types-compatible-with-musl-fixes-musl-buil.patch
new file mode 100644
index 0000000000..ab3444b710
--- /dev/null
+++ b/package/libpjsip/0001-Use-mutex-types-compatible-with-musl-fixes-musl-buil.patch
@@ -0,0 +1,53 @@
+From ba1057d74aac6c2dde5477bd6a2deea79f14962c Mon Sep 17 00:00:00 2001
+From: Luca Ceresoli <luca@lucaceresoli.net>
+Date: Sat, 12 Mar 2016 15:19:34 +0100
+Subject: [PATCH 1/2] Use mutex types compatible with musl (fixes musl build)
+
+PTHREAD_MUTEX_FAST_NP and PTHREAD_MUTEX_RECURSIVE_NP are not defined
+in the musl C library. Use values that map to the same mutex type in
+GNU libc and uClibc-ng.
+
+Fixes the following build errors when building with musl:
+
+ ../src/pj/os_core_unix.c: In function 'init_mutex':
+ ../src/pj/os_core_unix.c:1128:40: error: 'PTHREAD_MUTEX_FAST_NP' undeclared (first use in this function)
+ rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_FAST_NP);
+ ^
+ ../src/pj/os_core_unix.c:1128:40: note: each undeclared identifier is reported only once for each function it appears in
+ ../src/pj/os_core_unix.c:1138:40: error: 'PTHREAD_MUTEX_RECURSIVE_NP' undeclared (first use in this function)
+ rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE_NP);
+ ^
+
+Original patch:
+http://git.alpinelinux.org/cgit/aports/plain/main/pjproject/musl-fixes.patch
+
+Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
+---
+ pjlib/src/pj/os_core_unix.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/pjlib/src/pj/os_core_unix.c b/pjlib/src/pj/os_core_unix.c
+index 1c87b2f..f08ba27 100644
+--- a/pjlib/src/pj/os_core_unix.c
++++ b/pjlib/src/pj/os_core_unix.c
+@@ -1125,7 +1125,7 @@ static pj_status_t init_mutex(pj_mutex_t *mutex, const char *name, int type)
+ if (type == PJ_MUTEX_SIMPLE) {
+ #if (defined(PJ_LINUX) && PJ_LINUX!=0) || \
+ defined(PJ_HAS_PTHREAD_MUTEXATTR_SETTYPE)
+- rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_FAST_NP);
++ rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_NORMAL);
+ #elif (defined(PJ_RTEMS) && PJ_RTEMS!=0) || \
+ defined(PJ_PTHREAD_MUTEXATTR_T_HAS_RECURSIVE)
+ /* Nothing to do, default is simple */
+@@ -1135,7 +1135,7 @@ static pj_status_t init_mutex(pj_mutex_t *mutex, const char *name, int type)
+ } else {
+ #if (defined(PJ_LINUX) && PJ_LINUX!=0) || \
+ defined(PJ_HAS_PTHREAD_MUTEXATTR_SETTYPE)
+- rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE_NP);
++ rc = pthread_mutexattr_settype(&attr, PTHREAD_MUTEX_RECURSIVE);
+ #elif (defined(PJ_RTEMS) && PJ_RTEMS!=0) || \
+ defined(PJ_PTHREAD_MUTEXATTR_T_HAS_RECURSIVE)
+ // Phil Torre <ptorre@zetron.com>:
+--
+1.9.1
+
diff --git a/package/libpjsip/0002-Replace-__sched_priority-with-sched_priority-fixes-m.patch b/package/libpjsip/0002-Replace-__sched_priority-with-sched_priority-fixes-m.patch
new file mode 100644
index 0000000000..e1cb00229a
--- /dev/null
+++ b/package/libpjsip/0002-Replace-__sched_priority-with-sched_priority-fixes-m.patch
@@ -0,0 +1,82 @@
+From cca93ce25f993c97ef3d96fa32461d5717c30518 Mon Sep 17 00:00:00 2001
+From: Luca Ceresoli <luca@lucaceresoli.net>
+Date: Sat, 12 Mar 2016 15:31:47 +0100
+Subject: [PATCH 2/2] Replace __sched_priority with sched_priority(fixes musl
+ build)
+
+The musl C library defines sched_priority, not __sched_priority as GNU
+libc and uClibc-ng do. Use sched_priority instead.
+
+This does not break compatibility with GNU libc and uClibc-ng because
+both define in sched.h:
+
+ #define sched_priority __sched_priority
+
+Fixes the following build errors when building with musl:
+
+ ../src/samples/siprtp.c: In function 'boost_priority':
+ ../src/samples/siprtp.c:1137:7: error: 'struct sched_param' has no member named '__sched_priority'
+ tp.__sched_priority = max_prio;
+ ^
+ In file included from /home/murray/devel/buildroot/test-musl-eabi/build/libpjsip-2.4.5/pjlib/include/pj/except.h:30:0,
+ from /home/murray/devel/buildroot/test-musl-eabi/build/libpjsip-2.4.5/pjlib/include/pjlib.h:35,
+ from ../src/samples/siprtp.c:76:
+ ../src/samples/siprtp.c:1146:18: error: 'struct sched_param' has no member named '__sched_priority'
+ policy, tp.__sched_priority));
+ ^
+
+Original patch:
+http://git.alpinelinux.org/cgit/aports/plain/main/pjproject/musl-fixes.patch
+
+Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
+---
+ pjsip-apps/src/samples/siprtp.c | 10 +++++-----
+ 1 file changed, 5 insertions(+), 5 deletions(-)
+
+diff --git a/pjsip-apps/src/samples/siprtp.c b/pjsip-apps/src/samples/siprtp.c
+index 796464f..6e32a8f 100644
+--- a/pjsip-apps/src/samples/siprtp.c
++++ b/pjsip-apps/src/samples/siprtp.c
+@@ -1134,7 +1134,7 @@ static void boost_priority(void)
+ PJ_RETURN_OS_ERROR(rc));
+ return;
+ }
+- tp.__sched_priority = max_prio;
++ tp.sched_priority = max_prio;
+
+ rc = sched_setscheduler(0, POLICY, &tp);
+ if (rc != 0) {
+@@ -1143,7 +1143,7 @@ static void boost_priority(void)
+ }
+
+ PJ_LOG(4, (THIS_FILE, "New process policy=%d, priority=%d",
+- policy, tp.__sched_priority));
++ policy, tp.sched_priority));
+
+ /*
+ * Adjust thread scheduling algorithm and priority
+@@ -1156,10 +1156,10 @@ static void boost_priority(void)
+ }
+
+ PJ_LOG(4, (THIS_FILE, "Old thread policy=%d, priority=%d",
+- policy, tp.__sched_priority));
++ policy, tp.sched_priority));
+
+ policy = POLICY;
+- tp.__sched_priority = max_prio;
++ tp.sched_priority = max_prio;
+
+ rc = pthread_setschedparam(pthread_self(), policy, &tp);
+ if (rc != 0) {
+@@ -1169,7 +1169,7 @@ static void boost_priority(void)
+ }
+
+ PJ_LOG(4, (THIS_FILE, "New thread policy=%d, priority=%d",
+- policy, tp.__sched_priority));
++ policy, tp.sched_priority));
+ }
+
+ #else
+--
+1.9.1
+
OpenPOWER on IntegriCloud