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

[Paul Eggleton]

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

-- 

Paul Eggleton
Intel Open Source Technology Centre