-
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
Stop harvesting packages in runtime: remove prebuilts #1850
Merged
Conversation
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
dseefeld
approved these changes
Nov 2, 2020
dseefeld
added a commit
to dseefeld/source-build
that referenced
this pull request
Nov 3, 2020
Include patch to remove import of restore/harvestPackages.targets in runtime libraries build to fix issue with portable build
The last leg succeeded but timed out during publish/cleanup. Now it's not able to get build agents for a retry due to https://github.com/dotnet/core-eng/issues/11320. Going ahead with merge since it was effectively green. |
dseefeld
added a commit
that referenced
this pull request
Nov 3, 2020
* Update to 5.0.100-rtm.20521.5, BAR Build ID 68494 * Update previously source-built to N-1 * Reconcile patches Note: Removed aspnetcore version number patch because it is set to a stable 5.0 version. Note: Removed roslyn Humanizer removal patch because there is now more usage of Humanizer in the code base. May need to redo this patch or add Humanizer. (See #1767) * Update to 5.0.100-rtm.20522.4, BAR Build ID 68583 * Update commits in Version.Details.xml and reconcile patches * Remove temporary RC1 fix in aspnetcore * Install built runtime for usage by aspnetcore * Update roslyn to build CodeStyle packages * Update / add installer patches * Fix broken patch * Update prevSourceBuilt to GA version * Pin System.Security.Cryptography.Cng version in runtime build * Update path to sdk installation * Update smoke test nuget.config entries * Update online prebuilt baseline * Update package name for aspnetcore reference * Add runtime feed to smoke-testNuGet.Config * Update runtime config entry * Update to 20201026.5, BAR Build ID 68880 * Set UseStableVersions flag * Update online baseline * Update offline prebuilt baseline * Include patch from #1850 Include patch to remove import of restore/harvestPackages.targets in runtime libraries build to fix issue with portable build * Re-add portable build only prebuilts
MichaelSimons
pushed a commit
to dotnet/source-build-externals
that referenced
this pull request
Feb 8, 2022
* Update to 5.0.100-rtm.20521.5, BAR Build ID 68494 * Update previously source-built to N-1 * Reconcile patches Note: Removed aspnetcore version number patch because it is set to a stable 5.0 version. Note: Removed roslyn Humanizer removal patch because there is now more usage of Humanizer in the code base. May need to redo this patch or add Humanizer. (See dotnet/source-build#1767) * Update to 5.0.100-rtm.20522.4, BAR Build ID 68583 * Update commits in Version.Details.xml and reconcile patches * Remove temporary RC1 fix in aspnetcore * Install built runtime for usage by aspnetcore * Update roslyn to build CodeStyle packages * Update / add installer patches * Fix broken patch * Update prevSourceBuilt to GA version * Pin System.Security.Cryptography.Cng version in runtime build * Update path to sdk installation * Update smoke test nuget.config entries * Update online prebuilt baseline * Update package name for aspnetcore reference * Add runtime feed to smoke-testNuGet.Config * Update runtime config entry * Update to 20201026.5, BAR Build ID 68880 * Set UseStableVersions flag * Update online baseline * Update offline prebuilt baseline * Include patch from dotnet/source-build#1850 Include patch to remove import of restore/harvestPackages.targets in runtime libraries build to fix issue with portable build * Re-add portable build only prebuilts
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Conditionally remove the import of
restore\harvestPackages.targets
to remove some PackageDownload elements bringing in prebuilts.For #1844.