All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Smith <msmith@cbnco.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH 2/2] grub: allow menu.lst to be	overridden	using GRUB_CONFIG_PACKAGE.
Date: Thu, 07 May 2009 15:03:18 -0400	[thread overview]
Message-ID: <4A033076.1080604@cbnco.com> (raw)
In-Reply-To: <1241678721.23982.71.camel@lenovo.internal.reciva.com>

Phil Blundell wrote:
> On Wed, 2009-05-06 at 18:38 -0400, Denys Dmytriyenko wrote:
>> On Wed, May 06, 2009 at 03:55:57PM -0400, Michael Smith wrote:

>> It was discussed many times here - no USE flags in OE.
> 
> Although you're right that it comes up here frequently, I think this "no
> USE flags" dogma is somewhat misguided.  It seems, frankly, absurd to
> require people to effectively fork a recipe whenever they want to make
> some or other configuration change.

I agree. If OpenEmbedded is a toolset for building distributions, rather 
than just a single distribution, then it'd be great if there were an 
easy way for distribution builders to make different packaging choices 
than the ones that have been made for Angstrom, without maintaining a 
forked copy of everything.

There might be ways to do it without USE flags. I think some tweaks to 
the overlay mechanism would help here. First, if I could create a 
package in an overlay that references files from the upstream in 
SRC_URI, I wouldn't have to copy all the patches, initscripts, etc. over 
to the overlay. For example, if my tree looks like the one below:

top/
  |-- local (overlay)
  |   `-- recipes
  |       `-- grub
              |-- grub_0.97.bb (overlay)
  |           `-- files
  |               `-- menu.lst (overlay)
  `-- openembedded (upstream)
      `-- recipes
          `-- grub
              |-- grub_0.97.bb (upstream)
              `-- files
                  |-- menu.lst (upstream)
                  `-- some.patch (upstream)

My grub_0.97.bb could reference file://some.patch in its SRC_URI and 
bitbake could look in the next COLLECTION after failing to find it in 
the local one. In fact, I wouldn't even really need a bbfile in the 
overlay; just having files/menu.lst in the overlay could be enough to 
replace it. I guess this might just require some fudging to how 
FILESPATH is set up in base.bbclass.

Second, if a recipe could include a recipe of the same name and let 
bitbake search for it in the next COLLECTION, overlay packages could get 
a lot smaller. Say I need to add a private patch to grub -- my overlay 
grub_0.97.bb, if I had one, could just look like this:

require_next ${PN}/${PN}_${PV}.bb
SRC_URI += "file://some-private.patch;patch=1"

Something similar might be useful for INHERIT. Instead of creating an 
overlay bbfile for linux_${PN}.bb, I could just override kernel.bbclass 
and then pull in the original.

Is this something people would be interested in, or am I making it more 
complicated than it needs to be?

Mike



  reply	other threads:[~2009-05-07 19:09 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-06 19:55 [PATCH 1/2] Remove unused grub-0.93 Michael Smith
2009-05-06 19:55 ` [PATCH 2/2] grub: allow menu.lst to be overridden using GRUB_CONFIG_PACKAGE Michael Smith
2009-05-06 22:38   ` Denys Dmytriyenko
2009-05-06 23:40     ` Tom Rini
2009-05-07  6:45     ` Phil Blundell
2009-05-07 19:03       ` Michael Smith [this message]
2009-05-15 22:21         ` Chris Larson
2009-05-16 19:03           ` Denys Dmytriyenko
2009-05-07 19:47       ` Denys Dmytriyenko
2009-05-07 18:24     ` Michael Smith
2009-05-07 18:27       ` [PATCH 1/3] Remove unused grub-0.93 Michael Smith
2009-05-07 18:27         ` [PATCH 2/3] grub: use ${libdir}, not /usr/lib Michael Smith
2009-05-07 18:27           ` [PATCH 3/3] grub: split menu.lst into a separate package, grub-config Michael Smith
2009-05-07 19:02             ` Chris Larson
2009-05-07 22:19               ` Michael Smith
2009-05-07 22:37                 ` manual missing dependencies section? Rich Pixley
2009-05-07 23:59                   ` Cliff Brake
2009-05-07 19:52             ` [PATCH 3/3] grub: split menu.lst into a separate package, grub-config Denys Dmytriyenko
2009-05-07 19:18           ` [PATCH 2/3] grub: use ${libdir}, not /usr/lib Leon Woestenberg
2009-05-07 19:26             ` Otavio Salvador
2009-05-07 18:36               ` Phil Blundell
2009-05-07 19:58               ` Michael Smith
2009-05-07 20:02                 ` Otavio Salvador
2009-05-07 20:05                 ` Leon Woestenberg
2009-05-07 22:24                   ` Michael Smith
2009-05-08  1:35                     ` Otavio Salvador
2009-05-08 19:15                       ` Leon Woestenberg
2009-05-09  0:46                         ` Otavio Salvador
2009-05-12  2:53                   ` Michael Smith
2009-05-12  2:53                   ` [PATCH] Remove unused grub-0.93 Michael Smith
2009-05-12  2:53                     ` [PATCH] grub: Split the grub-install script and friends into a separate package Michael Smith
2009-05-12  3:02                       ` Tom Rini
2009-05-12  3:18                         ` Michael Smith
2009-05-12  4:13                           ` Tom Rini
2009-05-07 20:02             ` [PATCH 2/3] grub: use ${libdir}, not /usr/lib Graeme Gregory
2009-05-07 19:25         ` [PATCH 1/3] Remove unused grub-0.93 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=4A033076.1080604@cbnco.com \
    --to=msmith@cbnco.com \
    --cc=openembedded-devel@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.