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 --]
next prev parent 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