[Buildroot] [git commit] libpjsip: fix musl build

Peter Korsgaard peter at korsgaard.com
Tue Mar 15 19:58:36 UTC 2016


commit: https://git.buildroot.net/buildroot/commit/?id=4507e261c0c3c5f6726a7951c055c6d0c8b86510
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

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 at free-electrons.com>
Signed-off-by: Luca Ceresoli <luca at lucaceresoli.net>
Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
---
 ...ypes-compatible-with-musl-fixes-musl-buil.patch | 53 ++++++++++++++
 ...ched_priority-with-sched_priority-fixes-m.patch | 82 ++++++++++++++++++++++
 2 files changed, 135 insertions(+)

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 0000000..ab3444b
--- /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 at 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 at 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 at 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 0000000..e1cb002
--- /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 at 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 at 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
+


More information about the buildroot mailing list