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

NuGet updater should specify a value for $TMP and $TEMP #11595

Open
brettfo opened this issue Feb 14, 2025 · 0 comments
Open

NuGet updater should specify a value for $TMP and $TEMP #11595

brettfo opened this issue Feb 14, 2025 · 0 comments
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet

Comments

@brettfo
Copy link
Contributor

brettfo commented Feb 14, 2025

Found during an internal log audit.

An internal project with a custom target was trying to copy a file with the following:

...
<Copy SourceFiles="some_file.txt" DestinationFiles="$(TEMP)/some_file.txt" />
...

But the updater image doesn't have the environment variable $TEMP defined so an attempt was made to copy the file to the root of the image which isn't allowed.

The Dockerfile should specify a value for both $TMP and $TEMP (and possibly $TMPDIR) so operations like this can succeed.

@brettfo brettfo added the L: dotnet:nuget NuGet packages via nuget or dotnet label Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dotnet:nuget NuGet packages via nuget or dotnet
Projects
Status: No status
Development

No branches or pull requests

1 participant