From: Steve Kenton <skenton@ou.edu>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] dvdauthor : new package
Date: Mon, 12 Jan 2015 09:33:45 -0600 [thread overview]
Message-ID: <54B3E959.2010504@ou.edu> (raw)
In-Reply-To: <20150112092800.08ce4350@free-electrons.com>
On 01/12/2015 02:28 AM, Thomas Petazzoni wrote:
>> /data/bcdist/test/buildroot-2014.11/output/host/usr/include/libxml2/libxml/encoding.h:28:19: fatal error: iconv.h: No such file or directory
>> #include <iconv.h>
>
> This is a *host* header, so it shouldn't be included when building a
> target package.
>
>> ^
>> compilation terminated.
>> make[3]: *** [readxml.o] Error 1
The build triggers lots of automatic reconfiguration, I guess it must be something on
the host side getting rebuilt then. I'll double check while I'm updating the patch.
>> + depends on BR2_LARGEFILE && !BR2_PREFER_STATIC_LIB # libdvdread
>
> BR2_PREFER_STATIC_LIB has been renamed to BR2_STATIC_LIBS. Please make
> sure to test your patches against the latest master, since this is
> where they will be applied.
OK
>> +# Note - Despite lack of a version bump the two line gentoo
>> +# patch to src/spuunmux.c against "stock" dvdauthor-0.7.1
>> +# - if (setjmp(png_ptr->jmpbuf))
>> +# + if (setjmp(png_jmpbuf(png_ptr)))
>> +# to allow use of libpng15/16 is already applied to downloadable
>> +# dvdauthor-0.7.1.tar.gz
>
> This should go to the commit log.
Moved
> Also, does this means that your libpng12 patch is now unnecessary?
Yes, Gustavo Zacarias pointed that patch out to me and it indeed makes libpng12 unnecessary
>> +DVDAUTHOR_DEPENDENCIES = host-pkgconf host-libxml2 host-freetype libxml2 freetype libpng libdvdread
>
> I'm a little bit surprised by the need for host-libxml2 and
> host-freetype, but I admittedly haven't looked in detail.
I added them because of host side configure problems:
checking for libxml - version >= 2.6.0... no
*** The xml2-config script installed by LIBXML could not be found
configure:5176: checking for freetype-config
configure:5192: found /usr/bin/freetype-config
This last one sent me on the wild goose chase for pnglib12 because
it let the Ubuntu environment leak into the buildroot configuration
next prev parent reply other threads:[~2015-01-12 15:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-12 0:39 [Buildroot] [PATCH] dvdauthor : new package Steve Kenton
2015-01-12 8:28 ` Thomas Petazzoni
2015-01-12 15:33 ` Steve Kenton [this message]
2015-01-12 15:37 ` Thomas Petazzoni
2015-01-12 16:14 ` Steve Kenton
2015-01-12 17:26 ` Steve Kenton
2015-01-12 18:32 ` Thomas Petazzoni
2015-01-12 18:53 ` Steve Kenton
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=54B3E959.2010504@ou.edu \
--to=skenton@ou.edu \
--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