[Buildroot] [PATCH 1/3] package/x11r7/xserver_xorg-server: fix optional glx dependency

[Buildroot] [PATCH 1/3] package/x11r7/xserver_xorg-server: fix optional glx dependency

[Bernd Kuhls]

Typo was added by commit
https://git.buildroot.net/buildroot/commit/?id=c2856aa2857d616788684a053f1c886da00d977f

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
index c15047cbe4..41c82ad930 100644
--- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
+++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
@@ -120,7 +120,7 @@ else # modular
 XSERVER_XORG_SERVER_CONF_OPTS += --disable-kdrive --disable-xfbdev
 endif
 
-ifeq ($(BR2_PACKAGE_MESA3D_OPENGL_GLX)),y)
+ifeq ($(BR2_PACKAGE_MESA3D_OPENGL_GLX),y)
 XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
 XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d
 else
-- 
2.20.1

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Bernd Kuhls]

Due to the switch of mesa3d to the meson build system the upstream
tarball does not include dri.pc.in anymore, it is created during the
build now:
https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93

In order to provide dri.pc we need to add a version taken from a dri-
based mesa3d build to our source repository, by doing so we can reduce
the number of sed commands.

Without this patch the build of xserver_xorg-server, switched to virtual
opengl provider, will fail:

checking for XSERVERCFLAGS... no
configure: error: Package requirements (fixesproto >= 5.0 damageproto
 >= 1.1 xcmiscproto >= 1.2.0 xtrans >= 1.3.5 bigreqsproto >= 1.1.0
 xproto >= 7.0.31 randrproto >= 1.6.0 renderproto >= 0.11 xextproto
 >= 7.2.99.901 inputproto >= 2.3 kbproto >= 1.0.3 fontsproto >= 2.1.3
 pixman-1 >= 0.27.2 videoproto recordproto >= 1.13.99.1 resourceproto
 >= 1.2.0 xf86driproto >= 2.1.0 glproto >= 1.4.17 dri >= 7.8.0 dri3proto
 presentproto >= 1.1 xineramaproto xkbfile  pixman-1 >= 0.27.2 xfont2
 >= 2.0.0 xau xshmfence >= 1.1 xdmcp) were not met:

Package 'dri', required by 'virtual:world', not found


Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
v2: no changes, included in new series switching xserver_xorg-server
    to virtual opengl provider

 package/mesa3d-headers/dri.pc            | 10 ++++++++++
 package/mesa3d-headers/mesa3d-headers.mk |  9 ++-------
 2 files changed, 12 insertions(+), 7 deletions(-)
 create mode 100644 package/mesa3d-headers/dri.pc

diff --git a/package/mesa3d-headers/dri.pc b/package/mesa3d-headers/dri.pc
new file mode 100644
index 0000000000..bd9a33b6c6
--- /dev/null
+++ b/package/mesa3d-headers/dri.pc
@@ -0,0 +1,10 @@
+prefix=/usr
+libdir=${prefix}/lib
+includedir=${prefix}/include
+
+dridriverdir=/usr/lib/dri
+
+Name: dri
+Description: Direct Rendering Infrastructure
+Version: @VERSION@
+Cflags: -I${includedir}
diff --git a/package/mesa3d-headers/mesa3d-headers.mk b/package/mesa3d-headers/mesa3d-headers.mk
index 8670184742..99d6e52498 100644
--- a/package/mesa3d-headers/mesa3d-headers.mk
+++ b/package/mesa3d-headers/mesa3d-headers.mk
@@ -34,13 +34,8 @@ ifeq ($(BR2_PACKAGE_XORG7),y)
 # Not using $(SED) because we do not want to work in-place, and $(SED)
 # contains -i.
 define MESA3D_HEADERS_BUILD_DRI_PC
-	sed -e 's:@\(exec_\)\?prefix@:/usr:' \
-	    -e 's:@libdir@:$${exec_prefix}/lib:' \
-	    -e 's:@includedir@:$${prefix}/include:' \
-	    -e 's:@DRI_DRIVER_INSTALL_DIR@:$${libdir}/dri:' \
-	    -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
-	    -e 's:@DRI_PC_REQ_PRIV@::' \
-	    $(@D)/src/mesa/drivers/dri/dri.pc.in \
+	sed -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
+	    package/mesa3d-headers/dri.pc \
 	    >$(@D)/src/mesa/drivers/dri/dri.pc
 endef
 
-- 
2.20.1

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Vincent Fazio]

Tested xserver_xorg-server and nvidia-driver builds with this patch applied

Tested-by: Vincent Fazio <vfazio@xes-inc.com>

On 10/23/19 1:22 AM, Bernd Kuhls wrote:
> Due to the switch of mesa3d to the meson build system the upstream
> tarball does not include dri.pc.in anymore, it is created during the
> build now:
> https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93
>
> In order to provide dri.pc we need to add a version taken from a dri-
> based mesa3d build to our source repository, by doing so we can reduce
> the number of sed commands.
>
> Without this patch the build of xserver_xorg-server, switched to virtual
> opengl provider, will fail:
>
> checking for XSERVERCFLAGS... no
> configure: error: Package requirements (fixesproto >= 5.0 damageproto
>   >= 1.1 xcmiscproto >= 1.2.0 xtrans >= 1.3.5 bigreqsproto >= 1.1.0
>   xproto >= 7.0.31 randrproto >= 1.6.0 renderproto >= 0.11 xextproto
>   >= 7.2.99.901 inputproto >= 2.3 kbproto >= 1.0.3 fontsproto >= 2.1.3
>   pixman-1 >= 0.27.2 videoproto recordproto >= 1.13.99.1 resourceproto
>   >= 1.2.0 xf86driproto >= 2.1.0 glproto >= 1.4.17 dri >= 7.8.0 dri3proto
>   presentproto >= 1.1 xineramaproto xkbfile  pixman-1 >= 0.27.2 xfont2
>   >= 2.0.0 xau xshmfence >= 1.1 xdmcp) were not met:
>
> Package 'dri', required by 'virtual:world', not found
>
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> v2: no changes, included in new series switching xserver_xorg-server
>      to virtual opengl provider
>
>   package/mesa3d-headers/dri.pc            | 10 ++++++++++
>   package/mesa3d-headers/mesa3d-headers.mk |  9 ++-------
>   2 files changed, 12 insertions(+), 7 deletions(-)
>   create mode 100644 package/mesa3d-headers/dri.pc
>
> diff --git a/package/mesa3d-headers/dri.pc b/package/mesa3d-headers/dri.pc
> new file mode 100644
> index 0000000000..bd9a33b6c6
> --- /dev/null
> +++ b/package/mesa3d-headers/dri.pc
> @@ -0,0 +1,10 @@
> +prefix=/usr
> +libdir=${prefix}/lib
> +includedir=${prefix}/include
> +
> +dridriverdir=/usr/lib/dri
> +
> +Name: dri
> +Description: Direct Rendering Infrastructure
> +Version: @VERSION@
> +Cflags: -I${includedir}
> diff --git a/package/mesa3d-headers/mesa3d-headers.mk b/package/mesa3d-headers/mesa3d-headers.mk
> index 8670184742..99d6e52498 100644
> --- a/package/mesa3d-headers/mesa3d-headers.mk
> +++ b/package/mesa3d-headers/mesa3d-headers.mk
> @@ -34,13 +34,8 @@ ifeq ($(BR2_PACKAGE_XORG7),y)
>   # Not using $(SED) because we do not want to work in-place, and $(SED)
>   # contains -i.
>   define MESA3D_HEADERS_BUILD_DRI_PC
> -	sed -e 's:@\(exec_\)\?prefix@:/usr:' \
> -	    -e 's:@libdir@:$${exec_prefix}/lib:' \
> -	    -e 's:@includedir@:$${prefix}/include:' \
> -	    -e 's:@DRI_DRIVER_INSTALL_DIR@:$${libdir}/dri:' \
> -	    -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
> -	    -e 's:@DRI_PC_REQ_PRIV@::' \
> -	    $(@D)/src/mesa/drivers/dri/dri.pc.in \
> +	sed -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
> +	    package/mesa3d-headers/dri.pc \
>   	    >$(@D)/src/mesa/drivers/dri/dri.pc
>   endef
>   

-- 
Vincent Fazio
Embedded Software Engineer - Linux
Extreme Engineering Solutions, Inc
http://www.xes-inc.com

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Thomas Petazzoni]

Hello Bernd,

On Wed, 23 Oct 2019 08:22:19 +0200
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:

> Due to the switch of mesa3d to the meson build system the upstream
> tarball does not include dri.pc.in anymore, it is created during the
> build now:
> https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93
> 
> In order to provide dri.pc we need to add a version taken from a dri-
> based mesa3d build to our source repository, by doing so we can reduce
> the number of sed commands.

I am sorry, but I don't understand the reasoning behind this patch. If
the new meson build system produces a dri.pc file, why don't we use it?
What is wrong with it?

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Bernd Kuhls]

[posted and mailed]

Thomas Petazzoni <thomas.petazzoni@bootlin.com> 
wrote in news:20200425225823.2f68e8ef at windsurf.home:

> Hello Bernd,
> 
> On Wed, 23 Oct 2019 08:22:19 +0200
> Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> 
>> Due to the switch of mesa3d to the meson build system the upstream
>> tarball does not include dri.pc.in anymore, it is created during the
>> build now:
>> 
https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?
h=19.2#n93
>> 
>> In order to provide dri.pc we need to add a version taken from a dri-
>> based mesa3d build to our source repository, by doing so we can reduce
>> the number of sed commands.
> 
> I am sorry, but I don't understand the reasoning behind this patch. If
> the new meson build system produces a dri.pc file, why don't we use it?
> What is wrong with it?

Hi Thomas,

the mesa3d-headers package does not use the mesa3d meson build system, it 
uses the buildroot generic-package infrastructure and copies files from the 
mesa3d source tarball, which is lacking dri.pc.in now, to staging dir.

Regards, Bernd

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Yann E. MORIN]

On 2020-04-25 22:58 +0200, Thomas Petazzoni spake thusly:
> Hello Bernd,
> 
> On Wed, 23 Oct 2019 08:22:19 +0200
> Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> 
> > Due to the switch of mesa3d to the meson build system the upstream
> > tarball does not include dri.pc.in anymore, it is created during the
> > build now:
> > https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93
> > 
> > In order to provide dri.pc we need to add a version taken from a dri-
> > based mesa3d build to our source repository, by doing so we can reduce
> > the number of sed commands.
> 
> I am sorry, but I don't understand the reasoning behind this patch. If
> the new meson build system produces a dri.pc file, why don't we use it?
> What is wrong with it?

Because mesa3d-headers is not a meson package, but a generic package: it
only installs a select set of headers, and dri.pc when xorg is enabled.

So, the dri.pc generated by the meson buildsystem, is not generated.

Since there is now no template anymore in the source tree, we need to
get one from wmewhere...

Regards,
Yann E. MORIN.

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

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Yann E. MORIN]

Bernd, All,

On 2019-10-23 08:22 +0200, Bernd Kuhls spake thusly:
> Due to the switch of mesa3d to the meson build system the upstream
> tarball does not include dri.pc.in anymore, it is created during the
> build now:
> https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93
> 
> In order to provide dri.pc we need to add a version taken from a dri-
> based mesa3d build to our source repository, by doing so we can reduce
> the number of sed commands.
> 
> Without this patch the build of xserver_xorg-server, switched to virtual
> opengl provider, will fail:
> 
> checking for XSERVERCFLAGS... no
> configure: error: Package requirements (fixesproto >= 5.0 damageproto
>  >= 1.1 xcmiscproto >= 1.2.0 xtrans >= 1.3.5 bigreqsproto >= 1.1.0
>  xproto >= 7.0.31 randrproto >= 1.6.0 renderproto >= 0.11 xextproto
>  >= 7.2.99.901 inputproto >= 2.3 kbproto >= 1.0.3 fontsproto >= 2.1.3
>  pixman-1 >= 0.27.2 videoproto recordproto >= 1.13.99.1 resourceproto
>  >= 1.2.0 xf86driproto >= 2.1.0 glproto >= 1.4.17 dri >= 7.8.0 dri3proto
>  presentproto >= 1.1 xineramaproto xkbfile  pixman-1 >= 0.27.2 xfont2
>  >= 2.0.0 xau xshmfence >= 1.1 xdmcp) were not met:
> 
> Package 'dri', required by 'virtual:world', not found
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

I have entirely reworked the commit log to be more explicit about the
meson buildsystem referecne that is confusing.

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
> v2: no changes, included in new series switching xserver_xorg-server
>     to virtual opengl provider
> 
>  package/mesa3d-headers/dri.pc            | 10 ++++++++++
>  package/mesa3d-headers/mesa3d-headers.mk |  9 ++-------
>  2 files changed, 12 insertions(+), 7 deletions(-)
>  create mode 100644 package/mesa3d-headers/dri.pc
> 
> diff --git a/package/mesa3d-headers/dri.pc b/package/mesa3d-headers/dri.pc
> new file mode 100644
> index 0000000000..bd9a33b6c6
> --- /dev/null
> +++ b/package/mesa3d-headers/dri.pc
> @@ -0,0 +1,10 @@
> +prefix=/usr
> +libdir=${prefix}/lib
> +includedir=${prefix}/include
> +
> +dridriverdir=/usr/lib/dri
> +
> +Name: dri
> +Description: Direct Rendering Infrastructure
> +Version: @VERSION@
> +Cflags: -I${includedir}
> diff --git a/package/mesa3d-headers/mesa3d-headers.mk b/package/mesa3d-headers/mesa3d-headers.mk
> index 8670184742..99d6e52498 100644
> --- a/package/mesa3d-headers/mesa3d-headers.mk
> +++ b/package/mesa3d-headers/mesa3d-headers.mk
> @@ -34,13 +34,8 @@ ifeq ($(BR2_PACKAGE_XORG7),y)
>  # Not using $(SED) because we do not want to work in-place, and $(SED)
>  # contains -i.
>  define MESA3D_HEADERS_BUILD_DRI_PC
> -	sed -e 's:@\(exec_\)\?prefix@:/usr:' \
> -	    -e 's:@libdir@:$${exec_prefix}/lib:' \
> -	    -e 's:@includedir@:$${prefix}/include:' \
> -	    -e 's:@DRI_DRIVER_INSTALL_DIR@:$${libdir}/dri:' \
> -	    -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
> -	    -e 's:@DRI_PC_REQ_PRIV@::' \
> -	    $(@D)/src/mesa/drivers/dri/dri.pc.in \
> +	sed -e 's:@VERSION@:$(MESA3D_HEADERS_VERSION):' \
> +	    package/mesa3d-headers/dri.pc \
>  	    >$(@D)/src/mesa/drivers/dri/dri.pc
>  endef
>  
> -- 
> 2.20.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

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

[Buildroot] [PATCH v2 2/3] package/mesa3d-headers: fix install of dri.pc

[Peter Korsgaard]

>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 > Bernd, All,
 > On 2019-10-23 08:22 +0200, Bernd Kuhls spake thusly:
 >> Due to the switch of mesa3d to the meson build system the upstream
 >> tarball does not include dri.pc.in anymore, it is created during the
 >> build now:
 >> https://cgit.freedesktop.org/mesa/mesa/tree/src/mesa/drivers/dri/meson.build?h=19.2#n93
 >> 
 >> In order to provide dri.pc we need to add a version taken from a dri-
 >> based mesa3d build to our source repository, by doing so we can reduce
 >> the number of sed commands.
 >> 
 >> Without this patch the build of xserver_xorg-server, switched to virtual
 >> opengl provider, will fail:
 >> 
 >> checking for XSERVERCFLAGS... no
 >> configure: error: Package requirements (fixesproto >= 5.0 damageproto
 >> >= 1.1 xcmiscproto >= 1.2.0 xtrans >= 1.3.5 bigreqsproto >= 1.1.0
 >> xproto >= 7.0.31 randrproto >= 1.6.0 renderproto >= 0.11 xextproto
 >> >= 7.2.99.901 inputproto >= 2.3 kbproto >= 1.0.3 fontsproto >= 2.1.3
 >> pixman-1 >= 0.27.2 videoproto recordproto >= 1.13.99.1 resourceproto
 >> >= 1.2.0 xf86driproto >= 2.1.0 glproto >= 1.4.17 dri >= 7.8.0 dri3proto
 >> presentproto >= 1.1 xineramaproto xkbfile  pixman-1 >= 0.27.2 xfont2
 >> >= 2.0.0 xau xshmfence >= 1.1 xdmcp) were not met:
 >> 
 >> Package 'dri', required by 'virtual:world', not found
 >> 
 >> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>

 > I have entirely reworked the commit log to be more explicit about the
 > meson buildsystem referecne that is confusing.

 > Applied to master, thanks.

Committed to 2020.02.x, thanks.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 3/3] package/x11r7/xserver_xorg-server: switch to virtual opengl provider

[Bernd Kuhls]

This allows to use xserver_xorg-server without mesa3d.

Build-tested using this defconfig:

BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
BR2_TOOLCHAIN_BUILDROOT_CXX=y
BR2_PACKAGE_XORG7=y
BR2_PACKAGE_XSERVER_XORG_SERVER=y
BR2_PACKAGE_NVIDIA_DRIVER=y

Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
index 41c82ad930..8ccc04224d 100644
--- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
+++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
@@ -120,9 +120,9 @@ else # modular
 XSERVER_XORG_SERVER_CONF_OPTS += --disable-kdrive --disable-xfbdev
 endif
 
-ifeq ($(BR2_PACKAGE_MESA3D_OPENGL_GLX),y)
+ifeq ($(BR2_PACKAGE_HAS_LIBGL),y)
 XSERVER_XORG_SERVER_CONF_OPTS += --enable-dri --enable-glx
-XSERVER_XORG_SERVER_DEPENDENCIES += mesa3d
+XSERVER_XORG_SERVER_DEPENDENCIES += libgl
 else
 XSERVER_XORG_SERVER_CONF_OPTS += --disable-dri --disable-glx
 endif
-- 
2.20.1

[Buildroot] [PATCH 3/3] package/x11r7/xserver_xorg-server: switch to virtual opengl provider

[Thomas Petazzoni]

On Wed, 23 Oct 2019 08:22:20 +0200
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:

> This allows to use xserver_xorg-server without mesa3d.
> 
> Build-tested using this defconfig:
> 
> BR2_TOOLCHAIN_BUILDROOT_GLIBC=y
> BR2_TOOLCHAIN_BUILDROOT_CXX=y
> BR2_PACKAGE_XORG7=y
> BR2_PACKAGE_XSERVER_XORG_SERVER=y
> BR2_PACKAGE_NVIDIA_DRIVER=y
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[Buildroot] [PATCH 3/3] package/x11r7/xserver_xorg-server: switch to virtual opengl provider

[Bernd Kuhls]

Am Fri, 25 Oct 2019 20:10:55 +0200 schrieb Thomas Petazzoni:

> On Wed, 23 Oct 2019 08:22:20 +0200 Bernd Kuhls
> <bernd.kuhls@t-online.de> wrote:
> 
>> This allows to use xserver_xorg-server without mesa3d.
>> 
>> Build-tested using this defconfig:
>> 
>> BR2_TOOLCHAIN_BUILDROOT_GLIBC=y BR2_TOOLCHAIN_BUILDROOT_CXX=y
>> BR2_PACKAGE_XORG7=y BR2_PACKAGE_XSERVER_XORG_SERVER=y
>> BR2_PACKAGE_NVIDIA_DRIVER=y
>> 
>> Signed-off-by: Bernd Kuhls
>> <bernd.kuhls@t-online.de>
>> ---
>>  package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> Applied to master, thanks.
> 
> Thomas

Hi Thomas,

you applied patches 1 & 3 of this series but did not commit patch 2:
http://patchwork.ozlabs.org/patch/1181847/

This will cause a build error with xserver_xorg-server as noted in the 
commit message of patch 2.

Regards, Bernd

[Buildroot] [PATCH 1/3] package/x11r7/xserver_xorg-server: fix optional glx dependency

[Thomas Petazzoni]

On Wed, 23 Oct 2019 08:22:18 +0200
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:

> Typo was added by commit
> https://git.buildroot.net/buildroot/commit/?id=c2856aa2857d616788684a053f1c886da00d977f
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com