All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Hatle <mark.hatle@windriver.com>
To: <openembedded-core@lists.openembedded.org>
Subject: Re: [RFC] policy proposal: INC_PR
Date: Tue, 20 Sep 2011 14:10:00 -0500	[thread overview]
Message-ID: <4E78E508.9080706@windriver.com> (raw)
In-Reply-To: <CALT56yOyp2VYEziGiZj4PLjYHFhTkxisKwWnP7_Uy69Cc1ufYw@mail.gmail.com>

On 9/20/11 2:04 PM, Dmitry Eremin-Solenikov wrote:
> Hello, colleagues,
> 
> While debugging some stuff in oe-core & company I've noticed that
> lot's of packages
> either don't use INC_PR, or misuse it (e.g. .inc has INC_PR, but then
> .bb just defines PR = "rX").

I've noticed similar things.  I'd agree, we should define and use INC_PR for
items that have .inc files.  There have been many times that I need to fix a bug
in the .inc file and end up manually updating the PR is 2 or 3 recipes that use
the .inc.

One question though, how do we handle packages with multilib .inc files?

INC_PR += ...  (or is it .=)

--Mark

> From my previous experience with oe-dev, I found INC_PR very usefull
> and error-prone feature.
> What about making usage of INC_PR a policy decision, demanding that
> all new packages should use INC_PR for their recipes, if .inc files
> are used. And then define a grace period
> during which all remaining packages should be converted to INC_PR (3
> months? Next release? I really don't know).
> 
> I'm sorry if this issue was already discussed somewhere and I'm
> duplicating the efforts
> or proposing already discarded idea.
> 




  reply	other threads:[~2011-09-20 19:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-20 19:04 [RFC] policy proposal: INC_PR Dmitry Eremin-Solenikov
2011-09-20 19:10 ` Mark Hatle [this message]
2011-09-20 20:30   ` Richard Purdie
2011-09-20 20:36     ` Koen Kooi
2011-09-20 20:47       ` 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=4E78E508.9080706@windriver.com \
    --to=mark.hatle@windriver.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.