Mesa (master): Revert "egl: put full path to libEGL_mesa.so in GLVND json"

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Feb 6 00:41:42 UTC 2020


Module: Mesa
Branch: master
Commit: 76f300f2e498f101c58e182eac3ece11e527d297
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=76f300f2e498f101c58e182eac3ece11e527d297

Author: Eric Engestrom <eric at engestrom.ch>
Date:   Wed Feb  5 22:58:23 2020 +0000

Revert "egl: put full path to libEGL_mesa.so in GLVND json"

This reverts commit 0021f7dc307f4852955359adb5ac2b7667e6d4ac.

That commit had 2 issues:
- I missed the `.0` from the filename, causing issues on Debian & Ubuntu
  platforms.
- I didn't think about multilib/multi-arch systems, where we'd now need
  a separate json for each arch as they point to different libs.

Reverting this commit for now, I'll try again later.

Requested-by: Michel Dänzer <michel at daenzer.net>
Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2466
Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2471
Closes: https://gitlab.freedesktop.org/mesa/mesa/issues/2480
Signed-off-by: Eric Engestrom <eric at engestrom.ch>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3726>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3726>

---

 src/egl/glvnd_icd.py      | 24 ------------------------
 src/egl/main/50_mesa.json |  6 ++++++
 src/egl/meson.build       | 14 ++------------
 3 files changed, 8 insertions(+), 36 deletions(-)

diff --git a/src/egl/glvnd_icd.py b/src/egl/glvnd_icd.py
deleted file mode 100644
index 39850873a90..00000000000
--- a/src/egl/glvnd_icd.py
+++ /dev/null
@@ -1,24 +0,0 @@
-# Copyright 2019 Intel Corporation
-# SPDX-License-Identifier: MIT
-
-import argparse
-import json
-import os.path
-
-if __name__ == '__main__':
-    parser = argparse.ArgumentParser()
-    parser.add_argument('--out', help='Output json file.', required=True)
-    parser.add_argument('--lib-path', help='Path to libEGL_mesa.so', required=True)
-    args = parser.parse_args()
-
-    path = os.path.join(args.lib_path, 'libEGL_mesa.so')
-
-    json_data = {
-        'file_format_version': '1.0.0',
-        'ICD': {
-            'library_path': path,
-        },
-    }
-
-    with open(args.out, 'w') as f:
-        json.dump(json_data, f, indent=4, sort_keys=True, separators=(',', ': '))
diff --git a/src/egl/main/50_mesa.json b/src/egl/main/50_mesa.json
new file mode 100644
index 00000000000..8aaaa100ffa
--- /dev/null
+++ b/src/egl/main/50_mesa.json
@@ -0,0 +1,6 @@
+{
+    "file_format_version" : "1.0.0",
+    "ICD" : {
+        "library_path" : "libEGL_mesa.so.0"
+    }
+}
diff --git a/src/egl/meson.build b/src/egl/meson.build
index db34453ee9b..12d74ec37fa 100644
--- a/src/egl/meson.build
+++ b/src/egl/meson.build
@@ -152,18 +152,8 @@ else
   deps_for_egl += dep_glvnd
   files_egl += [g_egldispatchstubs_h, g_egldispatchstubs_c]
   files_egl += files('main/eglglvnd.c', 'main/egldispatchstubs.c')
-
-  egl_icd = custom_target(
-    'glvnd_icd',
-    input : 'glvnd_icd.py',
-    output : '50_mesa.json',
-    command : [
-      prog_python, '@INPUT@',
-      '--lib-path', join_paths(get_option('prefix'), get_option('libdir')),
-      '--out', '@OUTPUT@',
-    ],
-    build_by_default : true,
-    install : true,
+  install_data(
+    'main/50_mesa.json',
     install_dir : join_paths(get_option('datadir'), 'glvnd', 'egl_vendor.d')
   )
 endif



More information about the mesa-commit mailing list