From: Paul Barker <paul@paulbarker.me.uk>
To: Khem Raj <raj.khem@gmail.com>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>,
Koen Kooi <koen@dominion.thruhere.net>,
Koen Kooi <koen.kooi@linaro.org>,
Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] libarchive: add missing e2fsprogs dependency
Date: Tue, 6 May 2014 22:57:39 +0000 [thread overview]
Message-ID: <20140506225739.GA25360@gmail.com> (raw)
In-Reply-To: <CAMKF1srXCeedrKMS78WvgXO9PAMJe1sWXAywPSsgp1Tya7q7pA@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]
On Tue, May 06, 2014 at 11:25:31AM -0700, Khem Raj wrote:
> On Tue, May 6, 2014 at 3:33 AM, Paul Barker <paul@paulbarker.me.uk> wrote:
> > On Tue, May 06, 2014 at 10:18:41AM +0100, Paul Eggleton wrote:
> >>
> >> 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.
Yea that makes a lot more sense. I was in effect replying to the bug report
saying "There is no way to hard-disable this dependency.".
Sounds like it's worth using that header if possible though so ignore my
comment.
Cheers,
--
Paul Barker
Email: paul@paulbarker.me.uk
http://www.paulbarker.me.uk
[-- Attachment #2: Type: application/pgp-signature, Size: 501 bytes --]
prev parent reply other threads:[~2014-05-06 22:57 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-05 11:26 [PATCH] libarchive: add missing e2fsprogs dependency Koen Kooi
2014-05-05 21:54 ` Saul Wold
2014-05-06 5:52 ` Koen Kooi
2014-05-06 9:17 ` Paul Eggleton
2014-05-06 9:18 ` Paul Eggleton
2014-05-06 10:33 ` Paul Barker
2014-05-06 10:39 ` Paul Eggleton
2014-05-06 18:25 ` Khem Raj
2014-05-06 22:57 ` Paul Barker [this message]
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=20140506225739.GA25360@gmail.com \
--to=paul@paulbarker.me.uk \
--cc=koen.kooi@linaro.org \
--cc=koen@dominion.thruhere.net \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul.eggleton@linux.intel.com \
--cc=raj.khem@gmail.com \
/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 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.