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

[Paul Barker]

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)?

Thanks,

-- 
Paul Barker

Email: paul at paulbarker.me.uk
http://www.paulbarker.me.uk
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20140506/74646aa6/attachment-0002.sig>