diff options
author | Joel Klinghed <the_jk@spawned.biz> | 2018-02-27 21:41:48 +0100 |
---|---|---|
committer | Joel Klinghed <the_jk@spawned.biz> | 2018-02-27 21:48:42 +0100 |
commit | da017702613d718ac69ae213ee91358566f7b622 (patch) | |
tree | c414e7c2a6454d37e16daa0f8f2d9a2dbe8b9beb /mesonbuild/environment.py | |
parent | 16f80b4c506bba9839e30bd98fcbbafddcef3632 (diff) | |
download | meson-da017702613d718ac69ae213ee91358566f7b622.zip meson-da017702613d718ac69ae213ee91358566f7b622.tar.gz meson-da017702613d718ac69ae213ee91358566f7b622.tar.bz2 |
Fix coverage-xml and coverage-text targets for gcovr >= 3.1
In gcovr 3.1 the -r/--rootdir argument changed meaning causing
reports generated with gcovr 3.1 to not find the source files
and look for *.gcda in the whole source tree rather than the
build dir.
So, detect gcovr version and if 3.1 give build_root to -r instead
of source_root.
Diffstat (limited to 'mesonbuild/environment.py')
-rw-r--r-- | mesonbuild/environment.py | 20 |
1 files changed, 16 insertions, 4 deletions
diff --git a/mesonbuild/environment.py b/mesonbuild/environment.py index ccd85d5..099a3b3 100644 --- a/mesonbuild/environment.py +++ b/mesonbuild/environment.py @@ -77,19 +77,31 @@ cflags_mapping = {'c': 'CFLAGS', 'd': 'DFLAGS', 'vala': 'VALAFLAGS'} +def detect_gcovr(version='3.1', log=False): + gcovr_exe = 'gcovr' + try: + p, found = Popen_safe([gcovr_exe, '--version'])[0:2] + except (FileNotFoundError, PermissionError): + # Doesn't exist in PATH or isn't executable + return + found = search_version(found) + if p.returncode == 0: + if log: + mlog.log('Found gcovr-{} at {}'.format(found, shlex.quote(shutil.which(gcovr_exe)))) + return gcovr_exe, mesonlib.version_compare(found, '>=' + version) def find_coverage_tools(): - gcovr_exe = 'gcovr' + gcovr_exe, gcovr_3_1 = detect_gcovr() + lcov_exe = 'lcov' genhtml_exe = 'genhtml' - if not mesonlib.exe_exists([gcovr_exe, '--version']): - gcovr_exe = None if not mesonlib.exe_exists([lcov_exe, '--version']): lcov_exe = None if not mesonlib.exe_exists([genhtml_exe, '--version']): genhtml_exe = None - return gcovr_exe, lcov_exe, genhtml_exe + + return gcovr_exe, gcovr_3_1, lcov_exe, genhtml_exe def detect_ninja(version='1.5', log=False): for n in ['ninja', 'ninja-build']: |