Bug 95738

Thanks for this. Could I possibly persuade you to try to get this included in the webrtc upstream? That's where we take this code from.

  https://chromium.googlesource.com/external/webrtc/+/master/
  https://webrtc.org/contributing/

The attached patch does not fix the issue fully:


https://buildd.debian.org/status/logs.php?pkg=webrtc-audio-processing&ver=0.2-2&suite=experimental

(In reply to Arun Raghavan from comment #1)
> Thanks for this. Could I possibly persuade you to try to get this included
> in the webrtc upstream? That's where we take this code from.
> 
>   https://chromium.googlesource.com/external/webrtc/+/master/
>   https://webrtc.org/contributing/

sure, i will try.

(In reply to Felipe Sateler from comment #2)
> The attached patch does not fix the issue fully:
> 
> 
> https://buildd.debian.org/status/logs.php?pkg=webrtc-audio-processing&ver=0.
> 2-2&suite=experimental

it's another issue. please try following patch which should fix your issue


diff -up webrtc-audio-processing-0.2/webrtc/typedefs.h.typedef webrtc-audio-processing-0.2/webrtc/typedefs.h
--- webrtc-audio-processing-0.2/webrtc/typedefs.h.typedef       2016-05-12 09:08:53.885000410 -0500
+++ webrtc-audio-processing-0.2/webrtc/typedefs.h       2016-05-12 09:12:38.006851953 -0500
@@ -48,7 +48,19 @@
 #define WEBRTC_ARCH_32_BITS
 #define WEBRTC_ARCH_LITTLE_ENDIAN
 #else
-#error Please add support for your architecture in typedefs.h
+/* instead of failing, use typical unix defines... */
+#if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__
+#define WEBRTC_ARCH_LITTLE_ENDIAN
+#elif __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__
+#define WEBRTC_ARCH_BIG_ENDIAN
+#else
+#error __BYTE_ORDER__ is not defined
+#endif
+#if defined(__LP64__)
+#define WEBRTC_ARCH_64_BITS
+#else
+#define WEBRTC_ARCH_32_BITS
+#endif
 #endif
 
 #if !(defined(WEBRTC_ARCH_LITTLE_ENDIAN) ^ defined(WEBRTC_ARCH_BIG_ENDIAN))

@than

Thanks, that fixes the big endian issues.

-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/pulseaudio/pulseaudio/issues/127.