[OE-core] [PATCH] libarchive: add missing e2fsprogs dependency

Khem Raj raj.khem at gmail.com
Tue May 6 18:25:31 UTC 2014


On Tue, May 6, 2014 at 3:33 AM, Paul Barker <paul at paulbarker.me.uk> wrote:
> On Tue, May 06, 2014 at 10:18:41AM +0100, Paul Eggleton wrote:
>> On Tuesday 06 May 2014 10:17:03 Paul Eggleton wrote:
>> > On Tuesday 06 May 2014 07:52:23 Koen Kooi wrote:
>> > > Op 5 mei 2014, om 23:54 heeft Saul Wold <sgw at linux.intel.com> het volgende
>> >
>> > geschreven:
>> > > > On 05/05/2014 04:26 AM, Koen Kooi wrote:
>> > > >> This fixes the following build error:
>> > > >>        | libarchive/archive_entry.c:59:55: fatal error: ext2fs/ext2_fs.h: No
>> > > >>        | such file or directory>>
>> > > >>
>> > > >> There is no configure options to avoid this dependency.
>> > > >>
>> > > >> Signed-off-by: Koen Kooi <koen.kooi at linaro.org>
>> > > >> ---
>> > > >>
>> > > >>  meta/recipes-extended/libarchive/libarchive_3.1.2.bb | 2 ++
>> > > >>  1 file changed, 2 insertions(+)
>> > > >>
>> > > >> diff --git a/meta/recipes-extended/libarchive/libarchive_3.1.2.bb
>> > > >> b/meta/recipes-extended/libarchive/libarchive_3.1.2.bb index
>> > > >> eedbaa7..277d8c8 100644
>> > > >> --- a/meta/recipes-extended/libarchive/libarchive_3.1.2.bb
>> > > >> +++ b/meta/recipes-extended/libarchive/libarchive_3.1.2.bb
>> > > >> @@ -5,6 +5,8 @@ SECTION = "devel"
>> > > >>
>> > > >>  LICENSE = "BSD"
>> > > >>  LIC_FILES_CHKSUM =
>> > > >>  "file://COPYING;md5=b4e3ffd607d6686c6cb2f63394370841"
>> > > >>
>> > > >> +DEPENDS = "e2fsprogs"
>> > > >> +
>> > > >
>> > > > By doing this we also have to extend e2fprogs to be a nativesdk target.
>> > > >
>> > > > Is there not a way to make this optional by editing the configure.ac and
>> > > > adding a switch?  What functionality to we gain or loose with / without
>> > > > the ext2fs extensions?
>> > >
>> > > I really have no idea, I just had the above buildfailure yesterday.
>> >
>> > FYI this is a bug I was meant to be fixing in the 1.6 cycle (I noticed the
>> > error in one of my builds a few months ago). Simply adding e2fsprogs to
>> > DEPENDS adds time to the build, plus when I submitted a patch to do this I
>> > found that nativesdk-e2fsprogs doesn't actually build at the moment and
>> > since it's actually only the header it wants, I had in mind to just use the
>> > header from e2fsprogs-native. I haven't got around to testing the latter
>> > solution though.
>> >
>> > Anyway, I've now done what I should have done earlier and filed a bug:
>>
>> Oops, sent too early - bug link:
>>
>> https://bugzilla.yoctoproject.org/show_bug.cgi?id=6268
>>
>> Cheers,
>> Paul
>>
>
> I just had a quick look at this as I've done a bit of work with libarchive. It
> looks like as long as the header isn't found at configure time there's no
> problem at compile time. Could we work around this by modifying config.h in a
> do_configure_append() function to undefine the relevant symbol
> (HAVE_EXT2FS_EXT2_FS_H)?

It can be disabled in many ways and easiest is to cache the autconf
variable and set it to 'no'
but we loose The EXT2 ioctls that are used on Linux to get/set file
flags since our build systems
are mostly linux I would think this is a good thing to enable in libarchive.



More information about the Openembedded-core mailing list