From: Phil Blundell <philb@gnu.org>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE
Date: Wed, 23 Nov 2011 17:53:42 +0000 [thread overview]
Message-ID: <1322070823.26081.37.camel@phil-desktop> (raw)
In-Reply-To: <6713028.ehgggRD6ui@helios>
On Wed, 2011-11-23 at 17:45 +0000, Paul Eggleton wrote:
> I think I'm right in assuming that when we introduce a DISTRO_FEATURES feature
> we are almost always doing it to allow disabling some existing functionality,
> rather than enabling something new.
I'm not sure that this is true. There are several recent(ish) examples
of DISTRO_FEATURES that are for new things (e.g. directfb as the base
graphics layer, or gold as the linker), rather than just turning off
switches that were previously the default. And...
>In that case, should we not be providing the appropriate mechanism so
>that can exclude the features they don't want rather than including the
>ones that they do?
... it seems as though this is basically equivalent to the
"nopulseaudio" proposal, and would suck for the same reasons.
>This will solve the issue, but the more I think about it the more I
>don't like it - from the perspective of a new user it just puts a fairly
>arbitrary line between old features and new ones. If you want control
>over the full range of options you'll need to look at two places.
I'm not quite sure I understand what you're saying here. Richard's
proposal, if I understand it right, is basically giving DISTROs a way to
say "these are the features that I know about and have decided I don't
want" so that the compatibility code can give them sensible defaults for
the features that they aren't aware of.
It's true that this will (in some sense) segment them into "new" and
"old" features but it isn't going to be a hard, fixed division and I
would expect that most DISTROs will do a fairly good job of keeping up
to date with the newly introduced flags.
p.
next prev parent reply other threads:[~2011-11-23 18:00 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-23 15:32 [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE Paul Eggleton
2011-11-23 15:32 ` [PATCH 1/3] default-distrovars: add pulseaudio to DISTRO_FEATURES Paul Eggleton
2011-11-23 15:32 ` [PATCH 2/3] qt4: make pulseaudio support conditional upon DISTRO_FEATURES Paul Eggleton
2011-11-23 15:32 ` [PATCH 3/3] gst-plugins-good: " Paul Eggleton
2011-11-23 15:59 ` [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE Koen Kooi
2011-11-23 16:33 ` Paul Eggleton
2011-11-23 16:48 ` Phil Blundell
2011-11-23 17:09 ` Richard Purdie
2011-11-23 17:45 ` Paul Eggleton
2011-11-23 17:49 ` Paul Eggleton
2011-11-23 17:53 ` Phil Blundell [this message]
2011-11-23 18:22 ` Otavio Salvador
2011-12-29 12:55 ` Paul Eggleton
2012-01-16 17:58 ` Paul Eggleton
2012-01-16 18:12 ` Phil Blundell
2012-01-17 23:13 ` Process for New DISTRO_FEATURES (was: Re: [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE) Richard Purdie
2012-01-17 20:02 ` [PATCH 0/3] Make pulseaudio a DISTRO_FEATURE Tom Rini
2012-01-27 10:43 ` Samuel Stirtzel
2012-01-27 10:54 ` Paul Eggleton
2012-01-27 11:34 ` Samuel Stirtzel
2012-01-27 12:13 ` Paul Eggleton
2012-01-28 20:05 ` Khem Raj
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=1322070823.26081.37.camel@phil-desktop \
--to=philb@gnu.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