All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Eric Bénard" <eric@eukrea.com>
To: Otavio Salvador <otavio@ossystems.com.br>
Cc: "meta-freescale@yoctoproject.org" <meta-freescale@yoctoproject.org>
Subject: Re: [meta-fsl-arm][PATCH 1/3] u-boot: Rename recipe to u-boot-fsl
Date: Fri, 14 Dec 2012 16:30:24 +0100	[thread overview]
Message-ID: <20121214163024.7427cf1d@eb-e6520> (raw)
In-Reply-To: <CAP9ODKo1sYOt2rMe4LmF_hf2xqJt=f7bye5rZ2hMYZKpYaCuUQ@mail.gmail.com>

Le Fri, 14 Dec 2012 13:14:54 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :

> On Fri, Dec 14, 2012 at 12:26 PM, Eric Bénard <eric@eukrea.com> wrote:
> > Le Fri, 14 Dec 2012 16:15:33 +0200,
> > Andrei Gherzan <andrei@gherzan.ro> a écrit :
> >> What if uboot will have a git version in oe-core in the future? Or a
> >> greater one. How would you fix that? I still think this is a good longterm
> >> solution.
> >>
> > well, in that case the BSP recipe will be used as the layer has a higher
> > priority.
> >
> > And to not have this kind of issue, you can simpy add the following
> > lines to your BSP's u-boot recipe :
> > DEFAULT_PREFERENCE = "-1"
> > DEFAULT_PREFERENCE_machine = "1"
> >
> > or simply change to a u-boot_git.bbappend to just append your machine
> > specific changes to oe-core's default recipe.
> >
> > or add something like this in your BSP conf file :
> > PREFERRED_PROVIDER_virtual/kernel ?= "linux-yocto"
> > PREFERRED_VERSION_linux-yocto = "3.4%"
> >
> > Check 1.2.9 in BSP Guide for examples on how this can be done (example
> > for linux-yocto but the use case is the same here).
> 
> I agree it is a possible way of doing it however I also think we
> should opt for a safe route.
> 
> The Andrei's proposal make it harder to it to behave strangely so I
> think it is a good option for long term. Another positive result of it
> is that the new name makes clear we're not really using u-boot
> mainline but mainline + patches. I support this change as it improves
> the clearness for new users.
> 
that's your choice but please note that you open the door to renaming
any recipe :
- either to workaround a problem in an other (or in your own) layer
  instead of really solving it
- or simply each time you add a patch to a recipe which then becomes non
  mainline !

IMHO renaming the recipe is not the right way to do.

Eric


  reply	other threads:[~2012-12-14 15:30 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-14 11:57 [meta-fsl-arm][PATCH 1/3] u-boot: Rename recipe to u-boot-fsl Andrei Gherzan
2012-12-14 11:57 ` [meta-fsl-arm][PATCH 2/3] u-boot-fsl: This recipe provides u-boot Andrei Gherzan
2012-12-14 11:57 ` [meta-fsl-arm][PATCH 3/3] fsl-default-providers.inc: u-boot recipe was renamed to u-boot-fsl Andrei Gherzan
2012-12-14 12:00 ` [meta-fsl-arm][PATCH 1/3] u-boot: Rename recipe " Andrei Gherzan
2012-12-14 12:36   ` Daiane Angolini
2012-12-14 12:41     ` Otavio Salvador
2012-12-14 12:43       ` Andrei Gherzan
2012-12-14 13:20         ` Daiane Angolini
2012-12-14 13:23           ` Andrei Gherzan
2012-12-14 12:56       ` Daiane Angolini
2012-12-14 12:27 ` Daiane Angolini
2012-12-14 13:53 ` Eric Bénard
2012-12-14 13:56   ` Andrei Gherzan
2012-12-14 14:01     ` Eric Bénard
2012-12-14 14:04       ` Andrei Gherzan
2012-12-14 14:13         ` Eric Bénard
     [not found]           ` <CAK18fxF6VYBTZChgrO_n9PGge8tCWngyFPw=6JD+8sTBMPmeTQ@mail.gmail.com>
2012-12-14 14:26             ` Eric Bénard
2012-12-14 15:14               ` Otavio Salvador
2012-12-14 15:30                 ` Eric Bénard [this message]
2012-12-14 15:33                   ` Otavio Salvador
2012-12-14 16:50                     ` Daiane Angolini
2012-12-14 16:53                       ` Otavio Salvador
2012-12-14 17:29                         ` Andrei Gherzan
2012-12-14 17:52                           ` Otavio Salvador
2012-12-14 17:58                       ` Eric Bénard
2012-12-14 18:12                         ` Otavio Salvador
2012-12-14 18:22                           ` Eric Bénard
     [not found]                             ` <CA+jg_OWZpR7iazC=anVXbFFbzm8jV5O-f1ZwOoTuC3a3j-kzTA@mail.gmail.com>
2012-12-14 23:08                               ` Daiane Angolini
2012-12-14 23:11                                 ` Andrei Gherzan
2012-12-14 23:20                                 ` Eric Bénard
2012-12-15 16:04                                 ` Otavio Salvador
2012-12-15 18:18                                   ` Eric Bénard
2012-12-15 18:33                                     ` Otavio Salvador
2012-12-15 20:43                                       ` Andrei Gherzan
2012-12-16 14:30                                       ` Daiane Angolini
2012-12-16 17:06                                         ` Otavio Salvador
2012-12-19 19:35                                           ` McClintock Matthew-B29882
2012-12-19 19:40                                             ` Otavio Salvador
2012-12-19 19:46                                               ` McClintock Matthew-B29882
2012-12-19 19:54                                                 ` Otavio Salvador

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=20121214163024.7427cf1d@eb-e6520 \
    --to=eric@eukrea.com \
    --cc=meta-freescale@yoctoproject.org \
    --cc=otavio@ossystems.com.br \
    /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.