From c677925dc15a1f977e24d30a45d1b9b44289b880 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= Date: Thu, 13 Sep 2018 02:53:09 +0200 Subject: run_projects_tests: Fail if we install extra files even if not in cl As per commit 2340fd3, unexpected installed files are not reported anymore when using compilers other than 'cl', this regression was introduced in the attempt of not reporting extra .pdb files, but actually caused any non extra .pdb file in other compilers to be ignored. Fix boolean test, by reporting any extra file a part '.pdb' ones under non 'cl' compiler, while anyone under 'cl'. --- run_project_tests.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'run_project_tests.py') diff --git a/run_project_tests.py b/run_project_tests.py index a373aa0..841f072 100755 --- a/run_project_tests.py +++ b/run_project_tests.py @@ -193,8 +193,9 @@ def validate_install(srcdir, installdir, compiler, env): # Windows-specific tests check for the existence of installed PDB # files, but common tests do not, for obvious reasons. Ignore any # extra PDB files found. - if fname not in expected and not fname.endswith('.pdb') and compiler == 'cl': - ret_msg += 'Extra file {0} found.\n'.format(fname) + if fname not in expected: + if not (fname.endswith('.pdb') and compiler != 'cl'): + ret_msg += 'Extra file {0} found.\n'.format(fname) return ret_msg def log_text_file(logfile, testdir, stdo, stde): -- cgit v1.1