Openembedded Core Discussions
 help / color / mirror / Atom feed
* PRIORITY
@ 2011-07-01  9:41 Phil Blundell
  2011-07-01 10:09 ` PRIORITY Koen Kooi
  2011-07-01 10:15 ` PRIORITY Richard Purdie
  0 siblings, 2 replies; 6+ messages in thread
From: Phil Blundell @ 2011-07-01  9:41 UTC (permalink / raw)
  To: openembedded-core

We had some discussion of this a few weeks ago but I'm not sure it ever
reached a firm conclusion.

Is everyone agreed that the PRIORITY variable does not serve any useful
purpose in oe-core and can/should be removed from the recipes?

p.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: PRIORITY
  2011-07-01  9:41 PRIORITY Phil Blundell
@ 2011-07-01 10:09 ` Koen Kooi
  2011-07-01 10:15 ` PRIORITY Richard Purdie
  1 sibling, 0 replies; 6+ messages in thread
From: Koen Kooi @ 2011-07-01 10:09 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer


Op 1 jul 2011, om 11:41 heeft Phil Blundell het volgende geschreven:

> We had some discussion of this a few weeks ago but I'm not sure it ever
> reached a firm conclusion.
> 
> Is everyone agreed that the PRIORITY variable does not serve any useful
> purpose in oe-core and can/should be removed from the recipes?

I have no use for that field, so removing it would be fine by me.

regards,

Koen



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: PRIORITY
  2011-07-01  9:41 PRIORITY Phil Blundell
  2011-07-01 10:09 ` PRIORITY Koen Kooi
@ 2011-07-01 10:15 ` Richard Purdie
  2011-07-01 10:24   ` PRIORITY Phil Blundell
  1 sibling, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2011-07-01 10:15 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Fri, 2011-07-01 at 10:41 +0100, Phil Blundell wrote:
> We had some discussion of this a few weeks ago but I'm not sure it ever
> reached a firm conclusion.
> 
> Is everyone agreed that the PRIORITY variable does not serve any useful
> purpose in oe-core and can/should be removed from the recipes?

I think that was the conclusion, yes. I'll take a patch (or a
commandline I guess).

Cheers,

Richard




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: PRIORITY
  2011-07-01 10:15 ` PRIORITY Richard Purdie
@ 2011-07-01 10:24   ` Phil Blundell
  2011-07-01 12:17     ` PRIORITY Frans Meulenbroeks
  2011-07-01 22:28     ` PRIORITY Richard Purdie
  0 siblings, 2 replies; 6+ messages in thread
From: Phil Blundell @ 2011-07-01 10:24 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Fri, 2011-07-01 at 11:15 +0100, Richard Purdie wrote:
> On Fri, 2011-07-01 at 10:41 +0100, Phil Blundell wrote:
> > We had some discussion of this a few weeks ago but I'm not sure it ever
> > reached a firm conclusion.
> > 
> > Is everyone agreed that the PRIORITY variable does not serve any useful
> > purpose in oe-core and can/should be removed from the recipes?
> 
> I think that was the conclusion, yes. I'll take a patch (or a
> commandline I guess).

I did:

$ find . -regex ".*\.\(bb\|inc\)$" | xargs sed -i '/^PRIORITY = ".*"$/d'

which seemed to produce the desired results.  I can send a patch if you
like but it would be somewhat large and fairly uninteresting.

p.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: PRIORITY
  2011-07-01 10:24   ` PRIORITY Phil Blundell
@ 2011-07-01 12:17     ` Frans Meulenbroeks
  2011-07-01 22:28     ` PRIORITY Richard Purdie
  1 sibling, 0 replies; 6+ messages in thread
From: Frans Meulenbroeks @ 2011-07-01 12:17 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

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

2011/7/1 Phil Blundell <pb@pbcl.net>

> On Fri, 2011-07-01 at 11:15 +0100, Richard Purdie wrote:
> > On Fri, 2011-07-01 at 10:41 +0100, Phil Blundell wrote:
> > > We had some discussion of this a few weeks ago but I'm not sure it ever
> > > reached a firm conclusion.
> > >
> > > Is everyone agreed that the PRIORITY variable does not serve any useful
> > > purpose in oe-core and can/should be removed from the recipes?
> >
> > I think that was the conclusion, yes. I'll take a patch (or a
> > commandline I guess).
>
> I did:
>
> $ find . -regex ".*\.\(bb\|inc\)$" | xargs sed -i '/^PRIORITY = ".*"$/d'
>
> which seemed to produce the desired results.  I can send a patch if you
> like but it would be somewhat large and fairly uninteresting.
>
> I'd suggest a regex that matches for any whitespace in order also to catch
things like:
./libgpevtype/libgpevtype_0.50.bb:PRIORITY    = "optional"

Have fun! Frans

[-- Attachment #2: Type: text/html, Size: 1389 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: PRIORITY
  2011-07-01 10:24   ` PRIORITY Phil Blundell
  2011-07-01 12:17     ` PRIORITY Frans Meulenbroeks
@ 2011-07-01 22:28     ` Richard Purdie
  1 sibling, 0 replies; 6+ messages in thread
From: Richard Purdie @ 2011-07-01 22:28 UTC (permalink / raw)
  To: Patches and discussions about the oe-core layer

On Fri, 2011-07-01 at 11:24 +0100, Phil Blundell wrote:
> On Fri, 2011-07-01 at 11:15 +0100, Richard Purdie wrote:
> > On Fri, 2011-07-01 at 10:41 +0100, Phil Blundell wrote:
> > > We had some discussion of this a few weeks ago but I'm not sure it ever
> > > reached a firm conclusion.
> > > 
> > > Is everyone agreed that the PRIORITY variable does not serve any useful
> > > purpose in oe-core and can/should be removed from the recipes?
> > 
> > I think that was the conclusion, yes. I'll take a patch (or a
> > commandline I guess).
> 
> I did:
> 
> $ find . -regex ".*\.\(bb\|inc\)$" | xargs sed -i '/^PRIORITY = ".*"$/d'
> 
> which seemed to produce the desired results.  I can send a patch if you
> like but it would be somewhat large and fairly uninteresting.

I've gone ahead and pushed such a commit, thanks!

Richard




^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-07-01 22:32 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-01  9:41 PRIORITY Phil Blundell
2011-07-01 10:09 ` PRIORITY Koen Kooi
2011-07-01 10:15 ` PRIORITY Richard Purdie
2011-07-01 10:24   ` PRIORITY Phil Blundell
2011-07-01 12:17     ` PRIORITY Frans Meulenbroeks
2011-07-01 22:28     ` PRIORITY Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox