Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [RFC PATCH 0/1] QA check for defined packages
Date: Wed, 12 Oct 2011 20:18:25 +0100	[thread overview]
Message-ID: <1318447114.23801.178.camel@ted> (raw)
In-Reply-To: <4E95D1C1.2020000@linux.intel.com>

On Wed, 2011-10-12 at 10:43 -0700, Joshua Lock wrote:
> On 10/12/2011 05:37 AM, Richard Purdie wrote:
> > On Tue, 2011-10-11 at 15:29 -0700, Joshua Lock wrote:
> >> I'm looking for some comments on this WIP patch.
> >>
> >> The reason I've added this is because the hob GUI requires us to offer much
> >> more reliable metadata - we show the user available packages, as defined by
> >> each recipe, to add to their image. Should a recipe define a package and yet
> >> not actually create it and the user has included that in their image we cause
> >> errors at build time.
> >>
> >> However whilst the idea seems sound enough, there are some caveats - running
> >> a world build with this patch I see failures fairly early on at least a few
> >> of which I'd expect we'll need to special-case:
> >> * eglibc-local
> >> * linux-yocto
> >> * linux-libc-headers
> >> * gcc-runtime
> >>
> >> Thoughts?
> > 
> > I think we probably do need to cleanup some of that data.
> > 
> > I have some thoughts about where we're at with hob and this is probably
> > as good a time as any to share them.
> > 
> > Effectively the problem is that there is a large body of data we only
> > compute during the build process. This includes what the exact runtime
> > dependencies of packages are, which packages exactly are generated
> > (PACKAGES_DYNAMIC), what the size of the packages are, how long they
> > take to build and so on. Some things we can fix, some things we can hack
> > around but at the end of the day there are some things we just plain
> > can't change.
> > 
> > I'm beginning to think we need to have two phases of control of the
> > system:
> > 
> > a) The build phase
> > 
> > This is the step that generates the package feeds.
> > 
> > b) The image construction phase
> > 
> > This is the step that takes the package feed data and turns it into an
> > image.
> 
> I actually think this is a neat idea, in fact we have the beginnings of
> a Gtk+ GUI to create images from a package feed which Rob Bradford
> worked on some 3 years or so ago - puccho. It's no doubt bit-rotten but
> may be worth a look.
> 
> I think we can reuse pieces from puccho and hob to create a GUI per this
> high-level design and I think we'd be much better off for it.
> 
> > Obviously, you can skip to b) if you already have a package feed.
> 
> a), right? Indeed I expect that this will be more in line with certain
> proposed use cases.

No, you'd skip the package feed generation and just end up using it so
skip a) and start from b).

> > So we'd be talking about two UI's that could effectively hand off to
> > each other and share a "build in progress" feedback to the user system.
> > The image construction dialog would have a "Missing Packages? Build them
> > here" type switch. This would mean the build system can continue on at
> > what it does best yet the UI can let the users do what they want to do,
> > particularly on a prebuilt package feed.
> 
> I like it. I think we had to "write one to throw away" to realise quite
> how much data we're missing up front but I support the proposed design.

I'd not say thrown away. We've moved a lot of the code forward and
significant pieces would get reused...

Cheers,

Richard




  reply	other threads:[~2011-10-12 19:24 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-11 22:29 [RFC PATCH 0/1] QA check for defined packages Joshua Lock
2011-10-11 22:29 ` [RFC PATCH 1/1] insane.bbclass: add qa check to ensure declared packages exist Joshua Lock
2011-10-12 12:37 ` [RFC PATCH 0/1] QA check for defined packages Richard Purdie
2011-10-12 12:40   ` Koen Kooi
2011-10-12 17:43   ` Joshua Lock
2011-10-12 19:18     ` Richard Purdie [this message]
2011-10-12 19:23       ` Joshua Lock
2011-10-12 12:39 ` Richard Purdie
2011-10-12 17:44   ` Joshua Lock

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=1318447114.23801.178.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox