[xserver] build: Bump Mesa requirement to 17.1

Submitted by Adam Jackson on March 14, 2018, 6:43 p.m.

Details

Message ID 20180314184336.3725-1-ajax@redhat.com
State New
Headers show
Series "build: Bump Mesa requirement to 17.1" ( rev: 1 ) in X.org

Not browsing as part of any series.

Commit Message

Adam Jackson March 14, 2018, 6:43 p.m.
gbm_bo_get_modifier is new in 17.1, which is 10 months old and two
stable branches ago.

Signed-off-by: Adam Jackson <ajax@redhat.com>
---
 configure.ac    | 2 +-
 glx/meson.build | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/configure.ac b/configure.ac
index f82c0a66a..18cfda69e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1113,7 +1113,7 @@  case "$DRI2,$HAVE_DRI2PROTO" in
 	yes,yes | auto,yes)
 		AC_DEFINE(DRI2, 1, [Build DRI2 extension])
 		DRI2=yes
-		LIBGL="gl >= 9.2.0"
+		LIBGL="gl >= 17.1"
 		SDK_REQUIRED_MODULES="$SDK_REQUIRED_MODULES $DRI2PROTO"
 		;;
 esac
diff --git a/glx/meson.build b/glx/meson.build
index 5f93a75a5..3b0618f67 100644
--- a/glx/meson.build
+++ b/glx/meson.build
@@ -39,7 +39,7 @@  if build_glx
             common_dep,
             dl_dep,
             dependency('glproto', version: '>= 1.4.17'),
-            dependency('gl', version: '>= 9.2.0'),
+            dependency('gl', version: '>= 17.1.0'),
         ],
         c_args: [
             glx_align64,
@@ -70,7 +70,7 @@  if build_glx
             common_dep,
             dl_dep,
             dependency('glproto', version: '>= 1.4.17'),
-            dependency('gl', version: '>= 9.2.0'),
+            dependency('gl', version: '>= 17.1.0'),
         ],
     )
 endif

Comments

On 2018-03-14 07:43 PM, Adam Jackson wrote:
> gbm_bo_get_modifier is new in 17.1, which is 10 months old and two
> stable branches ago.
> 
> Signed-off-by: Adam Jackson <ajax@redhat.com>
> ---
>  configure.ac    | 2 +-
>  glx/meson.build | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index f82c0a66a..18cfda69e 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1113,7 +1113,7 @@ case "$DRI2,$HAVE_DRI2PROTO" in
>  	yes,yes | auto,yes)
>  		AC_DEFINE(DRI2, 1, [Build DRI2 extension])
>  		DRI2=yes
> -		LIBGL="gl >= 9.2.0"
> +		LIBGL="gl >= 17.1"
>  		SDK_REQUIRED_MODULES="$SDK_REQUIRED_MODULES $DRI2PROTO"
>  		;;
>  esac
> diff --git a/glx/meson.build b/glx/meson.build
> index 5f93a75a5..3b0618f67 100644
> --- a/glx/meson.build
> +++ b/glx/meson.build
> @@ -39,7 +39,7 @@ if build_glx
>              common_dep,
>              dl_dep,
>              dependency('glproto', version: '>= 1.4.17'),
> -            dependency('gl', version: '>= 9.2.0'),
> +            dependency('gl', version: '>= 17.1.0'),
>          ],
>          c_args: [
>              glx_align64,
> @@ -70,7 +70,7 @@ if build_glx
>              common_dep,
>              dl_dep,
>              dependency('glproto', version: '>= 1.4.17'),
> -            dependency('gl', version: '>= 9.2.0'),
> +            dependency('gl', version: '>= 17.1.0'),
>          ],
>      )
>  endif
> 

Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
On 14 March 2018 at 18:43, Adam Jackson <ajax@redhat.com> wrote:
> gbm_bo_get_modifier is new in 17.1, which is 10 months old and two
> stable branches ago.
>
> Signed-off-by: Adam Jackson <ajax@redhat.com>
> ---
>  configure.ac    | 2 +-
>  glx/meson.build | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index f82c0a66a..18cfda69e 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1113,7 +1113,7 @@ case "$DRI2,$HAVE_DRI2PROTO" in
>         yes,yes | auto,yes)
>                 AC_DEFINE(DRI2, 1, [Build DRI2 extension])
>                 DRI2=yes
> -               LIBGL="gl >= 9.2.0"
> +               LIBGL="gl >= 17.1"
If the concern is gbm_bo_get_modifier then the LIBGBM version should
be bumped, right?
Then one can drop all the GBM_BO_WITH_MODIFIERS hunks, analogous to my
earlier series [1]

-Emil

[1] https://patchwork.freedesktop.org/series/39553/