-
Notifications
You must be signed in to change notification settings - Fork 132
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
Update to 5.0 GA #1845
Merged
Merged
Update to 5.0 GA #1845
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
cde2f44
Update to 5.0.100-rtm.20521.5, BAR Build ID 68494
dseefeld 754ba87
Update previously source-built to N-1
dseefeld 9d1cbb7
Reconcile patches
dseefeld 0a95e42
Update to 5.0.100-rtm.20522.4, BAR Build ID 68583
dseefeld ce1386d
Update commits in Version.Details.xml and reconcile patches
dseefeld 0a3ce07
Remove temporary RC1 fix in aspnetcore
dseefeld f434b4e
Install built runtime for usage by aspnetcore
dseefeld f46b98f
Update roslyn to build CodeStyle packages
dseefeld 317e0ae
Update / add installer patches
dseefeld 084a2a7
Fix broken patch
dseefeld 5160b7c
Update prevSourceBuilt to GA version
dseefeld adae8b2
Pin System.Security.Cryptography.Cng version in runtime build
dseefeld 9a9a56f
Update path to sdk installation
dseefeld e99ca58
Update smoke test nuget.config entries
dseefeld 3fcb32f
Update online prebuilt baseline
dseefeld 909a8cc
Update package name for aspnetcore reference
dseefeld 50bfc0d
Add runtime feed to smoke-testNuGet.Config
dseefeld 82e9958
Update runtime config entry
dseefeld 059cd96
Update to 20201026.5, BAR Build ID 68880
dseefeld ecdb22a
Set UseStableVersions flag
dseefeld 45e1c26
Update online baseline
dseefeld a77ec25
Update offline prebuilt baseline
dseefeld 943c33a
Include patch from https://github.com/dotnet/source-build/pull/1850/
dseefeld a1ad7a4
Merge branch 'master' into 5.0GA
dseefeld 9e05d3e
Re-add portable build only prebuilts
dseefeld 5fa1a69
Merge branch '5.0GA' of https://github.com/dseefeld/source-build into…
dseefeld File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 changes: 28 additions & 0 deletions
28
patches/installer/0008-Update-blob-versions-to-use-ref-pkg-versions-to-trac.patch
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
From 992512755a9df08ccfd531f7507c298aa5b22112 Mon Sep 17 00:00:00 2001 | ||
From: dseefeld <dseefeld@microsoft.com> | ||
Date: Thu, 29 Oct 2020 16:01:59 +0000 | ||
Subject: [PATCH] Update blob versions to use ref pkg versions to track stable | ||
versions | ||
|
||
--- | ||
src/redist/targets/GenerateLayout.targets | 4 ++-- | ||
1 file changed, 2 insertions(+), 2 deletions(-) | ||
|
||
diff --git a/src/redist/targets/GenerateLayout.targets b/src/redist/targets/GenerateLayout.targets | ||
index 9e7249d..f1b11d4 100644 | ||
--- a/src/redist/targets/GenerateLayout.targets | ||
+++ b/src/redist/targets/GenerateLayout.targets | ||
@@ -11,8 +11,8 @@ | ||
|
||
<PropertyGroup> | ||
<!-- Blob storage directories are not stabilized, so these must refer to a package that does not stabilize --> | ||
- <AspNetCoreBlobVersion>$(VSRedistCommonAspNetCoreSharedFrameworkx6450PackageVersion)</AspNetCoreBlobVersion> | ||
- <CoreSetupBlobVersion>$(MicrosoftNETCoreAppInternalPackageVersion)</CoreSetupBlobVersion> | ||
+ <AspNetCoreBlobVersion>$(MicrosoftAspNetCoreAppRefPackageVersion)</AspNetCoreBlobVersion> | ||
+ <CoreSetupBlobVersion>$(MicrosoftNETCoreAppRefPackageVersion)</CoreSetupBlobVersion> | ||
dseefeld marked this conversation as resolved.
Show resolved
Hide resolved
|
||
<WindowsDesktopBlobVersion>$(MicrosoftWindowsDesktopAppPackageVersion)</WindowsDesktopBlobVersion> | ||
|
||
<!-- Change these individually to or $(CoreSetupBlobVersion), $(AspNetCoreBlobVersion), or appropriate fixed version depending if corresponding .Ref packages are unpinned. --> | ||
-- | ||
1.8.3.1 | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Can we delete this line? #1627
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.
No. Without it, Roslyn wasn't building stable versions.
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.
Why is that a goal? The Microsoft Roslyn build didn't produce stable versions.
https://maestro-prod.westus2.cloudapp.azure.com/1297/https:%2F%2Fgithub.com%2Fdotnet%2Froslyn/68310/graph
And in Version.Details.xml,
<Dependency Name="Microsoft.NETCore.Compilers" Version="3.8.0-5.20519.18" ...
vs.
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.
Hmm... A later repo was looking for a stable version and wasn't finding it. I was mistaken in my fix for that.
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 it's fine to leave #1627 tracking it for followup but wanted to dig into any issues to make it easier to tackle later.
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.
Without the patch, the production build produces
Microsoft.NETCore.Compilers.3.8.0-dev.20519.18.nupkg
. The "dev" version doesn't seem right.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.
That sounds like a missing
-ci
orOfficialBuildId
or something along those lines.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 didn't change roslyn.proj. It has
OfficialBuild=true
here:source-build/repos/roslyn.proj
Line 12 in 92d2aef
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.
Adding
--ci
to roslyn.proj caused it to generate the desired version.Microsoft.Net.Compilers.Toolset.3.8.0-5.20519.18.nupkg
I'll fix this under #1627 once this PR is merged