Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v10 00/28] efl bump to 1.15.2
Date: Sat, 19 Dec 2015 16:57:40 +0100	[thread overview]
Message-ID: <20151219155740.GB3394@free.fr> (raw)
In-Reply-To: <56756A03.7050701@openwide.fr>

Romain, All,

On 2015-12-19 15:30 +0100, Romain Naour spake thusly:
> Le 18/12/2015 00:11, Thomas Petazzoni a ?crit :
> > On Tue, 15 Dec 2015 23:40:12 +0100, Romain Naour wrote:
[--SNIP--]
> >>   package/efl/libefl: add OpenGLES support
> > This one I wasn't sure about the "choice"; we had some discussion about
> > it with Yann on IRC.
> It's not clear to me when we can really use the full OpenGL support.
> I thought that the full OpenGL support was only needed by some specific hardware...

You don't care about the hardware in this case. If there is a full
OpenGL provider *and* Xorg is enabled, then you can do full OpenGL.
It is the responsibility of the libgl provider to provide hardware
access (e.g. mesa3d, nvidia-driver, rpi-userland...)

So, in case my previous comments were not clear, here we go:

  - full OpenGL currently requires Xorg and a libgl provider.

Or, put in other words:

  - if you have Xorg and you have a libgl provider, then you can do full
    OpenGL.

So, for EFL, I would add an new entry to the choice:

    config BR2_PACKAGE_EFL_GL
        bool "opengl"
        depends on BR2_PACKAGE_HAS_LIBGL
        depends on BR2_PACKAGE_LIBEFL_X_XLIB

    comment "opengl support needs X11 support and a libgl provider"
        depends on !BR2_PACKAGE_HAS_LIBGL || !BR2_PACKAGE_LIBEFL_X_XLIB


Alternatively, you can go with a select, like:

    config BR2_PACKAGE_EFL_GL
        bool "opengl"
        depends on BR2_PACKAGE_HAS_LIBGL
        depends on BR2_PACKAGE_XORG7  # X_XLIB
        select BR2_PACKAGE_LIBEFL_X_XLIB

    comment "opengl support needs Xorg and a libgl provider"
        depends on !BR2_PACKAGE_HAS_LIBGL || !BR2_PACKAGE_XORG7


I think I'd prefer the first solution, but either is OK for me.

But now, if you are not interested in full OpenGL, or you can't test it,
or really don't care, just leave it aside. Interested parties can submit
that support later.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

      parent reply	other threads:[~2015-12-19 15:57 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-15 22:40 [Buildroot] [PATCH v10 00/28] efl bump to 1.15.2 Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 01/28] package/efl/libefl: new package Romain Naour
2015-12-16 20:32   ` Thomas Petazzoni
2015-12-16 21:36     ` Yann E. MORIN
2015-12-16 21:45       ` Thomas Petazzoni
2015-12-17 22:09     ` Romain Naour
2015-12-17 22:17       ` Thomas Petazzoni
2015-12-17 22:21       ` Yann E. MORIN
2015-12-15 22:40 ` [Buildroot] [PATCH v10 02/28] package/efl/libefl: add harfbuzz dependency Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 03/28] package/efl/libefl: add wayland dependency Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 04/28] package/efl/libefl: add webp dependency Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 05/28] package/efl/libefl: add frame buffer config option Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 06/28] package/efl/libefl: add jp2k " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 07/28] package/efl/libefl: add X11 support Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 08/28] package/efl/libefl: add OpenGLES support Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 09/28] package/efl/libefl: enable xinput2.2 Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 10/28] package/libevas-generic-loaders: bump to version 1.15.0 Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 11/28] package/libevas-generic-loaders: add libraw dependency Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 12/28] package/efl/libelementary: bump to version 1.15.2 Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 13/28] package/efl: libefl and libelementary share the same version number Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 14/28] package/enlightenment: bump to version 0.19.12 Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 15/28] package/expedite: bump to efl-1.15 branch Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 16/28] package/dbus-cpp: switch to libefl Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 17/28] package/efl/libethumb: remove package Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 18/28] package/efl/libedje: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 19/28] package/efl/libembryo: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 20/28] package/efl/libeio: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 21/28] package/efl/libefreet: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 22/28] package/efl/libedbus: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 23/28] package/efl/libecore: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 24/28] package/efl/libevas: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 25/28] package/efl/libeet: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 26/28] package/efl/libeina: " Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 27/28] package/libemotion-generic-players: new package Romain Naour
2015-12-15 22:40 ` [Buildroot] [PATCH v10 28/28] package/efl: rename libefl to efl Romain Naour
2015-12-17 23:11 ` [Buildroot] [PATCH v10 00/28] efl bump to 1.15.2 Thomas Petazzoni
2015-12-19 14:30   ` Romain Naour
2015-12-19 14:47     ` Thomas Petazzoni
2015-12-19 15:57     ` Yann E. MORIN [this message]

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=20151219155740.GB3394@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /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