[Buildroot] [PATCH v2] package/efl: enable Wayland support if available

[Buildroot] [PATCH v2] package/efl: enable Wayland support if available

[Romain Naour]

If all the dependencies are present to select the efl wayland
support, enable it by default.

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
v2: rebase on master
---
 package/efl/Config.in | 1 +
 1 file changed, 1 insertion(+)

diff --git a/package/efl/Config.in b/package/efl/Config.in
index 5e89af6..a8f9d41 100644
--- a/package/efl/Config.in
+++ b/package/efl/Config.in
@@ -180,6 +180,7 @@ config BR2_PACKAGE_EFL_WAYLAND
 	depends on BR2_PACKAGE_EFL_EEZE # efl drm
 	depends on BR2_PACKAGE_EFL_OPENGLES # OpenGL ES with EGL support only
 	select BR2_PACKAGE_EFL_DRM
+	default y
 
 comment "Wayland support needs udev /dev management (eeze) and OpenGL ES w/ EGL, threads"
 	depends on BR2_PACKAGE_WAYLAND
-- 
2.9.3

[Buildroot] [PATCH v2] package/efl: enable Wayland support if available

[Thomas Petazzoni]

Hello,

On Mon,  6 Feb 2017 17:24:37 +0100, Romain Naour wrote:
> If all the dependencies are present to select the efl wayland
> support, enable it by default.
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> ---
> v2: rebase on master
> ---
>  package/efl/Config.in | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/package/efl/Config.in b/package/efl/Config.in
> index 5e89af6..a8f9d41 100644
> --- a/package/efl/Config.in
> +++ b/package/efl/Config.in
> @@ -180,6 +180,7 @@ config BR2_PACKAGE_EFL_WAYLAND
>  	depends on BR2_PACKAGE_EFL_EEZE # efl drm
>  	depends on BR2_PACKAGE_EFL_OPENGLES # OpenGL ES with EGL support only
>  	select BR2_PACKAGE_EFL_DRM
> +	default y

Why would we do this? For the X.org support, we also have a "depends
on" and the option is not "default y". I don't see why the X.org and
Wayland cases should be handled differently. If there is a reason, it
should be explained in the commit log IMO.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com