Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v12 01/31] package/freeswitch: new package
Date: Wed, 13 Apr 2016 23:02:19 +0200	[thread overview]
Message-ID: <20160413230219.73e299ea@free-electrons.com> (raw)
In-Reply-To: <1460281299-2385-2-git-send-email-bernd.kuhls@t-online.de>

Hello,

On Sun, 10 Apr 2016 11:41:09 +0200, Bernd Kuhls wrote:
> This commit adds freeswitch without any configured modules and with a
> minimal set of non-optional dependencies. All other dependencies and
> modules will be added by further patches in this series.
> 
> PLease note that freeswitch source repo bundles some libraries which are
> also available as buildroot packages. The freeswitch build system does
> not allow to use system libraries in these cases:
> 
> apr, apr-util, libsrtp, libvpx, libyuv, sofia-sip, tiff
> 
> The reason are patches to these packages by the freeswitch project which
> are not yet upstream. There is an open JIRA report for this situation:
> https://freeswitch.org/jira/si/jira.issueviews:issue-html/FS-353/FS-353.html
> 
> More historic infos can be found here:
> http://article.gmane.org/gmane.comp.telephony.freeswitch.devel/2715
> https://freeswitch.org/the-missing-link/
> 
> In the 1.6.7 version bump libvpx & libyuv were also moved in-tree:
> https://freeswitch.org/stash/projects/FS/repos/freeswitch/commits/febe0f8dacea2d2a31902b3dc469be757f8c3c4d
> https://freeswitch.org/stash/projects/FS/repos/freeswitch/commits/44da905b4f6b3b35e94b4948fb70dec7b5071ded
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

I'm sorry, but this still doesn't build, with the exact same error I
had in my previous test of the freeswitch package:

BR2_arm=y
BR2_TOOLCHAIN_EXTERNAL=y
BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y
BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y
BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-2016.02-3-g762b7c9.tar.bz2"
BR2_TOOLCHAIN_EXTERNAL_GCC_4_7=y
BR2_TOOLCHAIN_EXTERNAL_HEADERS_3_10=y
BR2_TOOLCHAIN_EXTERNAL_LOCALE=y
# BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set
BR2_TOOLCHAIN_EXTERNAL_INET_RPC=y
BR2_TOOLCHAIN_EXTERNAL_CXX=y
BR2_INIT_NONE=y
BR2_SYSTEM_BIN_SH_NONE=y
# BR2_PACKAGE_BUSYBOX is not set
BR2_PACKAGE_FREESWITCH=y
# BR2_TARGET_ROOTFS_TAR is not set

gives:

  CCLD   fs_cli
libs/libzrtp/third_party/bnlib/libbn.a(lbnmem.o): In function `lbnMemWipe':
/home/thomas/projets/buildroot/output/build/freeswitch-1.6.7/libs/libzrtp/third_party/bnlib/lbnmem.c:69: undefined reference to `zrtp_memset'
libs/libzrtp/third_party/bnlib/libbn.a(lbnmem.o): In function `lbnMemAlloc':
/home/thomas/projets/buildroot/output/build/freeswitch-1.6.7/libs/libzrtp/third_party/bnlib/lbnmem.c:78: undefined reference to `zrtp_sys_alloc'
libs/libzrtp/third_party/bnlib/libbn.a(lbnmem.o): In function `lbnMemFree':
/home/thomas/projets/buildroot/output/build/freeswitch-1.6.7/libs/libzrtp/third_party/bnlib/lbnmem.c:87: undefined reference to `zrtp_sys_free'
collect2: error: ld returned 1 exit status
Makefile:1259: recipe for target 'fs_cli' failed
make[4]: *** [fs_cli] Error 1
make[4]: *** Waiting for unfinished jobs....

*** Warning: Linking the shared library libfreeswitch.la against the
*** static library libs/libzrtp/libzrtp.a is not portable!

*** Warning: Linking the shared library libfreeswitch.la against the
*** static library libs/libzrtp/third_party/bnlib/libbn.a is not portable!
Makefile:2803: recipe for target 'all-recursive' failed
make[3]: *** [all-recursive] Error 1
Makefile:1066: recipe for target 'all' failed
make[2]: *** [all] Error 2
package/pkg-generic.mk:195: recipe for target '/home/thomas/projets/buildroot/output/build/freeswitch-1.6.7/.stamp_built' failed
make[1]: *** [/home/thomas/projets/buildroot/output/build/freeswitch-1.6.7/.stamp_built] Error 2
Makefile:36: recipe for target '_all' failed
make: *** [_all] Error 2

Note that I have only PATCH 01/31 applied. If an ARM/uClibc
configuration doesn't build, a package is definitely not ready to go. I
would be OK to merge a package that doesn't build with musl, or on some
weird architectures, but certainly not something that doesn't build on
ARM/uClibc.

Could you look into this?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2016-04-13 21:02 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-10  9:41 [Buildroot] [PATCH v12 00/31] package/freeswitch: new package Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 01/31] " Bernd Kuhls
2016-04-13 21:02   ` Thomas Petazzoni [this message]
2016-04-14 19:04     ` Bernd Kuhls
2016-04-13 21:53   ` Arnout Vandecappelle
2016-04-10  9:41 ` [Buildroot] [PATCH v12 02/31] package/freeswitch: enable optional modules Bernd Kuhls
2016-04-13 21:56   ` Arnout Vandecappelle
2016-04-10  9:41 ` [Buildroot] [PATCH v12 03/31] package/freeswitch: add optional dependency to libuuid Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 04/31] package/freeswitch: add optional dependency to libedit Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 05/31] package/freeswitch: enable mod_enum Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 06/31] package/freeswitch: add optional dependency to libpng Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 07/31] package/freeswitch: add optional dependency to unixodbc Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 08/31] package/freeswitch: add optional dependency to freetype Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 09/31] package/freeswitch: enable mod_yaml Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 10/31] package/freeswitch: enable mod_lua Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 11/31] package/freeswitch: enable mod_bv Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 12/31] package/freeswitch: enable mod_codec2 Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 13/31] package/freeswitch: add optional dependency to libg7221 Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 14/31] package/freeswitch: enable mod_ilbc Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 15/31] package/freeswitch: enable mod_isac Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 16/31] package/freeswitch: enable mod_opus Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 17/31] package/freeswitch: enable mod_portaudio Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 18/31] package/freeswitch: enable mod_silk Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 19/31] package/freeswitch: enable mod_sndfile Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 20/31] package/freeswitch: enable mod_alsa Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 21/31] package/freeswitch: enable mod_soundtouch Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 22/31] package/freeswitch: enable mod_xml_rpc Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 23/31] package/freeswitch: add optional dependency to xz Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 24/31] package/freeswitch: enable mod_memcache Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 25/31] package/freeswitch: enable mod_imagick Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 26/31] package/freeswitch: enable mod_cv Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 27/31] package/freeswitch: enable mod_shout Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 28/31] package/freeswitch: add optional dependency to openldap Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 29/31] package/flite: bump version to 2.0.0 Bernd Kuhls
2016-04-13 22:16   ` Arnout Vandecappelle
2016-04-10  9:41 ` [Buildroot] [PATCH v12 30/31] package/freeswitch: enable mod_flite Bernd Kuhls
2016-04-10  9:41 ` [Buildroot] [PATCH v12 31/31] package/freeswitch: enable video support Bernd Kuhls

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160413230219.73e299ea@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox