package/mesa3d: bump to version 18.3.1
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
From: James Hilliard <[email protected]>
--- ...d-missing-include-stddef.h-in-egldevice.h.patch | 45 ++++++++++++++++++++++ package/mesa3d/mesa3d.hash | 10 ++--- package/mesa3d/mesa3d.mk | 2 +- 3 files changed, 51 insertions(+), 6 deletions(-) create mode 100644 package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch
package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch new file +45 -0
@@ -0,0 +1,45 @@ +From eb44c36cf1729e7e200b77cf8ea755dff72d1639 Mon Sep 17 00:00:00 2001 +From: Gurchetan Singh <[email protected]> +Date: Wed, 28 Nov 2018 08:39:34 -0800 +Subject: [PATCH] egl: add missing #include <stddef.h> in egldevice.h +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Otherwise, I get this error: + +main/egldevice.h:54:13: error: ?NULL? undeclared (first use in this function) + dev = NULL; + ^~~~ +with this config: + +./autogen.sh --enable-gles1 --enable-gles2 --with-platforms='surfaceless' --disable-glx + --with-dri-drivers="i965" --with-gallium-drivers="" --enable-gbm + +v3: Use stddef.h (Matt) +v4: Modify commit message (Eric) + +Reviewed-by: Matt Turner <[email protected]> +Reviewed-by: Eric Engestrom <[email protected]> +--- + src/egl/main/egldevice.h | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/src/egl/main/egldevice.h b/src/egl/main/egldevice.h +index ddcdcd1..83a47d5 100644 +--- a/src/egl/main/egldevice.h ++++ b/src/egl/main/egldevice.h +@@ -31,9 +31,9 @@ + + + #include <stdbool.h> ++#include <stddef.h> + #include "egltypedefs.h" + +- + #ifdef __cplusplus + extern "C" { + #endif +-- +2.7.4 +
package/mesa3d/mesa3d.hash +5 -5
@@ -1,7 +1,7 @@ -# From https://lists.freedesktop.org/archives/mesa-announce/2018-December/000482.html -md5 2d69eff8fe0c6e89bb793d4fd69b750d mesa-18.3.0.tar.xz -sha1 d4ddc4e7aa8e11a41d35b5d51476f867e81056ca mesa-18.3.0.tar.xz -sha256 b63f947e735d6ef3dfaa30c789a9adfbae18aea671191eaacde95a18c17fc38a mesa-18.3.0.tar.xz -sha512 6643d8a100c50efee7178fe950e7cccad24c3a98538d3a13c7c6570add30a56776b4e5f279e2b0a20c4038e682e9461ca5a4fd4ac23a7f60d2f4a0c972525c42 mesa-18.3.0.tar.xz +# From https://lists.freedesktop.org/archives/mesa-announce/2018-December/000484.html +md5 d60828056d77bfdbae0970f9b15fb1be mesa-18.3.1.tar.xz +sha1 50ba2d37647fea77ea19416e8a6ffed34c313330 mesa-18.3.1.tar.xz +sha256 5b1f827d28684a25f6657289f8b7d47ac56395988c7ac23e0ec9a62b644bdc63 mesa-18.3.1.tar.xz +sha512 a68d39158cf1e868d70730d0641a0cfe4c6e5b3cd1bc0c47f54022402aca03503933084f6ddc722bf88c9b6d1281ba5c847ec4fed8092a9b33f90527d08e12db mesa-18.3.1.tar.xz # License sha256 630e75b4fdeb75ee2bf9e55db54dd1e3ff7353d52d9314ca8512bfd460f8e24c docs/license.html
package/mesa3d/mesa3d.mk +1 -1
@@ -5,7 +5,7 @@ ################################################################################ # When updating the version, please also update mesa3d-headers -MESA3D_VERSION = 18.3.0 +MESA3D_VERSION = 18.3.1 MESA3D_SOURCE = mesa-$(MESA3D_VERSION).tar.xz MESA3D_SITE = https://mesa.freedesktop.org/archive MESA3D_LICENSE = MIT, SGI, Khronos
Signature · 1 line
2.7.4
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
Am Sat, 15 Dec 2018 16:10:38 +0800 schrieb
james.hilliard1-Re5JQEeQqe8AvxtiuMwx3w:
Quoted 3 lines
From: James Hilliard <[email protected]>
Signed-off-by James Hilliard <[email protected]>
Hi,
you need to update package/mesa3d-headers as well.
Regards, Bernd
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
In reply to james.hilliard1 at gmail.comJames, All,
On 2018-12-15 16:10 +0800, james.hilliard1 at gmail.com spake thusly:
Quoted 1 line
From: James Hilliard <[email protected]>
You need to provide a commit log that explains a bit the change, especially since it is not a simple bumnp (you add a new patch).
More comment below...
Quoted 2 lines
Signed-off-by James Hilliard <[email protected]>---
When you send an updated patch (a v2, v3...), please add a revision changelog here, see:
https://buildroot.org/downloads/manual/manual.html#_patch_revision_changelog
Quoted 25 lines
...d-missing-include-stddef.h-in-egldevice.h.patch | 45 ++++++++++++++++++++++ package/mesa3d/mesa3d.hash | 10 ++--- package/mesa3d/mesa3d.mk | 2 +- 3 files changed, 51 insertions(+), 6 deletions(-) create mode 100644 package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patchpackage/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch new file +13
-0@@ -0,0 +1,45 @@ +From eb44c36cf1729e7e200b77cf8ea755dff72d1639 Mon Sep 17 00:00:00 2001 +From: Gurchetan Singh <[email protected]> +Date: Wed, 28 Nov 2018 08:39:34 -0800 +Subject: [PATCH] egl: add missing #include <stddef.h> in egldevice.h +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Otherwise, I get this error: + +main/egldevice.h:54:13: error: ?NULL? undeclared (first use in this function) + dev = NULL; + ^~~~
package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch line 23
Quoted 10 lines
package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch new file +10
-0+with this config: + +./autogen.sh --enable-gles1 --enable-gles2 --with-platforms='surfaceless' --disable-glx + --with-dri-drivers="i965" --with-gallium-drivers="" --enable-gbm + +v3: Use stddef.h (Matt) +v4: Modify commit message (Eric) + +Reviewed-by: Matt Turner <[email protected]> +Reviewed-by: Eric Engestrom <[email protected]>
Here, you should also add your own Signed-off-by tag, and a note stating that this is backported from upstream or is grabbed from a ML thread or whatever, so that we know whether to remove it on next bump for example;
https://buildroot.org/downloads/manual/manual.html#_format_and_licensing_of_the_package_patches
https://buildroot.org/downloads/manual/manual.html#_integrating_patches_found_on_the_web
[--SNIP--]
package/mesa3d/mesa3d.mk line 8
Quoted 10 lines
package/mesa3d/mesa3d.mk +1
-1@@ -5,7 +5,7 @@ ################################################################################ # When updating the version, please also update mesa3d-headers -MESA3D_VERSION = 18.3.0 +MESA3D_VERSION = 18.3.1
As Bernd already pointed out, you need to also update mesa3d-headers too. What made you miss the notice just above? Do you have a suggestion so that the next one who bumps either mesa3d or mesa3d-headers does not miss it next time?
Regards,
Yann E. MORIN.
Quoted 10 lines
package/mesa3d/mesa3d.mk +0
-0MESA3D_SOURCE = mesa-$(MESA3D_VERSION).tar.xz MESA3D_SITE = https://mesa.freedesktop.org/archive MESA3D_LICENSE = MIT, SGI, KhronosSignature · 6 lines
2.7.4 _______________________________________________ buildroot mailing list buildroot at busybox.net http://lists.busybox.net/mailman/listinfo/buildroot
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
Quoted 50 lines · Yann E. MORIN
James, All,
On 2018-12-15 16:10 +0800, james.hilliard1 at gmail.com spake thusly:
Quoted 1 line
From: James Hilliard <[email protected]>
You need to provide a commit log that explains a bit the change, especially since it is not a simple bumnp (you add a new patch).
More comment below...
Quoted 2 lines
Signed-off-by James Hilliard <[email protected]>---
When you send an updated patch (a v2, v3...), please add a revision changelog here, see:
https://buildroot.org/downloads/manual/manual.html#_patch_revision_changelog
Quoted 31 lines
...d-missing-include-stddef.h-in-egldevice.h.patch | 45 ++++++++++++++++++++++ package/mesa3d/mesa3d.hash | 10 ++--- package/mesa3d/mesa3d.mk | 2 +- 3 files changed, 51 insertions(+), 6 deletions(-) create mode 100644 package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patchpackage/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch new file +19
-0@@ -0,0 +1,45 @@ +From eb44c36cf1729e7e200b77cf8ea755dff72d1639 Mon Sep 17 00:00:00 2001 +From: Gurchetan Singh <[email protected]> +Date: Wed, 28 Nov 2018 08:39:34 -0800 +Subject: [PATCH] egl: add missing #include <stddef.h> in egldevice.h +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Otherwise, I get this error: + +main/egldevice.h:54:13: error: ?NULL? undeclared (first use in this function) + dev = NULL; + ^~~~ +with this config: + +./autogen.sh --enable-gles1 --enable-gles2 --with-platforms='surfaceless' --disable-glx + --with-dri-drivers="i965" --with-gallium-drivers="" --enable-gbm + +v3: Use stddef.h (Matt)
Quoted 8 lines
package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch line 23
Quoted 4 lines
package/mesa3d/0004-egl-add-missing-include-stddef.h-in-egldevice.h.patch new file +4
-0+v4: Modify commit message (Eric) + +Reviewed-by: Matt Turner <[email protected]> +Reviewed-by: Eric Engestrom <[email protected]>Here, you should also add your own Signed-off-by tag, and a note stating that this is backported from upstream or is grabbed from a ML thread or whatever, so that we know whether to remove it on next bump for example;
will add for next version, it's a backport from upstream
Quoted 15 lines
[james.hilliard1 at gmail.com: backport from upstream commit]Signed-off-by James Hilliard <[email protected]>https://buildroot.org/downloads/manual/manual.html#_format_and_licensing_of_the_package_patches
https://buildroot.org/downloads/manual/manual.html#_integrating_patches_found_on_the_web[--SNIP--]
Quoted 7 lines
package/mesa3d/mesa3d.mk +0
-0@@ -5,7 +5,7 @@ ################################################################################
Quoted 8 lines
package/mesa3d/mesa3d.mk line 8
Quoted 3 lines
package/mesa3d/mesa3d.mk +1
-1# When updating the version, please also update mesa3d-headers -MESA3D_VERSION = 18.3.0 +MESA3D_VERSION = 18.3.1As Bernd already pointed out, you need to also update mesa3d-headers too. What made you miss the notice just above? Do you have a suggestion so that the next one who bumps either mesa3d or mesa3d-headers does not miss it next time?
It was 1am when I sent this and literally only looked for the version number to bump it when debugging a build failure. Wouldn't it make sense to have mesa3d and mesa3d-headers share the same package version so that only one needs to be bumped?
Quoted 22 lines
Regards,
Yann E. MORIN.Quoted 10 lines
package/mesa3d/mesa3d.mk +0
-1MESA3D_SOURCE = mesa-$(MESA3D_VERSION).tar.xz MESA3D_SITE = https://mesa.freedesktop.org/archive MESA3D_LICENSE = MIT, SGI, Khronos --2.7.4
_______________________________________________
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 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |'------------------------------^-------^------------------^--------------------'
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
James, All,
On 2018-12-15 03:56 -0700, James Hilliard spake thusly:
Quoted 1 line · Yann E. MORIN
[--SNIP--]
Quoted 15 lines
Quoted 7 lines
Quoted 7 lines
package/mesa3d/mesa3d.mk +0
-0@@ -5,7 +5,7 @@ ################################################################################Quoted 8 lines
package/mesa3d/mesa3d.mk line 8
Quoted 3 lines
package/mesa3d/mesa3d.mk +1
-1# When updating the version, please also update mesa3d-headers -MESA3D_VERSION = 18.3.0 +MESA3D_VERSION = 18.3.1As Bernd already pointed out, you need to also update mesa3d-headers too. What made you miss the notice just above? Do you have a suggestion so that the next one who bumps either mesa3d or mesa3d-headers does not miss it next time?
Quoted 1 line
It was 1am when I sent this
Aha! :-)
Quoted 4 lines
and literally only looked for the version
number to bump it when debugging a build failure. Wouldn't it make sense to have mesa3d and mesa3d-headers share the same package version so that only one needs to be bumped?
That's unfortunately not possible, see the comment in mesa3d-headers...
Regards,
Yann E. MORIN.
Quoted 21 lines
Quoted 21 lines
Regards,
Yann E. MORIN.Quoted 10 lines
package/mesa3d/mesa3d.mk +0
-1MESA3D_SOURCE = mesa-$(MESA3D_VERSION).tar.xz MESA3D_SITE = https://mesa.freedesktop.org/archive MESA3D_LICENSE = MIT, SGI, Khronos --2.7.4
_______________________________________________
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 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |'------------------------------^-------^------------------^--------------------'
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
Quoted 23 lines · Yann E. MORIN
James, All,
On 2018-12-15 03:56 -0700, James Hilliard spake thusly:
Quoted 1 line · Yann E. MORIN
[--SNIP--]
Quoted 15 lines
Quoted 7 lines
Quoted 7 lines
package/mesa3d/mesa3d.mk +0
-0@@ -5,7 +5,7 @@ ################################################################################Quoted 8 lines
package/mesa3d/mesa3d.mk line 8
Quoted 3 lines
package/mesa3d/mesa3d.mk +1
-1# When updating the version, please also update mesa3d-headers -MESA3D_VERSION = 18.3.0 +MESA3D_VERSION = 18.3.1As Bernd already pointed out, you need to also update mesa3d-headers too. What made you miss the notice just above? Do you have a suggestion so that the next one who bumps either mesa3d or mesa3d-headers does not miss it next time?
Quoted 1 line
It was 1am when I sent this
Quoted 8 lines
Aha! :-)
Quoted 4 lines
and literally only looked for the version
number to bump it when debugging a build failure. Wouldn't it make sense to have mesa3d and mesa3d-headers share the same package version so that only one needs to be bumped?That's unfortunately not possible, see the comment in mesa3d-headers...
I think you could turn mesa3d-headers into a virtual/fake package similar to how I did it with my systemd-boot patch. Then you wouldn't have to refer to variables in mesa3d.
Quoted 33 lines
Regards,
Yann E. MORIN.Quoted 14 lines
Quoted 14 lines
Regards,
Yann E. MORIN.Quoted 10 lines
package/mesa3d/mesa3d.mk +0
-1MESA3D_SOURCE = mesa-$(MESA3D_VERSION).tar.xz MESA3D_SITE = https://mesa.freedesktop.org/archive MESA3D_LICENSE = MIT, SGI, Khronos --2.7.4
_______________________________________________
buildroot mailing list
buildroot at busybox.net
http://lists.busybox.net/mailman/listinfo/buildrootQuoted 7 lines
Quoted 7 lines
--
.-----------------.--------------------.------------------.--------------------.| 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. |'------------------------------^-------^------------------^--------------------'
--
.-----------------.--------------------.------------------.--------------------.| 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. |'------------------------------^-------^------------------^--------------------'
PATCH v2 1/1 package/mesa3d: bump to version 18.3.1
James, All,
On 2018-12-15 04:15 -0700, James Hilliard spake thusly:
Quoted 9 lines · Yann E. MORIN
Quoted 5 lines
On 2018-12-15 03:56 -0700, James Hilliard spake thusly:
Quoted 3 lines
Wouldn't it make
sense to have mesa3d and mesa3d-headers share the same package version so that only one needs to be bumped?That's unfortunately not possible, see the comment in mesa3d-headers...
I think you could turn mesa3d-headers into a virtual/fake package similar to how I did it with my systemd-boot patch. Then you wouldn't have to refer to variables in mesa3d.
No, because that would make mesa3d more complex, for little benefit, see the commit log of 8138a36018 (package/mesa3d-headers: new package), in which there is an blurb about that.
Regards,
Yann E. MORIN.
Signature · 6 lines
.-----------------.--------------------.------------------.--------------------. | 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. | '------------------------------^-------^------------------^--------------------'