From a3aa5b59cd04f296333228915199a881f4b077a8 Mon Sep 17 00:00:00 2001 From: Rémi Lacroix Date: Mon, 29 Apr 2024 23:47:37 +0200 Subject: dotnet-core-sdk: Fix environment setup (#43842) The "DOTNET_CLI_TELEMETRY_OPTOUT" environment variable should be defined when using the product, not when installing it (the installation phase is just extract the files anyway). Also use `~` instead of `-` to check for the variant and fix the second argument for `env.set` which should also be a string. --- var/spack/repos/builtin/packages/dotnet-core-sdk/package.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'var') diff --git a/var/spack/repos/builtin/packages/dotnet-core-sdk/package.py b/var/spack/repos/builtin/packages/dotnet-core-sdk/package.py index 723d502141..6f74fe3d5d 100644 --- a/var/spack/repos/builtin/packages/dotnet-core-sdk/package.py +++ b/var/spack/repos/builtin/packages/dotnet-core-sdk/package.py @@ -86,9 +86,9 @@ class DotnetCoreSdk(Package): variant("telemetry", default=False, description="allow collection of telemetry data") - def setup_build_environment(self, env): - if "-telemetry" in self.spec: - env.set("DOTNET_CLI_TELEMETRY_OPTOUT", 1) + def setup_run_environment(self, env): + if "~telemetry" in self.spec: + env.set("DOTNET_CLI_TELEMETRY_OPTOUT", "1") def install(self, spec, prefix): mkdirp("bin") -- cgit v1.2.3-70-g09d2