* [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile
@ 2015-07-29 21:04 Bernd Kuhls
2015-07-30 19:58 ` Jörg Krause
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Bernd Kuhls @ 2015-07-29 21:04 UTC (permalink / raw)
To: buildroot
libz is needed by libid3tag, fixes
http://autobuild.buildroot.net/results/85e/85e53f2b30f107f486be08549ce6df08953ea499/
libm is needed by libexif, changes
checking for exif_data_new_from_file in -lexif... no
checking for exif_data_new_from_file in -lexif... no
from the forementioned autobuilder config.log to
checking for exif_data_new_from_file in -lexif... yes
The pkgconfig file of libvorbis is called vorbis.pc, fix wrong naming. Also
we need to link against libsqlite3, so add that lib to the pkgconf call.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/minidlna/minidlna.mk | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/package/minidlna/minidlna.mk b/package/minidlna/minidlna.mk
index ab93484..2f75993 100644
--- a/package/minidlna/minidlna.mk
+++ b/package/minidlna/minidlna.mk
@@ -17,9 +17,9 @@ MINIDLNA_DEPENDENCIES = \
ifeq ($(BR2_STATIC_LIBS),y)
# the configure script / Makefile forgets to link with some of the dependent
# libraries breaking static linking, so help it along
-MINIDLNA_STATIC_LIBS += `$(PKG_CONFIG_HOST_BINARY) --libs libavcodec libexif libvorbis`
+MINIDLNA_STATIC_LIBS += `$(PKG_CONFIG_HOST_BINARY) --libs libavcodec libexif vorbis sqlite3`
MINIDLNA_STATIC_LIBS += $(if $(BR2_NEEDS_GETTEXT_IF_LOCALE),-lintl)
-MINIDLNA_CONF_ENV += LIBS="$(MINIDLNA_STATIC_LIBS)"
+MINIDLNA_CONF_ENV += LIBS="$(MINIDLNA_STATIC_LIBS) -lm -lz"
else
MINIDLNA_CONF_OPTS = \
--disable-static
--
2.1.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile
2015-07-29 21:04 [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile Bernd Kuhls
@ 2015-07-30 19:58 ` Jörg Krause
2015-07-30 21:33 ` Thomas Petazzoni
2015-08-29 21:26 ` Thomas Petazzoni
2 siblings, 0 replies; 4+ messages in thread
From: Jörg Krause @ 2015-07-30 19:58 UTC (permalink / raw)
To: buildroot
On Mi, 2015-07-29 at 23:04 +0200, Bernd Kuhls wrote:
> libz is needed by libid3tag, fixes
> http://autobuild.buildroot.net/results/85e/85e53f2b30f107f486be08549c
> e6df08953ea499/
>
> libm is needed by libexif, changes
>
> checking for exif_data_new_from_file in -lexif... no
> checking for exif_data_new_from_file in -lexif... no
>
> from the forementioned autobuilder config.log to
>
> checking for exif_data_new_from_file in -lexif... yes
>
> The pkgconfig file of libvorbis is called vorbis.pc, fix wrong
> naming. Also
> we need to link against libsqlite3, so add that lib to the pkgconf
> call.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
minidlna successfully compiles in a static build after applying the
patch. Tested with the config file from the autobuilder.
Tested-by: J?rg Krause <joerg.krause@embedded.rocks>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile
2015-07-29 21:04 [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile Bernd Kuhls
2015-07-30 19:58 ` Jörg Krause
@ 2015-07-30 21:33 ` Thomas Petazzoni
2015-08-29 21:26 ` Thomas Petazzoni
2 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2015-07-30 21:33 UTC (permalink / raw)
To: buildroot
Bernd,
On Wed, 29 Jul 2015 23:04:31 +0200, Bernd Kuhls wrote:
> diff --git a/package/minidlna/minidlna.mk b/package/minidlna/minidlna.mk
> index ab93484..2f75993 100644
> --- a/package/minidlna/minidlna.mk
> +++ b/package/minidlna/minidlna.mk
> @@ -17,9 +17,9 @@ MINIDLNA_DEPENDENCIES = \
> ifeq ($(BR2_STATIC_LIBS),y)
> # the configure script / Makefile forgets to link with some of the dependent
> # libraries breaking static linking, so help it along
> -MINIDLNA_STATIC_LIBS += `$(PKG_CONFIG_HOST_BINARY) --libs libavcodec libexif libvorbis`
> +MINIDLNA_STATIC_LIBS += `$(PKG_CONFIG_HOST_BINARY) --libs libavcodec libexif vorbis sqlite3`
> MINIDLNA_STATIC_LIBS += $(if $(BR2_NEEDS_GETTEXT_IF_LOCALE),-lintl)
> -MINIDLNA_CONF_ENV += LIBS="$(MINIDLNA_STATIC_LIBS)"
> +MINIDLNA_CONF_ENV += LIBS="$(MINIDLNA_STATIC_LIBS) -lm -lz"
Would it be possible to stop hacking this to death? What about instead
taking the saner approach of switching minidlna configure.ac to use
PKG_CHECK_MODULES(), which will remove a lot of crap from it, and make
it work properly for cross-compilation.
Some libraries, such as libid3tag, would also need to install a .pc
file, obviously.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile
2015-07-29 21:04 [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile Bernd Kuhls
2015-07-30 19:58 ` Jörg Krause
2015-07-30 21:33 ` Thomas Petazzoni
@ 2015-08-29 21:26 ` Thomas Petazzoni
2 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2015-08-29 21:26 UTC (permalink / raw)
To: buildroot
Dear Bernd Kuhls,
On Wed, 29 Jul 2015 23:04:31 +0200, Bernd Kuhls wrote:
> libz is needed by libid3tag, fixes
> http://autobuild.buildroot.net/results/85e/85e53f2b30f107f486be08549ce6df08953ea499/
>
> libm is needed by libexif, changes
>
> checking for exif_data_new_from_file in -lexif... no
> checking for exif_data_new_from_file in -lexif... no
>
> from the forementioned autobuilder config.log to
>
> checking for exif_data_new_from_file in -lexif... yes
>
> The pkgconfig file of libvorbis is called vorbis.pc, fix wrong naming. Also
> we need to link against libsqlite3, so add that lib to the pkgconf call.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/minidlna/minidlna.mk | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
It's now too late for 2015.08. Therefore, could you engage the process
of moving minidlna (and its dependencies to use pkg-config), in order
to fix this properly? minidlna Git repository seems to be active, so
there is a chance of getting such changes merged.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-08-29 21:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-29 21:04 [Buildroot] [PATCH 1/1] package/minidlna: More fixes to static compile Bernd Kuhls
2015-07-30 19:58 ` Jörg Krause
2015-07-30 21:33 ` Thomas Petazzoni
2015-08-29 21:26 ` Thomas Petazzoni
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.