Unset compiler envvars in unit tests.

Author: jpakkane

mesonbuild/environment.py

@@ -24,6 +24,16 @@
 
 build_filename = 'meson.build'
 
+# Environment variables that each lang uses.
+cflags_mapping = {'c': 'CFLAGS',
+                  'cpp': 'CXXFLAGS',
+                  'objc': 'OBJCFLAGS',
+                  'objcpp': 'OBJCXXFLAGS',
+                  'fortran': 'FFLAGS',
+                  'd': 'DFLAGS',
+                  'vala': 'VALAFLAGS'}
+
+
 def find_coverage_tools():
     gcovr_exe = 'gcovr'
     lcov_exe = 'lcov'
@@ -809,15 +819,6 @@ def log_var(var, val):
     if hasattr(compiler, 'get_linker_exelist'):
         compiler_is_linker = (compiler.get_exelist() == compiler.get_linker_exelist())
 
-    # Compile flags
-    cflags_mapping = {'c': 'CFLAGS',
-                      'cpp': 'CXXFLAGS',
-                      'objc': 'OBJCFLAGS',
-                      'objcpp': 'OBJCXXFLAGS',
-                      'fortran': 'FFLAGS',
-                      'd': 'DFLAGS',
-                      'vala': 'VALAFLAGS'}
-
     if lang not in cflags_mapping.keys():
         return [], [], []
 

run_unittests.py

@@ -1556,5 +1556,15 @@ def test_subdir(self):
         self.assertEqual(s2, self.read_contents('sub2/meson.build'))
 
 
+def unset_envs():
+    # For unit tests we must fully control all commend lines
+    # so that there are no unexpected changes coming from the
+    # environment, for example when doing a package build.
+    varnames = ['CPPFLAGS', 'LDFLAGS'] + list(mesonbuild.environment.cflags_mapping.values())
+    for v in varnames:
+        if v in os.environ:
+            del os.environ[v]
+
 if __name__ == '__main__':
+    unset_envs()
     unittest.main(buffer=True)