From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/2] libsdl: enable opengl for libsdl
Date: Tue, 16 Aug 2011 11:19:37 +0100 [thread overview]
Message-ID: <1313489977.13995.13.camel@rex> (raw)
In-Reply-To: <21E0E8C4-433D-417B-ADAE-F4D444C9B35C@dominion.thruhere.net>
On Tue, 2011-08-16 at 09:52 +0200, Koen Kooi wrote:
> Op 15 aug. 2011, om 15:05 heeft Jiajun Xu het volgende geschreven:
>
> > To enable opengl for libsdl, we need DEPENDS on virtual/libgl. Add a check for
> > "opengl" in DISTRO_FEATURES and when it's set, "--enable-video-opengl"
> > and "virtual/libgl" will be added into EXTRA_CONF and DEPENDS.
>
> Is that really a distro feature? Opengl has always been a machine feature in OE, so with this the situation gets a lot worse.
Not really. This is one of the flags that makes sense at both a machine
*and* distro level. The distro flag means "build GL components" or can
be used to globally disable GL. The machine flag means the hardware can
support accelerated GL. This isn't the only such flag and was
anticipated long ago and why we have COMBINED_FEATURES (or whatever its
called).
The tricky piece to fit into this is software rendering. There are some
cases where we might want to build and include mesa-xlib even though we
know it will be slow (e.g. qemu machines to allow fuller testing).
Should or shouldn't those machines list opengl? Should we always fall
back to software rendering or error? Those questions are harder to
answer.
Cheers,
Richard
next prev parent reply other threads:[~2011-08-16 10:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-15 13:04 [PATCH 0/2] Extend target version for libsdl and enable openGL for it Jiajun Xu
2011-08-15 13:05 ` [PATCH 1/2] libsdl: add target version Jiajun Xu
2011-08-15 13:05 ` [PATCH 2/2] libsdl: enable opengl for libsdl Jiajun Xu
2011-08-16 7:52 ` Koen Kooi
2011-08-16 10:19 ` Richard Purdie [this message]
2011-08-15 14:02 ` [PATCH 0/2] Extend target version for libsdl and enable openGL for it Richard Purdie
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=1313489977.13995.13.camel@rex \
--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