GNOME Bugzilla – Bug 754997
cerbero: add recipe for spandsp
Last modified: 2015-10-04 16:56:20 UTC
Created attachment 311282 [details] [review] recipes: add spandsp.recipe Cerbero is missing a recipe for building spandsp, which is needed for building the spandsp plugin in gst-plugins-bad
Attachment 311282 [details] pushed as 6f770b9 - recipes: add spandsp.recipe
Fails to build on Android: https://ci.gstreamer.net/job/cerbero-android-armv7/4029/console
Also all the other Android variants
And Windows: https://ci.gstreamer.net/job/cerbero-cross-mingw32/3059/console
How could I know, I was also waiting for the build results.
Please, wait for all the test results before reverting: commit b98f0e6914fcd5b4209aad28e4e4844565dbfbf1 Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 17:35:26 2015 +0100 spandsp: Compile and package this plugin Compile and package spandsp plugin. https://bugzilla.gnome.org/show_bug.cgi?id=754997 (no need to revert the spandsp recipe patch). For Android (and cross compilation) we'll need: os.environ['ac_cv_func_malloc_0_nonnull'] ='yes' os.environ['ac_cv_func_realloc_0_nonnull'] ='yes' It's a bug in autoconf
Sure, I was not going to revert it at this point, we're not going to release 1.7.1 tomorrow ;)
Also the Debian packaging fails, probably because spandsp:libs is not part of any .package: https://ci.gstreamer.net/job/cerbero-debian/4028/console But you probably know that already :)
Ok, this should make some progress already. Windows shall still fail, Olivier pointer to that: http://stackoverflow.com/questions/10059200/how-do-i-fix-undefined-reference-to-imp commit 98b2f6d036fc6705d6a075c38663f78b671082bc Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 18:50:46 2015 +0100 spandsp: Also package it This is needed to generate correct packages. Fortunatly the debian packager complains, otherwise we just have packages with missing files. https://bugzilla.gnome.org/show_bug.cgi?id=754997 commit 9bb909e54f63cbf328fed998d999d4a1eb29bbde Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 18:49:46 2015 +0100 spandsp: Skip two autoconf check when cross-compiling In cross-compilation, some autoconf check failed when they should not which lead to link error. https://bugzilla.gnome.org/show_bug.cgi?id=754997
Is someone going to work on fixing it for Windows?
And still waiting for the builder to catchup, this one should help too: commit b5ba15b218bce8587de29fd33b713400f427a1f2 Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 20:06:06 2015 +0100 spandsp: Fix build with mingw This is a specific fix, an upstream solution would be nicer eventually. We basically properly select the naming convention and ensure we use the stdcall convention for gethostname. https://bugzilla.gnome.org/show_bug.cgi?id=754997
commit 851b094c061137c79f243f7bf9e5d80cdbc09bdb Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 22:42:36 2015 +0100 spandsp: Always skip ac_cv_func_ check for Android This should fix a non-cross compiling case, Android x86_54 where this check was failing. https://bugzilla.gnome.org/show_bug.cgi?id=754997 commit 5d6929c7c8258aec95fd214a3c241ed8ba14b755 Author: Nicolas Dufresne <nicolas.dufresne@collabora.com> Date: Sat Oct 3 22:35:01 2015 +0100 spandsp: Set empty patches array By not setting an empty patches array, we endup with the patches being randomly applied to other recipies. https://bugzilla.gnome.org/show_bug.cgi?id=754997
It's also failing for cerbero/fedora builds : https://ci.gstreamer.net/job/cerbero-fedora/3450/console
(In reply to Edward Hervey from comment #13) > It's also failing for cerbero/fedora builds : > https://ci.gstreamer.net/job/cerbero-fedora/3450/console That fully works locally (F22), with none of the other "File not found" error we see with cpio in that trace. Is it possible to check if there isn't some disk space issue there, and maybe a wipe ?
All good now