All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: "Burton, Ross" <ross.burton@intel.com>
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 1/3] bitbake: add file-native to ASSUME_PROVIDED
Date: Mon, 19 Oct 2015 17:24:21 +0100	[thread overview]
Message-ID: <1445271861.17974.123.camel@linuxfoundation.org> (raw)
In-Reply-To: <CAJTo0LZhMs6Pr85oq2H_Tut12yx_f1JEbM+nsWpVOf1UC98ABg@mail.gmail.com>

On Mon, 2015-10-19 at 17:11 +0100, Burton, Ross wrote:
> 
> On 19 October 2015 at 16:37, Burton, Ross <ross.burton@intel.com>
> wrote:
>         Not sure what to do here.  If we install file-native directly
>         into the sysroot we get occasional races on the file binary
>          causing build failures.  If we make file-native
>         ASSUME_PROVIDED then rpm (and subversion) change their
>         behaviour depending on the presence of libmagic-dev on the
>         host (well, rpm4 requires libmagic, rpm5 and subversion change
>         their configuration).
>         
> 
> Two proposals:
> 
> 
> 1) Tell RPM and Subversion to explicitly enable libmagic support so
> they fail if libmagic-dev isn't installed on the host, and document
> that libmagic-dev is a build requirement
> 2) Change file-replacement-native to install libmagic, and add
> file-replacement-native as a dependency to subversion-native and
> rpm-native.
>
> Does anyone have a strong opinion either way?

I tend to become nervous about -dev requirements so whilst ugly, I'm
leaning towards 2) which is what we've done for other similar issues
IIRC.

Cheers,

Richard




  reply	other threads:[~2015-10-19 16:24 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-19 10:05 [PATCH 1/3] bitbake: add file-native to ASSUME_PROVIDED Ross Burton
2015-10-19 10:05 ` [PATCH 2/3] sanity: check that the host has file installed Ross Burton
2015-10-19 12:09   ` Martin Jansa
2015-10-19 12:12     ` Burton, Ross
2015-10-19 13:34       ` Paul Eggleton
2015-10-19 13:40         ` Burton, Ross
2015-10-19 10:05 ` [PATCH 3/3] file: don't replace host file when built natively Ross Burton
2015-10-19 15:37 ` [PATCH 1/3] bitbake: add file-native to ASSUME_PROVIDED Burton, Ross
2015-10-19 16:11   ` Burton, Ross
2015-10-19 16:24     ` Richard Purdie [this message]
2015-10-19 16:52     ` Alex Franco

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=1445271861.17974.123.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=ross.burton@intel.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.