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

Unset NUGET_PACKAGES in official build only #9523

Conversation

rainersigwald
Copy link
Member

The devdiv build machine images appear to have started setting the
environment variable NUGET_PACKAGES, which caused a mismatch between
the location where drop.app was restored (repo-local location) and
where it was used from (environment-variable defined machine location),
causing build failures.

Co-authored-by: AR-May 67507805+AR-May@users.noreply.github.com

@ghost
Copy link

ghost commented Dec 12, 2023

Hello! I noticed that you're targeting one of our servicing branches. Please consider updating the version.

@rainersigwald rainersigwald force-pushed the make-official-build-robust-to-NUGET_PACKAGES branch from 82589de to 28d5df2 Compare December 12, 2023 17:37
The devdiv build machine images appear to have started setting the
environment variable `NUGET_PACKAGES`, which caused a mismatch between
the location where `drop.app` was restored (repo-local location) and
where it was used from (environment-variable defined machine location),
causing build failures.

Co-authored-by: AR-May <67507805+AR-May@users.noreply.github.com>
@rainersigwald rainersigwald force-pushed the make-official-build-robust-to-NUGET_PACKAGES branch from 28d5df2 to e27564e Compare December 12, 2023 17:39
Copy link
Member

@AR-May AR-May left a comment

Choose a reason for hiding this comment

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

LGTM

@rainersigwald
Copy link
Member Author

Bypassing policy and PR builds since it's an official-build-break fix change.

@rainersigwald rainersigwald merged commit 67d742f into dotnet:vs17.9 Dec 12, 2023
6 of 8 checks passed
@rainersigwald rainersigwald deleted the make-official-build-robust-to-NUGET_PACKAGES branch December 12, 2023 18:33
ladipro pushed a commit that referenced this pull request Dec 12, 2023
The devdiv build machine images appear to have started setting the
environment variable `NUGET_PACKAGES`, which caused a mismatch between
the location where `drop.app` was restored (repo-local location) and
where it was used from (environment-variable defined machine location),
causing build failures.

Co-authored-by: AR-May <67507805+AR-May@users.noreply.github.com>
ladipro pushed a commit that referenced this pull request Dec 12, 2023
The devdiv build machine images appear to have started setting the
environment variable `NUGET_PACKAGES`, which caused a mismatch between
the location where `drop.app` was restored (repo-local location) and
where it was used from (environment-variable defined machine location),
causing build failures.

Co-authored-by: AR-May <67507805+AR-May@users.noreply.github.com>
AR-May added a commit to AR-May/msbuild that referenced this pull request Dec 13, 2023
AR-May added a commit that referenced this pull request Dec 13, 2023
This reverts commit 67d742f.
This workaround no more needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants