All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: openembedded-devel@lists.openembedded.org,
	openembedded-core@lists.openembedded.org
Subject: Re: [oe] [meta-oe][PATCH] bash-completion: remove allarch
Date: Fri, 7 Mar 2014 19:28:58 +0100	[thread overview]
Message-ID: <20140307182858.GL26981@jama> (raw)
In-Reply-To: <3158053.US3HkBfSAc@peggleto-mobl5.ger.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 2432 bytes --]

On Fri, Mar 07, 2014 at 10:09:00AM +0000, Paul Eggleton wrote:
> On Thursday 06 March 2014 20:21:22 Martin Jansa wrote:
> > On Thu, Mar 06, 2014 at 05:59:29PM +0000, Paul Eggleton wrote:
> > > On Thursday 06 March 2014 18:04:50 Martin Jansa wrote:
> > > > * it has runtime dependency on TUNE_PKGARCH bash, so it cannot be
> > > > allarch
> > > 
> > > As we've already discussed this is not universally true. There are other
> > > ways to solve this.
> > 
> > Like making it special case like packagegroups? That is still making me
> > headaches when some library (for some reason) included in packagegroup
> > is renamed thanks to debian.bbclass and packagegroup isn't rebuilt, so
> > it breaks do_rootfs..
> 
> Did you report this? FWIW it's the first I recall hearing of the problem; 
> perhaps I just missed it.

I've discussed this with RP on IRC, but haven't filled the bugzilla
ticket, my fault, will do that later.

> > I would rather build bash-completion only once per architecture than
> > rebuilding it as "allarch" every time I'm building for MACHINE with
> > different TUNE_PKGARCH.
> 
> As I said last time, I'm not arguing that rebuilding allarch recipes on 
> machine change is preferable - obviously it isn't. On the other hand, what 
> you're doing with this kind of change is telling the build system that the 
> recipe needs to be built differently depending on what the target architecture 
> is, which is *not* true; this is only being done to work around the build 
> system making an assumption that the rebuild needs to happen. Fix the 
> assumption and the problem is fixed, properly. 
> 
> If we need to make changes to the core or BitBake to make it easier to handle 
> this properly, by all means let's do that; but if we go down the road of 
> applying the workaround to every recipe where we hit this issue (and as we 
> have seen it's not just one or two recipes) we will never get around to fixing 
> the underlying problem.

What I'm trying to do is to prevent rebuilding them until the
underlaying problem is fixed (which can be tracked in bugzilla with 2
very simple recipes as reproducer).

I don't want this to show in every build and as "known-possitive" in
every sstate-diff-machine.sh call. It's less pain to just rebuild them
once per TUNE_PKGARCH even when they could be allarch.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Martin Jansa <martin.jansa@gmail.com>
To: Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: openembedded-devel@lists.openembedded.org,
	openembedded-core@lists.openembedded.org
Subject: Re: [meta-oe][PATCH] bash-completion: remove allarch
Date: Fri, 7 Mar 2014 19:28:58 +0100	[thread overview]
Message-ID: <20140307182858.GL26981@jama> (raw)
In-Reply-To: <3158053.US3HkBfSAc@peggleto-mobl5.ger.corp.intel.com>

[-- Attachment #1: Type: text/plain, Size: 2432 bytes --]

On Fri, Mar 07, 2014 at 10:09:00AM +0000, Paul Eggleton wrote:
> On Thursday 06 March 2014 20:21:22 Martin Jansa wrote:
> > On Thu, Mar 06, 2014 at 05:59:29PM +0000, Paul Eggleton wrote:
> > > On Thursday 06 March 2014 18:04:50 Martin Jansa wrote:
> > > > * it has runtime dependency on TUNE_PKGARCH bash, so it cannot be
> > > > allarch
> > > 
> > > As we've already discussed this is not universally true. There are other
> > > ways to solve this.
> > 
> > Like making it special case like packagegroups? That is still making me
> > headaches when some library (for some reason) included in packagegroup
> > is renamed thanks to debian.bbclass and packagegroup isn't rebuilt, so
> > it breaks do_rootfs..
> 
> Did you report this? FWIW it's the first I recall hearing of the problem; 
> perhaps I just missed it.

I've discussed this with RP on IRC, but haven't filled the bugzilla
ticket, my fault, will do that later.

> > I would rather build bash-completion only once per architecture than
> > rebuilding it as "allarch" every time I'm building for MACHINE with
> > different TUNE_PKGARCH.
> 
> As I said last time, I'm not arguing that rebuilding allarch recipes on 
> machine change is preferable - obviously it isn't. On the other hand, what 
> you're doing with this kind of change is telling the build system that the 
> recipe needs to be built differently depending on what the target architecture 
> is, which is *not* true; this is only being done to work around the build 
> system making an assumption that the rebuild needs to happen. Fix the 
> assumption and the problem is fixed, properly. 
> 
> If we need to make changes to the core or BitBake to make it easier to handle 
> this properly, by all means let's do that; but if we go down the road of 
> applying the workaround to every recipe where we hit this issue (and as we 
> have seen it's not just one or two recipes) we will never get around to fixing 
> the underlying problem.

What I'm trying to do is to prevent rebuilding them until the
underlaying problem is fixed (which can be tracked in bugzilla with 2
very simple recipes as reproducer).

I don't want this to show in every build and as "known-possitive" in
every sstate-diff-machine.sh call. It's less pain to just rebuild them
once per TUNE_PKGARCH even when they could be allarch.

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]

  reply	other threads:[~2014-03-07 18:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-06 17:04 [meta-oe][PATCH] bash-completion: remove allarch Martin Jansa
2014-03-06 17:59 ` Paul Eggleton
2014-03-06 19:21   ` Martin Jansa
2014-03-07 10:09     ` [oe] " Paul Eggleton
2014-03-07 10:09       ` Paul Eggleton
2014-03-07 18:28       ` Martin Jansa [this message]
2014-03-07 18:28         ` Martin Jansa
2014-03-15 16:01         ` [oe] " Martin Jansa
2014-03-15 16:01           ` Martin Jansa
2014-03-07  8:41 ` Matthieu CRAPET
2014-03-11 14:31   ` 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=20140307182858.GL26981@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=openembedded-devel@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 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.