* [PATCH] meson: move SPHINX_ARGS references within "if build_docs"
@ 2020-10-16 9:14 Paolo Bonzini
2020-10-16 9:25 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 2+ messages in thread
From: Paolo Bonzini @ 2020-10-16 9:14 UTC (permalink / raw)
To: qemu-devel; +Cc: luoyonggang
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
tests/qapi-schema/meson.build | 88 +++++++++++++++++------------------
1 file changed, 44 insertions(+), 44 deletions(-)
diff --git a/tests/qapi-schema/meson.build b/tests/qapi-schema/meson.build
index 1f222a7a13..66c7f04cf8 100644
--- a/tests/qapi-schema/meson.build
+++ b/tests/qapi-schema/meson.build
@@ -219,53 +219,53 @@ qapi_doc = custom_target('QAPI doc',
'-p', 'doc-good-', '@INPUT0@' ],
depend_files: qapi_gen_depends)
-# Test the document-comment document generation code by running a test schema
-# file through Sphinx's plain-text builder and comparing the result against
-# a golden reference. This is in theory susceptible to failures if Sphinx
-# changes its output, but the text output has historically been very stable
-# (no changes between Sphinx 1.6 and 3.0), so it is a better bet than
-# texinfo or HTML generation, both of which have had changes. We might
-# need to add more sophisticated logic here in future for some sort of
-# fuzzy comparison if future Sphinx versions produce different text,
-# but for now the simple comparison suffices.
-qapi_doc_out = custom_target('QAPI rST doc',
- output: ['doc-good.txt'],
- input: files('doc-good.json', 'doc-good.rst'),
- build_by_default: build_docs,
- depend_files: sphinx_extn_depends,
- # We use -E to suppress Sphinx's caching, because
- # we want it to always really run the QAPI doc
- # generation code. It also means we don't
- # clutter up the build dir with the cache.
- command: [SPHINX_ARGS,
- '-b', 'text', '-E',
- '-c', meson.source_root() / 'docs',
- '-D', 'master_doc=doc-good',
- meson.current_source_dir(),
- meson.current_build_dir()])
+if build_docs
+ # Test the document-comment document generation code by running a test schema
+ # file through Sphinx's plain-text builder and comparing the result against
+ # a golden reference. This is in theory susceptible to failures if Sphinx
+ # changes its output, but the text output has historically been very stable
+ # (no changes between Sphinx 1.6 and 3.0), so it is a better bet than
+ # texinfo or HTML generation, both of which have had changes. We might
+ # need to add more sophisticated logic here in future for some sort of
+ # fuzzy comparison if future Sphinx versions produce different text,
+ # but for now the simple comparison suffices.
+ qapi_doc_out = custom_target('QAPI rST doc',
+ output: ['doc-good.txt'],
+ input: files('doc-good.json', 'doc-good.rst'),
+ build_by_default: true,
+ depend_files: sphinx_extn_depends,
+ # We use -E to suppress Sphinx's caching, because
+ # we want it to always really run the QAPI doc
+ # generation code. It also means we don't
+ # clutter up the build dir with the cache.
+ command: [SPHINX_ARGS,
+ '-b', 'text', '-E',
+ '-c', meson.source_root() / 'docs',
+ '-D', 'master_doc=doc-good',
+ meson.current_source_dir(),
+ meson.current_build_dir()])
-# Fix possible inconsistency in line endings in generated output and
-# in the golden reference (which could otherwise cause test failures
-# on Windows hosts). Unfortunately diff --strip-trailing-cr
-# is GNU-diff only. The odd-looking perl is because we must avoid
-# using an explicit '\' character in the command arguments to
-# a custom_target(), as Meson will unhelpfully replace it with a '/'
-# (https://github.com/mesonbuild/meson/issues/1564)
-qapi_doc_out_nocr = custom_target('QAPI rST doc newline-sanitized',
- output: ['doc-good.txt.nocr'],
- input: qapi_doc_out[0],
- build_by_default: build_docs,
- command: ['perl', '-pe', '$x = chr 13; s/$x$//', '@INPUT@'],
- capture: true)
+ # Fix possible inconsistency in line endings in generated output and
+ # in the golden reference (which could otherwise cause test failures
+ # on Windows hosts). Unfortunately diff --strip-trailing-cr
+ # is GNU-diff only. The odd-looking perl is because we must avoid
+ # using an explicit '\' character in the command arguments to
+ # a custom_target(), as Meson will unhelpfully replace it with a '/'
+ # (https://github.com/mesonbuild/meson/issues/1564)
+ qapi_doc_out_nocr = custom_target('QAPI rST doc newline-sanitized',
+ output: ['doc-good.txt.nocr'],
+ input: qapi_doc_out[0],
+ build_by_default: true,
+ command: ['perl', '-pe', '$x = chr 13; s/$x$//', '@INPUT@'],
+ capture: true)
-qapi_doc_ref_nocr = custom_target('QAPI rST doc reference newline-sanitized',
- output: ['doc-good.ref.nocr'],
- input: files('doc-good.txt'),
- build_by_default: build_docs,
- command: ['perl', '-pe', '$x = chr 13; s/$x$//', '@INPUT@'],
- capture: true)
+ qapi_doc_ref_nocr = custom_target('QAPI rST doc reference newline-sanitized',
+ output: ['doc-good.ref.nocr'],
+ input: files('doc-good.txt'),
+ build_by_default: true,
+ command: ['perl', '-pe', '$x = chr 13; s/$x$//', '@INPUT@'],
+ capture: true)
-if build_docs
# "full_path()" needed here to work around
# https://github.com/mesonbuild/meson/issues/7585
test('QAPI rST doc', diff, args: ['-u', qapi_doc_ref_nocr[0].full_path(),
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] meson: move SPHINX_ARGS references within "if build_docs"
2020-10-16 9:14 [PATCH] meson: move SPHINX_ARGS references within "if build_docs" Paolo Bonzini
@ 2020-10-16 9:25 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 2+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-10-16 9:25 UTC (permalink / raw)
To: Paolo Bonzini; +Cc: Yonggang Luo, qemu-devel@nongnu.org Developers
[-- Attachment #1: Type: text/plain, Size: 7210 bytes --]
Le ven. 16 oct. 2020 11:15, Paolo Bonzini <pbonzini@redhat.com> a écrit :
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
> tests/qapi-schema/meson.build | 88 +++++++++++++++++------------------
> 1 file changed, 44 insertions(+), 44 deletions(-)
>
> diff --git a/tests/qapi-schema/meson.build b/tests/qapi-schema/meson.build
> index 1f222a7a13..66c7f04cf8 100644
> --- a/tests/qapi-schema/meson.build
> +++ b/tests/qapi-schema/meson.build
> @@ -219,53 +219,53 @@ qapi_doc = custom_target('QAPI doc',
> '-p', 'doc-good-', '@INPUT0@' ],
> depend_files: qapi_gen_depends)
>
> -# Test the document-comment document generation code by running a test
> schema
> -# file through Sphinx's plain-text builder and comparing the result
> against
> -# a golden reference. This is in theory susceptible to failures if Sphinx
> -# changes its output, but the text output has historically been very
> stable
> -# (no changes between Sphinx 1.6 and 3.0), so it is a better bet than
> -# texinfo or HTML generation, both of which have had changes. We might
> -# need to add more sophisticated logic here in future for some sort of
> -# fuzzy comparison if future Sphinx versions produce different text,
> -# but for now the simple comparison suffices.
> -qapi_doc_out = custom_target('QAPI rST doc',
> - output: ['doc-good.txt'],
> - input: files('doc-good.json',
> 'doc-good.rst'),
> - build_by_default: build_docs,
> - depend_files: sphinx_extn_depends,
> - # We use -E to suppress Sphinx's caching,
> because
> - # we want it to always really run the QAPI
> doc
> - # generation code. It also means we don't
> - # clutter up the build dir with the cache.
> - command: [SPHINX_ARGS,
> - '-b', 'text', '-E',
> - '-c', meson.source_root() / 'docs',
> - '-D', 'master_doc=doc-good',
> - meson.current_source_dir(),
> - meson.current_build_dir()])
> +if build_docs
> + # Test the document-comment document generation code by running a test
> schema
> + # file through Sphinx's plain-text builder and comparing the result
> against
> + # a golden reference. This is in theory susceptible to failures if
> Sphinx
> + # changes its output, but the text output has historically been very
> stable
> + # (no changes between Sphinx 1.6 and 3.0), so it is a better bet than
> + # texinfo or HTML generation, both of which have had changes. We might
> + # need to add more sophisticated logic here in future for some sort of
> + # fuzzy comparison if future Sphinx versions produce different text,
> + # but for now the simple comparison suffices.
> + qapi_doc_out = custom_target('QAPI rST doc',
> + output: ['doc-good.txt'],
> + input: files('doc-good.json',
> 'doc-good.rst'),
> + build_by_default: true,
> + depend_files: sphinx_extn_depends,
> + # We use -E to suppress Sphinx's caching,
> because
> + # we want it to always really run the QAPI
> doc
> + # generation code. It also means we don't
> + # clutter up the build dir with the cache.
> + command: [SPHINX_ARGS,
> + '-b', 'text', '-E',
> + '-c', meson.source_root() /
> 'docs',
> + '-D', 'master_doc=doc-good',
> + meson.current_source_dir(),
> + meson.current_build_dir()])
>
> -# Fix possible inconsistency in line endings in generated output and
> -# in the golden reference (which could otherwise cause test failures
> -# on Windows hosts). Unfortunately diff --strip-trailing-cr
> -# is GNU-diff only. The odd-looking perl is because we must avoid
> -# using an explicit '\' character in the command arguments to
> -# a custom_target(), as Meson will unhelpfully replace it with a '/'
> -# (https://github.com/mesonbuild/meson/issues/1564)
> -qapi_doc_out_nocr = custom_target('QAPI rST doc newline-sanitized',
> - output: ['doc-good.txt.nocr'],
> - input: qapi_doc_out[0],
> - build_by_default: build_docs,
> - command: ['perl', '-pe', '$x = chr 13;
> s/$x$//', '@INPUT@'],
> - capture: true)
> + # Fix possible inconsistency in line endings in generated output and
> + # in the golden reference (which could otherwise cause test failures
> + # on Windows hosts). Unfortunately diff --strip-trailing-cr
> + # is GNU-diff only. The odd-looking perl is because we must avoid
> + # using an explicit '\' character in the command arguments to
> + # a custom_target(), as Meson will unhelpfully replace it with a '/'
> + # (https://github.com/mesonbuild/meson/issues/1564)
> + qapi_doc_out_nocr = custom_target('QAPI rST doc newline-sanitized',
> + output: ['doc-good.txt.nocr'],
> + input: qapi_doc_out[0],
> + build_by_default: true,
> + command: ['perl', '-pe', '$x = chr
> 13; s/$x$//', '@INPUT@'],
> + capture: true)
>
> -qapi_doc_ref_nocr = custom_target('QAPI rST doc reference
> newline-sanitized',
> - output: ['doc-good.ref.nocr'],
> - input: files('doc-good.txt'),
> - build_by_default: build_docs,
> - command: ['perl', '-pe', '$x = chr 13;
> s/$x$//', '@INPUT@'],
> - capture: true)
> + qapi_doc_ref_nocr = custom_target('QAPI rST doc reference
> newline-sanitized',
> + output: ['doc-good.ref.nocr'],
> + input: files('doc-good.txt'),
> + build_by_default: true,
> + command: ['perl', '-pe', '$x = chr
> 13; s/$x$//', '@INPUT@'],
> + capture: true)
>
> -if build_docs
> # "full_path()" needed here to work around
> # https://github.com/mesonbuild/meson/issues/7585
> test('QAPI rST doc', diff, args: ['-u',
> qapi_doc_ref_nocr[0].full_path(),
> --
> 2.26.2
>
>
>
[-- Attachment #2: Type: text/html, Size: 9718 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-10-16 9:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-16 9:14 [PATCH] meson: move SPHINX_ARGS references within "if build_docs" Paolo Bonzini
2020-10-16 9:25 ` Philippe Mathieu-Daudé
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.