From ae509db48198840a607f943cf29b48765fb339da Mon Sep 17 00:00:00 2001 From: mtklein Date: Thu, 1 Sep 2016 13:19:20 -0700 Subject: GN: expunge all environment variables that have no effect on GN This should have no practical effect. I just want to make sure no environment variables beyond $PATH have any effect on GN builds. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2300173002 Review-Url: https://codereview.chromium.org/2300173002 --- infra/bots/recipe_modules/flavor/gn_android_flavor.py | 14 ++++++++------ infra/bots/recipe_modules/flavor/gn_flavor.py | 14 ++++++++------ 2 files changed, 16 insertions(+), 12 deletions(-) (limited to 'infra/bots/recipe_modules/flavor') diff --git a/infra/bots/recipe_modules/flavor/gn_android_flavor.py b/infra/bots/recipe_modules/flavor/gn_android_flavor.py index 9f3480a1a7..12832ecc83 100644 --- a/infra/bots/recipe_modules/flavor/gn_android_flavor.py +++ b/infra/bots/recipe_modules/flavor/gn_android_flavor.py @@ -9,6 +9,11 @@ class GNAndroidFlavorUtils(default_flavor.DefaultFlavorUtils): def supported(self): return 'GN_Android' == self.m.vars.builder_cfg.get('extra_config', '') + def _run(self, title, cmd): + path = self.m.vars.default_env['PATH'] + self.m.vars.default_env = {'PATH': path} + self.m.run(self.m.step, title, cmd=cmd, cwd=self.m.vars.skia_dir, env={}) + def compile(self, unused_target, **kwargs): compiler = self.m.vars.builder_cfg.get('compiler') configuration = self.m.vars.builder_cfg.get('configuration') @@ -35,9 +40,6 @@ class GNAndroidFlavorUtils(default_flavor.DefaultFlavorUtils): 'target_cpu': quote(target_arch), }.iteritems())) - run = lambda title, cmd: self.m.run(self.m.step, title, cmd=cmd, - cwd=self.m.vars.skia_dir, **kwargs) - - run('fetch-gn', [self.m.vars.skia_dir.join('bin', 'fetch-gn')]) - run('gn gen', ['gn', 'gen', self.out_dir, '--args=' + gn_args]) - run('ninja', ['ninja', '-C', self.out_dir]) + self._run('fetch-gn', [self.m.vars.skia_dir.join('bin', 'fetch-gn')]) + self._run('gn gen', ['gn', 'gen', self.out_dir, '--args=' + gn_args]) + self._run('ninja', ['ninja', '-C', self.out_dir]) diff --git a/infra/bots/recipe_modules/flavor/gn_flavor.py b/infra/bots/recipe_modules/flavor/gn_flavor.py index 66497b5d58..f759d4bcc4 100644 --- a/infra/bots/recipe_modules/flavor/gn_flavor.py +++ b/infra/bots/recipe_modules/flavor/gn_flavor.py @@ -15,6 +15,11 @@ class GNFlavorUtils(default_flavor.DefaultFlavorUtils): extra_config.startswith('SK') ]) + def _run(self, title, cmd): + path = self.m.vars.default_env['PATH'] + self.m.vars.default_env = {'PATH': path} + self.m.run(self.m.step, title, cmd=cmd, cwd=self.m.vars.skia_dir) + def compile(self, unused_target, **kwargs): """Build Skia with GN.""" compiler = self.m.vars.builder_cfg.get('compiler', '') @@ -55,9 +60,6 @@ class GNFlavorUtils(default_flavor.DefaultFlavorUtils): 'is_debug': 'true' if configuration == 'Debug' else 'false', }.iteritems())) - run = lambda title, cmd: self.m.run(self.m.step, title, cmd=cmd, - cwd=self.m.vars.skia_dir, **kwargs) - - run('fetch-gn', [self.m.vars.skia_dir.join('bin', 'fetch-gn')]) - run('gn gen', ['gn', 'gen', self.out_dir, '--args=' + gn_args]) - run('ninja', ['ninja', '-C', self.out_dir]) + self._run('fetch-gn', [self.m.vars.skia_dir.join('bin', 'fetch-gn')]) + self._run('gn gen', ['gn', 'gen', self.out_dir, '--args=' + gn_args]) + self._run('ninja', ['ninja', '-C', self.out_dir]) -- cgit v1.2.3