[igt-dev] [PATCH i-g-t] build: rename Meson options

* [igt-dev] [PATCH i-g-t] build: rename Meson options
@ 2019-07-05 13:45 Simon Ser
  2019-07-05 14:29 ` [igt-dev] ✗ GitLab.Pipeline: warning for " Patchwork
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Simon Ser @ 2019-07-05 13:45 UTC (permalink / raw)
  To: igt-dev

Meson emits a warning because we use options beginning with "build_":

    DEPRECATION: Option uses prefix "build_", which is reserved for Meson. This will become an error in the future.

Rename our options so that we don't use the Meson-reserved prefix.

While at it, also make other build options and descriptions more consistent.

Signed-off-by: Simon Ser <simon.ser@intel.com>
---

Suggestions to further improve our build options are welcome.

 man/meson.build     |  2 +-
 meson.build         | 12 ++++++------
 meson_options.txt   | 30 +++++++++++++++---------------
 overlay/meson.build |  2 +-
 runner/meson.build  |  2 +-
 5 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/man/meson.build b/man/meson.build
index 2c1396af2738..2187e8c505c1 100644
--- a/man/meson.build
+++ b/man/meson.build
@@ -1,4 +1,4 @@
-build_man = get_option('build_man')
+build_man = get_option('man')

 manpages = [
 	'intel_aubdump',
diff --git a/meson.build b/meson.build
index f0cb2543ca64..5d32efd33933 100644
--- a/meson.build
+++ b/meson.build
@@ -77,10 +77,10 @@ foreach cc_arg : cc_args
   endif
 endforeach

-build_chamelium = get_option('build_chamelium')
-build_docs = get_option('build_docs')
-build_tests = not get_option('build_tests').disabled()
-with_libdrm = get_option('with_libdrm')
+build_chamelium = get_option('chamelium')
+build_docs = get_option('docs')
+build_tests = not get_option('tests').disabled()
+with_libdrm = get_option('libdrm_drivers')

 build_info = ['Build type: ' + get_option('buildtype')]

@@ -118,13 +118,13 @@ pciaccess = dependency('pciaccess', version : '>=0.10')
 libkmod = dependency('libkmod')
 libprocps = dependency('libprocps', required : true)

-libunwind = dependency('libunwind', required : get_option('with_libunwind'))
+libunwind = dependency('libunwind', required : get_option('libunwind'))
 build_info += 'With libunwind: @0@'.format(libunwind.found())

 libdw = dependency('libdw', required : true)
 pixman = dependency('pixman-1', required : true)

-valgrind = dependency('valgrind', required : get_option('with_valgrind'))
+valgrind = dependency('valgrind', required : get_option('valgrind'))
 if valgrind.found()
 	config.set('HAVE_VALGRIND', 1)
 endif
diff --git a/meson_options.txt b/meson_options.txt
index 9cca0c4f47eb..f980d3e005a6 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -1,6 +1,6 @@
-option('build_overlay',
+option('overlay',
        type : 'feature',
-       description : 'Build overlay')
+       description : 'Tools: overlay')

 option('overlay_backends',
        type : 'array',
@@ -8,39 +8,39 @@ option('overlay_backends',
        choices : [ 'auto', 'x', 'xv' ],
        description : 'Overlay backends to enable')

-option('build_chamelium',
+option('chamelium',
        type : 'feature',
-       description : 'Build chamelium test')
+       description : 'Tests: Chamelium')

-option('with_valgrind',
+option('valgrind',
        type : 'feature',
        description : 'Build with support for valgrind annotations')

-option('build_man',
+option('man',
        type : 'feature',
-       description : 'Build man pages')
+       description : 'Man pages')

-option('build_docs',
+option('docs',
        type : 'feature',
-       description : 'Build documentation')
+       description : 'Documentation')

-option('build_tests',
+option('tests',
        type : 'feature',
-       description : 'Build tests')
+       description : 'Tests')

-option('with_libdrm',
+option('libdrm_drivers',
        type : 'array',
        value : ['auto'],
        choices : ['', 'auto', 'intel', 'nouveau', 'amdgpu'],
        description : 'libdrm libraries to be used')

-option('with_libunwind',
+option('libunwind',
        type : 'feature',
        description : 'Use libunwind')

-option('build_runner',
+option('runner',
        type : 'feature',
-       description : 'Build test runner')
+       description : 'Test runner')

 option('use_rpath',
        type : 'boolean',
diff --git a/overlay/meson.build b/overlay/meson.build
index d2d2b16a8d1b..0a99076206c3 100644
--- a/overlay/meson.build
+++ b/overlay/meson.build
@@ -1,4 +1,4 @@
-build_overlay = get_option('build_overlay')
+build_overlay = get_option('overlay')
 overlay_backends = get_option('overlay_backends')

 gpu_overlay_src = [
diff --git a/runner/meson.build b/runner/meson.build
index 4eff193afa8d..86521f94d88a 100644
--- a/runner/meson.build
+++ b/runner/meson.build
@@ -1,4 +1,4 @@
-build_runner = get_option('build_runner')
+build_runner = get_option('runner')

 runnerlib_sources = [ 'settings.c',
 		      'job_list.c',
--
2.22.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply related	[flat|nested] 6+ messages in thread