public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Brian Harring <ferringb@gmail.com>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] Re: Reorganizing handling of target specific profiles (Was: Split desktop profile patches & news item for review)
Date: Sat, 13 Mar 2010 21:25:45 -0800	[thread overview]
Message-ID: <20100314052545.GB17983@hrair> (raw)
In-Reply-To: <1268524966.12656.18.camel@localhost>

[-- Attachment #1: Type: text/plain, Size: 3418 bytes --]

On Sun, Mar 14, 2010 at 02:02:46AM +0200, Mart Raudsepp wrote:
> On Sat, 2010-03-13 at 13:16 -0800, Brian Harring wrote:
> > While I agree in principle within mixins, no one here is discussing 
> > the QA affect of it- right now we can do visibility scans of 
> > combinations of gnome + amd64 + 2010 because they're defined.
> 
> What sort of QA affects do you imagine it having?

Simple enough.  Right now, you change a profile, or want to stable a 
pkg, you can do a scan and identify all new visibility issues from 
profiles- you can validate up front that for the list of 
supported/stable profiles, these changes occur.

If things are shifted over to prefering users mixing/matching profiles 
on their own (meaning we no longer have a gnome amd64 2010 for 
example), devs no longer get QA warnings when they break stuff for it.

Users see it however.

> Though I'm talking in the context of what I proposed - using it for just
> the target profiles that only tweak USE flags and other such defaults,
> nothing else.

Current QA (repoman/pcheck), if a use flag is defaulted on, it's deps 
in a pkg must be visible.  Via repoman/pcheck, they ensure that the 
default use configuration for a profile, all visible pkgs dependencies 
are visible (making the pkg fully usable).

Consider mixing/matching gnome/kde with a profile that has quite a few 
packages masked- say mips.  To be clear, this is a hypothetical case- 
I know it exists, I'm just choosing two likely targets.  Say gnome 
requires some codecs use flag flipped on triggering a dep on a pkg 
masked for mips.

I'm not saying these situations can't be worked around- we already fix 
them now as they come up.  The thing is, whenever you change something 
now, those profile combinations are validated- with mix/match 
approach, that validation won't be occuring, the users will be the 
ones seeing the breakage rather than the dev.

> I considered QA affects for that case, at least in my own
> thoughts. I think QA would be checked anyhow there, as part of all USE
> flags enabled assumpting testing or static testing of various USE flag
> combinations of a package (e.g, for statically finding circular
> dependencies or the like).

Either you're suggesting that repoman/pcheck would have to scan all 
arbitrary combinations of gnome/kde/desktop w/ misc arches, or you 
need to be a fair bit more precise about how QA tools would identify 
what profile combinations to check.

If you're proposing that the QA tool arbitrarily combines arches w/ 
various desktop/server mixins, I'll again note that the run time of 
visibility scans is directly related to # of profiles to check- for 
example, removal of mips profiles from profiles.desc is if memory 
serves me a ~33% reduction in visibility runtime for pcheck.

For repoman (which all devs have to use for commiting), # of profiles 
is even more of a critical value performance wise.


> Do you foresee bad QA affects for the for the
> desktop/developer/gnome/kde/server profiles case too, or just when
> mixing in selinux/toolchains/etc?

Issues will exist regardless of what the combination is.  The point 
I'm making is that with the current model, we catch those issues prior 
to commit- having users mix/match on their own means users will see 
those issues rather than devs.  Literally, they'll see the breakage.

~harring

[-- Attachment #2: Type: application/pgp-signature, Size: 198 bytes --]

  reply	other threads:[~2010-03-14  5:27 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-04 14:52 [gentoo-dev] Split desktop profile patches & news item for review Theo Chatzimichos
2010-03-04 15:29 ` Jeremy Olexa
2010-03-04 17:22 ` [gentoo-dev] " Duncan
2010-03-04 17:59 ` [gentoo-dev] " Sebastian Pipping
2010-03-04 18:15   ` Samuli Suominen
2010-03-04 22:36     ` Ben de Groot
2010-03-05  8:28 ` Joshua Saddler
2010-03-05 12:57   ` Ben de Groot
2010-03-05 13:46     ` Theo Chatzimichos
2010-03-05 17:59       ` Zeerak Mustafa Waseem
2010-03-05 19:01         ` [gentoo-dev] " Duncan
2010-03-05 19:11           ` Theo Chatzimichos
2010-03-05 19:24           ` Zeerak Mustafa Waseem
2010-03-05 19:12     ` [gentoo-dev] " Mike Frysinger
2010-03-08  1:17 ` [gentoo-dev] " Theo Chatzimichos
2010-03-11  1:36   ` Ben de Groot
2010-03-11 20:20     ` Mart Raudsepp
2010-03-11 22:20       ` Ben de Groot
2010-03-12  8:36         ` Mart Raudsepp
2010-03-12  9:48           ` Theo Chatzimichos
2010-03-12 17:39             ` Ben de Groot
2010-03-13 10:07               ` Theo Chatzimichos
2010-03-13 23:37             ` Mart Raudsepp
2010-03-12 15:47           ` Ben de Groot
2010-03-12 17:34             ` Duncan
2010-03-13 23:25             ` Mart Raudsepp
2010-03-23 14:29   ` Theo Chatzimichos
2010-03-08 17:13 ` [gentoo-dev] Reorganizing handling of target specific profiles (Was: Split desktop profile patches & news item for review) Mart Raudsepp
2010-03-08 22:40   ` [gentoo-dev] " Peter Hjalmarsson
2010-03-08 22:44     ` Alec Warner
2010-03-13 21:16     ` Brian Harring
2010-03-14  0:02       ` Mart Raudsepp
2010-03-14  5:25         ` Brian Harring [this message]
2010-03-09  1:26   ` [gentoo-dev] " Robin H. Johnson

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=20100314052545.GB17983@hrair \
    --to=ferringb@gmail.com \
    --cc=gentoo-dev@lists.gentoo.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