Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Martin Jansa <martin.jansa@gmail.com>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>,
	OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] usbinit, weston-init, run-postinsts, qt-demo-init: Drop allarch
Date: Thu, 03 Jul 2014 17:45:34 +0100	[thread overview]
Message-ID: <1404405934.6362.89.camel@ted> (raw)
In-Reply-To: <20140623124437.GG2437@jama>

On Mon, 2014-06-23 at 14:44 +0200, Martin Jansa wrote:
> On Mon, Jun 23, 2014 at 02:39:22PM +0200, Martin Jansa wrote:
> > On Mon, Jun 23, 2014 at 01:12:25PM +0100, Burton, Ross wrote:
> > > On 19 June 2014 14:54, Paul Eggleton <paul.eggleton@linux.intel.com> wrote:
> > > >> * update-rc.d.bbclass now adds dependency on TUNE_PKGARCH initscripts, so
> > > >> these cannot be allarch
> > > >
> > > > As I've said before, I really dislike this change. The system should be made
> > > > to handle the situation properly instead of working around the issue.
> > > 
> > > Isn't this what the abi exclude thingy is for?
> > 
> > Do we want to maintain list of all recipes which inherit update-rc.c in
> > layer.conf?
> > 
> > That's what I did in one of our layers and the list is rather long.
> 
> Ah sorry for possible confusion, I had to list *all* of them because we
> have MACHINE_ARCH initscripts.
> 
> To fix current issues in oe-core we only need to list all recipes which
> inherit allarch *and* update-rc.d (unless we want to exclude it
> everywhere just for consistency).
> 
> > abi exclude thingy
> 
> is a bit dangerous, I still sometimes see issues when something is
> supposed to be abisafe and in some cases it isn't and I need to manually
> bump PR in the recipe which wasn't rebuilt.
> 
> So I'm trying to use SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS only in cases where
> it's really safe or when the dependent recipe takes forever to build (so
> I wan't to build it just once or once per TUNE_PKGARCH). For simple and
> quickly built recipe I really don't mind "building" them multiple times
> if it prevents "rebuilding" them every single time I change MACHINE.

Can't we just add "initscripts" to the list there. There shouldn't be a
need to rebuild initscripts any time something depending on it changes,
it should have a safe ABI in that sense?

Cheers,

Richard





  reply	other threads:[~2014-07-03 16:46 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-19 11:23 [PATCH] usbinit, weston-init, run-postinsts, qt-demo-init: Drop allarch Martin Jansa
2014-06-19 13:54 ` Paul Eggleton
2014-06-19 16:39   ` Martin Jansa
2014-06-19 17:03     ` Paul Eggleton
2014-06-23 12:12   ` Burton, Ross
2014-06-23 12:39     ` Martin Jansa
2014-06-23 12:44       ` Martin Jansa
2014-07-03 16:45         ` Richard Purdie [this message]
2014-07-03 17:19           ` Avoiding getting the login prompt on the LCD at boot Ulf Samuelsson
2014-07-03 20:31             ` Burton, Ross
2014-07-03 20:36               ` Ulf Samuelsson
2014-08-11 14:09   ` [PATCH] SIGGEN_EXCLUDERECIPES_ABISAFE: add initscripts Martin Jansa

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=1404405934.6362.89.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox