[Buildroot] [PATCH] package/mesa-demo: remove duplicate osmesa option

[Buildroot] [PATCH] package/mesa-demo: remove duplicate osmesa option

[Audrey Motheron]

--disable-osmesa option is unconditionally added to CONF_OPTS
even if --enable-osmesa is used latter.

Signed-off-by: Audrey Motheron <audrey.motheron@gmail.com>
---
 package/mesa3d-demos/mesa3d-demos.mk | 1 -
 1 file changed, 1 deletion(-)

diff --git a/package/mesa3d-demos/mesa3d-demos.mk b/package/mesa3d-demos/mesa3d-demos.mk
index cbc15c3b69..9182803748 100644
--- a/package/mesa3d-demos/mesa3d-demos.mk
+++ b/package/mesa3d-demos/mesa3d-demos.mk
@@ -12,7 +12,6 @@ MESA3D_DEMOS_DEPENDENCIES = host-pkgconf
 MESA3D_DEMOS_LICENSE = MIT
 
 MESA3D_DEMOS_CONF_OPTS += \
-	--disable-osmesa \
 	--disable-gles1
 
 ifeq ($(BR2_PACKAGE_XORG7)$(BR2_PACKAGE_HAS_LIBGL),yy)
-- 
2.14.3

[Buildroot] [PATCH] package/mesa-demo: remove duplicate osmesa option

[Adrian Perez de Castro]

Hello,

Thanks for the patch, indeed the option was being duplicated! Just one small
comment: usually the commit summary line has only the package name, without
the ?package/? prefix, and in this case the package name is ?mesa-demos?,
so the first line of the commit message should be.

On Mon, 12 Feb 2018 22:59:31 +0100, Audrey Motheron <audrey.motheron@gmail.com> wrote:
> --disable-osmesa option is unconditionally added to CONF_OPTS
> even if --enable-osmesa is used latter.
> 
> Signed-off-by: Audrey Motheron <audrey.motheron@gmail.com>
> ---
>  package/mesa3d-demos/mesa3d-demos.mk | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/package/mesa3d-demos/mesa3d-demos.mk b/package/mesa3d-demos/mesa3d-demos.mk
> index cbc15c3b69..9182803748 100644
> --- a/package/mesa3d-demos/mesa3d-demos.mk
> +++ b/package/mesa3d-demos/mesa3d-demos.mk
> @@ -12,7 +12,6 @@ MESA3D_DEMOS_DEPENDENCIES = host-pkgconf
>  MESA3D_DEMOS_LICENSE = MIT
>  
>  MESA3D_DEMOS_CONF_OPTS += \
> -	--disable-osmesa \
>  	--disable-gles1
>  
>  ifeq ($(BR2_PACKAGE_XORG7)$(BR2_PACKAGE_HAS_LIBGL),yy)
> -- 
> 2.14.3

Reviewed-by: Adrian Perez de Castro <aperez@igalia.com>


--
 Adri?n ?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180212/95c882e4/attachment.asc>

[Buildroot] [PATCH] package/mesa-demo: remove duplicate osmesa option

[Thomas Petazzoni]

Hello,

On Mon, 12 Feb 2018 23:51:12 +0000, Adrian Perez de Castro wrote:
> Hello,
> 
> Thanks for the patch, indeed the option was being duplicated! Just one small
> comment: usually the commit summary line has only the package name, without
> the ?package/? prefix, and in this case the package name is ?mesa-demos?,

Actually the package name is mesa3d-demos :-)

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com

[Buildroot] [PATCH] package/mesa-demo: remove duplicate osmesa option

[Thomas Petazzoni]

Hello,

On Mon, 12 Feb 2018 22:59:31 +0100, Audrey Motheron wrote:
> --disable-osmesa option is unconditionally added to CONF_OPTS
> even if --enable-osmesa is used latter.
> 
> Signed-off-by: Audrey Motheron <audrey.motheron@gmail.com>
> ---
>  package/mesa3d-demos/mesa3d-demos.mk | 1 -
>  1 file changed, 1 deletion(-)

As Adrian said, the commit title should match the package name:
mesa3d-demos. I've fixed that up and applied to master. Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com