-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Merging dev/release/2.0 into release/2.0.0 #1776
Conversation
…t vs binary and which must have crlf (sln) vs lf (sh)
…ts in the test output folder
Moving to the repo toolset.
VSO builds are green, and publishing successfully, so we should be good to plumb this through to all the branches. CI will continue failing due to the netci.groovy changes which have to be checked in before they will work. |
@@ -1,21 +0,0 @@ | |||
@echo off |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think my comment about these still being useful was lost in the original PR. Can we please fix up the paths and retain these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a -dogfood
option and an explicit build\dogfood.bat
script
@@ -0,0 +1,3 @@ | |||
@echo off | |||
powershell -NoLogo -NoProfile -ExecutionPolicy ByPass -NoExit %~dp0Build.ps1 -dogfood %* | |||
exit /b %ErrorLevel% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will I actually get those environment variables set up for me if I do dogfood.cmd?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like they'd be trapped in the powershell process
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-NoExit
preserves the powershell context when the script finishes executing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see.
export DOTNET_MSBUILD_SDK_RESOLVER_SDKS_DIR="$MSBuildSDKsPath" | ||
export NETCoreSdkBundledVersionsProps="$DotNetRoot/sdk/$DotNetCliVersion/Microsoft.NETCoreSdk.BundledVersions.props" | ||
export CustomAfterMicrosoftCommonTargets="$MSBuildSDKsPath/Microsoft.NET.Build.Extensions/msbuildExtensions-ver/Microsoft.Common.Targets/ImportAfter/Microsoft.NET.Build.Extensions.targets" | ||
export MicrosoftNETBuildExtensionsTargets="$CustomAfterMicrosoftCommonTargets" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I want dotnet that was downloaded on my PATH too in this env.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its added to the path already down below
…201.3 (#1776) [main] Update dependencies from dotnet/roslyn
FYI. @dotnet/dotnet-cli