Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dotnet-core-sdk: Fix environment setup #43842

Merged
merged 1 commit into from Apr 29, 2024

Conversation

RemiLacroix-IDRIS
Copy link
Contributor

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.

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.
Copy link

spackbot-app bot commented Apr 25, 2024

Hi @RemiLacroix-IDRIS! I noticed that the following package(s) don't yet have maintainers:

  • dotnet-core-sdk

Are you interested in adopting any of these package(s)? If so, simply add the following to the package class:

    maintainers("RemiLacroix-IDRIS")

If not, could you contact the developers of this package and see if they are interested? You can quickly see who has worked on a package with spack blame:

$ spack blame dotnet-core-sdk

Thank you for your help! Please don't add maintainers without their consent.

You don't have to be a Spack expert or package developer in order to be a "maintainer," it just gives us a list of users willing to review PRs or debug issues relating to this package. A package can have multiple maintainers; just add a list of GitHub handles of anyone who wants to volunteer.

Copy link
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tldahlgren tldahlgren merged commit a3aa5b5 into spack:develop Apr 29, 2024
13 checks passed
wdconinc pushed a commit that referenced this pull request May 4, 2024
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.
@RemiLacroix-IDRIS RemiLacroix-IDRIS deleted the patch-1 branch May 7, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants