[Buildroot] [PATCH 2/2] package/janus-gateway: fix build of audiobridge without libogg

Fabrice Fontaine fontaine.fabrice at gmail.com
Sat Jun 20 14:35:01 UTC 2020


Fixes:
 - http://autobuild.buildroot.org/results/e9dd68127bd8e6337eac23d43e99399ec370b1e0

Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
---
 ...diobridge.c-fix-build-without-libogg.patch | 74 +++++++++++++++++++
 1 file changed, 74 insertions(+)
 create mode 100644 package/janus-gateway/0003-plugins-janus_audiobridge.c-fix-build-without-libogg.patch

diff --git a/package/janus-gateway/0003-plugins-janus_audiobridge.c-fix-build-without-libogg.patch b/package/janus-gateway/0003-plugins-janus_audiobridge.c-fix-build-without-libogg.patch
new file mode 100644
index 0000000000..1d25b1a071
--- /dev/null
+++ b/package/janus-gateway/0003-plugins-janus_audiobridge.c-fix-build-without-libogg.patch
@@ -0,0 +1,74 @@
+From b728e30508101093331b08ac617e6315ba82caa1 Mon Sep 17 00:00:00 2001
+From: Fabrice Fontaine <fontaine.fabrice at gmail.com>
+Date: Sat, 20 Jun 2020 15:38:08 +0200
+Subject: [PATCH] plugins/janus_audiobridge.c: fix build without libogg
+
+Build without libogg is broken since commit
+53761d07c5456424d368c708fda2757e6606a4bf:
+
+plugins/janus_audiobridge.c:1337:41: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 1337 |  janus_audiobridge_file_free(participant->annc);
+      |                                         ^~
+  CC       plugins/plugins_libjanus_nosip_la-janus_nosip.lo
+plugins/janus_audiobridge.c: In function 'janus_audiobridge_mixer_thread':
+plugins/janus_audiobridge.c:6394:9: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 6394 |     if(p->annc == NULL || g_atomic_int_get(&p->destroyed)) {
+      |         ^~
+plugins/janus_audiobridge.c:6398:16: warning: implicit declaration of function 'janus_audiobridge_file_read'; did you mean 'janus_audiobridge_mixer_thread'? [-Wimplicit-function-declaration]
+ 6398 |     int read = janus_audiobridge_file_read(p->annc, p->decoder, resampled, sizeof(resampled));
+      |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~
+      |                janus_audiobridge_mixer_thread
+plugins/janus_audiobridge.c:6398:16: warning: nested extern declaration of 'janus_audiobridge_file_read' [-Wnested-externs]
+plugins/janus_audiobridge.c:6398:45: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 6398 |     int read = janus_audiobridge_file_read(p->annc, p->decoder, resampled, sizeof(resampled));
+      |                                             ^~
+plugins/janus_audiobridge.c:6401:10: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 6401 |      if(p->annc->started) {
+      |          ^~
+plugins/janus_audiobridge.c:6428:10: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 6428 |     if(!p->annc->started) {
+      |          ^~
+plugins/janus_audiobridge.c:6430:7: error: 'janus_audiobridge_participant' {aka 'struct janus_audiobridge_participant'} has no member named 'annc'
+ 6430 |      p->annc->started = TRUE;
+      |       ^~
+  CC       plugins/plugins_libjanus_streaming_la-janus_streaming.lo
+
+Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
+[Upstream status: https://github.com/meetecho/janus-gateway/pull/2238]
+---
+ plugins/janus_audiobridge.c | 4 ++++
+ 1 file changed, 4 insertions(+)
+
+diff --git a/plugins/janus_audiobridge.c b/plugins/janus_audiobridge.c
+index 200a1f1d..ee931e24 100644
+--- a/plugins/janus_audiobridge.c
++++ b/plugins/janus_audiobridge.c
+@@ -1334,7 +1334,9 @@ static void janus_audiobridge_participant_free(const janus_refcount *participant
+ 		}
+ 		g_async_queue_unref(participant->outbuf);
+ 	}
++#ifdef HAVE_LIBOGG
+ 	janus_audiobridge_file_free(participant->annc);
++#endif
+ 	g_free(participant);
+ }
+ 
+@@ -6385,6 +6387,7 @@ static void *janus_audiobridge_mixer_thread(void *data) {
+ 			janus_mutex_unlock(&p->qmutex);
+ 			ps = ps->next;
+ 		}
++#ifdef HAVE_LIBOGG
+ 		/* If there are announcements playing, mix those too */
+ 		GList *anncs_list = g_hash_table_get_values(audiobridge->anncs);
+ 		if(anncs_list != NULL) {
+@@ -6458,6 +6461,7 @@ static void *janus_audiobridge_mixer_thread(void *data) {
+ 				ps = ps->next;
+ 			}
+ 		}
++#endif
+ 		/* Are we recording the mix? (only do it if there's someone in, though...) */
+ 		if(audiobridge->recording != NULL && g_list_length(participants_list) > 0) {
+ 			for(i=0; i<samples; i++) {
+-- 
+2.26.2
+
-- 
2.26.2




More information about the buildroot mailing list