All of lore.kernel.org
 help / color / mirror / Atom feed
From: Koen Kooi <koen@dominion.thruhere.net>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Cc: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/5] kernel.bbclass: respect MACHINE_KERNEL_PR
Date: Wed, 28 Sep 2011 13:42:09 -0500	[thread overview]
Message-ID: <89CD0BCB-1457-4C76-87BA-5ED03454C343@dominion.thruhere.net> (raw)
In-Reply-To: <1317221697.12332.13.camel@ted>



Op 28 sep. 2011 om 09:54 heeft Richard Purdie <richard.purdie@linuxfoundation.org> het volgende geschreven:

> On Tue, 2011-09-27 at 18:40 -0500, Koen Kooi wrote:
>> 
>> Op 27 sep. 2011 om 08:52 heeft Bruce Ashfield <bruce.ashfield@gmail.com> het volgende geschreven:
>> 
>>> On Sat, Sep 17, 2011 at 6:18 PM, Dmitry Eremin-Solenikov
>>> <dbaryshkov@gmail.com> wrote:
>>>> MACHINE_KERNEL_PR was introduced long ago in org.oe.dev. It's present in
>>>> meta-oe kernel.bbclass. Several machines depend on this functionality.
>>> 
>>> I don' t have a big problem with this, since the change is obviously
>>> harmless if it
>>> doesn't need to be used. But similar to my comment on patch 3/5, is there any
>>> history/technical reasons we can capture here ?
>>> 
>>> I did a quick search to dig up a bit on this myself, but a summary
>>> would definitely
>>> help, since I see that this has a long and sometimes twisting history. Is it as
>>> simple as ?
>>> 
>>> "A machine.conf or local.conf can increase MACHINE_KERNEL_PR to force
>>> rebuilds for kernel and external modules"
>> 
>> It is that simple :) 
> 
> I have reservations about this patch given we soon plan to stop needing
> to bump PR values. This patch is actually a perfect example of how brain
> dead the current manual PR bumping is and why we need something better.
> I don't want half the code in the repo to be a mass of PR values which
> need to be incremented manually under weird and wonderful circumstances
> which is where the direction things are currently going...
> 
> I'd like to hold off on this one.

This patch improves the current situation and I don't foresee the autoPR code working soon


> 
> Cheers,
> 
> Richard
> 
> 
> 
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core



  reply	other threads:[~2011-09-28 18:47 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-17 22:18 [PATCH 1/5] kernel.bbclass: blacklist 'perf-dbg' as well for the modules metapackage Dmitry Eremin-Solenikov
2011-09-17 22:18 ` [PATCH 2/5] kernel.bbclass: respect MACHINE_KERNEL_PR Dmitry Eremin-Solenikov
2011-09-17 22:24   ` Otavio Salvador
2011-09-27 13:52   ` Bruce Ashfield
2011-09-27 23:40     ` Koen Kooi
2011-09-28 14:54       ` Richard Purdie
2011-09-28 18:42         ` Koen Kooi [this message]
2011-09-28 18:50           ` Dmitry Eremin-Solenikov
2011-09-28 19:50           ` Richard Purdie
2011-09-28 20:04             ` Otavio Salvador
2011-10-20  6:23               ` Koen Kooi
2011-10-20 11:21                 ` Richard Purdie
2011-10-20 11:29                   ` Koen Kooi
2011-10-20 11:35                     ` Otavio Salvador
2011-10-20 11:53                     ` Frans Meulenbroeks
2011-10-20 12:38                     ` Richard Purdie
2011-10-20 12:54                       ` Koen Kooi
2011-10-20 13:02                         ` Otavio Salvador
2011-10-21 12:05                           ` Koen Kooi
2011-10-20 13:56                         ` Richard Purdie
2011-09-17 22:18 ` [PATCH 3/5] kernel.bbclass: move uImage handling to separate task Dmitry Eremin-Solenikov
2011-09-17 22:25   ` Otavio Salvador
2011-09-21 13:09     ` Dmitry Eremin-Solenikov
2011-09-27 13:39   ` Bruce Ashfield
2011-09-17 22:18 ` [PATCH 4/5] kernel.bbclass: handle .cis firmware Dmitry Eremin-Solenikov
2011-09-27 13:40   ` Bruce Ashfield
2011-09-28 14:50   ` Richard Purdie
2011-09-17 22:18 ` [PATCH 5/5] kernel.bbclass: remove unshipped files in do_install Dmitry Eremin-Solenikov
2011-09-27 13:41   ` Bruce Ashfield
2011-09-28 14:50   ` Richard Purdie
2011-09-17 22:23 ` [PATCH 1/5] kernel.bbclass: blacklist 'perf-dbg' as well for the modules metapackage Otavio Salvador
2011-09-22 12:25   ` Dmitry Eremin-Solenikov
2011-09-22 12:35     ` Koen Kooi
2011-09-22 13:00       ` Dmitry Eremin-Solenikov
2011-09-22 13:17         ` Koen Kooi
2011-09-22 13:28           ` Bruce Ashfield
2011-09-22 13:56             ` Koen Kooi
2011-09-22 14:04               ` Bruce Ashfield
2011-09-28  8:47 ` Dmitry Eremin-Solenikov
2011-09-28 14:52   ` Richard Purdie
2011-09-28 14:50 ` Richard Purdie

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=89CD0BCB-1457-4C76-87BA-5ED03454C343@dominion.thruhere.net \
    --to=koen@dominion.thruhere.net \
    --cc=dbaryshkov@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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.