All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.