-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Switch to new Microsoft.DotNet.SharedFramework.Sdk and refactor Host/Installer build subsets #38457
Merged
Merged
Changes from 107 commits
Commits
Show all changes
116 commits
Select commit
Hold shift + click to select a range
ba14e1d
Start creating runtime, apphost, and targeting packs using the new SD…
jkoritzinsky 6802ad1
Fix generation of runtime package, runtime symbols package, and host …
jkoritzinsky a36a296
Correcly include hosting files and DIA in the shared framework archiv…
jkoritzinsky c6f72c7
Include PackageOverrides.txt in the ref pack.
jkoritzinsky 8757c50
Fix product brand prefix.
jkoritzinsky cbd46d2
Add the shared host installer.
jkoritzinsky db70570
Add the hostfxr build and clean up MacOS building.
jkoritzinsky 7ad7304
Rename host and hostfxr projects to ensure we don't collide with the …
jkoritzinsky ece261c
Add bundle installer.
jkoritzinsky 84ab2b0
Merge branch 'master' of https://github.com/dotnet/runtime into adopt…
jkoritzinsky 0b54120
Update platform manifest entries to include linux/mac file variations…
jkoritzinsky 290d0fa
Update apphost pack to match master.
jkoritzinsky 2083c85
Fix typo
jkoritzinsky 49fbab6
Include manpages and fix linux package properties.
jkoritzinsky 0c9b00c
Fix linux package properties.
jkoritzinsky 6dfa868
Match short-description from packages in master.
jkoritzinsky 63cdc70
Update to uploaded package.
jkoritzinsky 5c637d2
Update metadata and SDK to generate an exact match of the productbuil…
jkoritzinsky c8bd51e
First draft moving crossgen2 to new SDK.
jkoritzinsky 890dc74
Get crossgen2 pack working with updated SDK.
jkoritzinsky 5ef903f
Merge remote-tracking branch 'dotnet/master' into adopt-unified-sharedfx
jkoritzinsky 98bb6c9
Build archives from the installer and bundle projects.
jkoritzinsky 4a0ecf0
Move the dotnet-runtime-deps installers to use the new SDK.
jkoritzinsky 3ac3696
Rough attempt to convert the build to use the new sfx sdk.
jkoritzinsky 361e750
Remove now empty subsets.
jkoritzinsky 13753c6
Update installer subsets for testing.
jkoritzinsky 31fcde4
Remove non-existent packaging project from subsets.
jkoritzinsky 6683b7a
Add missing Windows api shim dll to platform manifest.
jkoritzinsky 7137375
Update sharedfx sdk.
jkoritzinsky f8d1fbe
Add both versions of the long-name dac to the platform manifest.
jkoritzinsky 388711d
Skip builds for non-relevant components on Mono.
jkoritzinsky 006a761
Remove installer tests by default temporarily to validate product build.
jkoritzinsky 0fdcd70
Remove pkgprojs subset from mono installer build since it doesn't bui…
jkoritzinsky a118d3d
Skip the apphost pack on mono.
jkoritzinsky 1d84487
Update host test infra to use the new sfx projects.
jkoritzinsky 6752318
Delete an out of date test (the alternative case this test exercises …
jkoritzinsky ae145b3
Fix packaging tests and remove NETStandard packaging tests since we a…
jkoritzinsky e2e6624
Fixes for test official build failures. Re-eneable installer tests by…
jkoritzinsky 54496ab
Fix condition for building linux installers.
jkoritzinsky 0ce63ef
Exclude Mono header files from the deps.json. They don't need to be i…
jkoritzinsky 2612fba
Don't include the host files on the sfxproj.
jkoritzinsky d2adc6c
Add mono-specific files to the platform manifest.
jkoritzinsky 3d994ff
Remove extra copy step in installer job.
jkoritzinsky 06d242b
Include libmonosgen-2.0.a in platform manifest.
jkoritzinsky 90baa80
Fix change in linux-x64 test steps.
jkoritzinsky f5434af
Fix display name of linux package steps.
jkoritzinsky 7b7526d
Build the installers based on the portable build when applicable so t…
jkoritzinsky 1840af7
The installer build no longer consumes Microsoft.NETCore.Platforms an…
jkoritzinsky 15026fb
Set PackageBrandNameSuffix for runtime-deps packages.
jkoritzinsky 442fed4
Updated sharedfx sdk
jkoritzinsky a2cdf87
Fix debian runtime-deps dependency elements.
jkoritzinsky d9838ff
Merge branch 'master' of https://github.com/dotnet/runtime into adopt…
jkoritzinsky 2f6d892
Add new platform manifest entries.
jkoritzinsky b97b134
Add icudt.dat to the platform manifest.
jkoritzinsky c9597c4
Invert conditions to correctly skip generating installers on non-appl…
jkoritzinsky af5bcc8
Merge branch 'adopt-unified-sharedfx-official-build-test' of /mnt/d/s…
jkoritzinsky f521759
Create extra installer subset so we can build just the installers (an…
jkoritzinsky 4c5828d
Fix Crossgen2 package name.
jkoritzinsky dd86d7f
Merge branch 'master' into adopt-unified-sharedfx
jkoritzinsky d26bad7
Clean up some dead MSBuild goo and update the sharedfx package so we …
jkoritzinsky 494d9bd
Update SharedFX SDK.
jkoritzinsky cf95aa7
Merge branch 'master' into adopt-unified-sharedfx
jkoritzinsky d23d093
Fix discovery of the built dotnet layout for installer tests. Clean u…
jkoritzinsky 9c54206
Update to use the new refactored packages.
jkoritzinsky 160a2dd
Fix macOS build failures.
jkoritzinsky 1a35bcc
Fix linux installer build.
jkoritzinsky 4479907
Upgrade to newest version of the SDK and centrally manage versions.
jkoritzinsky a65d4c8
Merge branch 'master' of https://github.com/dotnet/runtime into adopt…
jkoritzinsky b4804c8
Update SDK to fix build skipping for the shared framework sdk.
jkoritzinsky d6be480
Update SDK with full fix for skipbuild with packaging.
jkoritzinsky 6f9b6d5
Another try for fixing the build-skip infra.
jkoritzinsky 78849bf
Upgrade SDK.
jkoritzinsky 9b3bb51
Fix package ids for mono packages.
jkoritzinsky d0f5313
Fix installer tests after update.
jkoritzinsky 8fe251c
Add System.Private.Runtime.InteropServices.Javascript.dll to platform…
jkoritzinsky 29f38aa
Merge branch 'master' of https://github.com/dotnet/runtime into adopt…
jkoritzinsky 1a71e7b
Fix Crossgen2 package to place all files in the tools folder.
jkoritzinsky 33407b5
Update to SDK with validation targets.
jkoritzinsky a87e9a1
Add new ICU files to manifest.
jkoritzinsky 7fe5c8e
Crossgen2 doesn't need to go through framework resolution so we can p…
jkoritzinsky a63c911
Put the property in a property group.
jkoritzinsky 14e80f4
Merge branch 'master' of https://github.com/dotnet/runtime into adopt…
jkoritzinsky 160bf5d
Ensure we build corehost before sfx since we no longer directly build…
jkoritzinsky ac5ce6c
Add missing entry for additional ICU data file.
jkoritzinsky 299745d
Remove step downloading all artifacts and remove MSBuild glue to hook…
jkoritzinsky 642e0f4
Merge branch 'master' into adopt-unified-sharedfx
jkoritzinsky faf52ec
Update SDK
jkoritzinsky a589182
Update SDK to get MSI fix.
jkoritzinsky 6ee0569
Merge branch 'master' into adopt-unified-sharedfx
jkoritzinsky 9e4ce3c
Don't even try building the hosting, crossgen2, or bundle installers …
jkoritzinsky 1f34d31
Remove out of date comment.
jkoritzinsky 1bd45aa
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky 829d01a
Rename subsets based on PR discussion.
jkoritzinsky a72bc60
Update platform manifest and fix path to 3rd party notices for instal…
jkoritzinsky 55d36ba
Remove test feed.
jkoritzinsky dbf59d7
Update subset references I missed.
jkoritzinsky 3b60cad
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky 3027275
Fixed the subset arguments in other yaml files.
jkoritzinsky 220044f
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky aafd6b1
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky 75a3c72
Add NuGet metadata to packs.
jkoritzinsky 417f0d6
Add diagnostics-specific incremental signing in CoreCLR when the file…
jkoritzinsky 77cf013
Pass sign type to job.
jkoritzinsky acf1a04
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky e2f5cb7
Merge branch 'master' into adopt-unified-sharedfx
jkoritzinsky 68909e2
Add new WASM files.
jkoritzinsky 147aec4
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky 152cf0e
Enable static graph restore for the whole repo
jkoritzinsky 3ac62de
Fix nits
jkoritzinsky 590d95d
Add back the runtime-deps packages. They were lost when we moved away…
jkoritzinsky e7bdc72
Don't disable NuGet static graph restore.
jkoritzinsky 454214d
Merge branch 'master' of github.com:dotnet/runtime into adopt-unified…
jkoritzinsky e111c6e
Make sure we sign the cross-arch same-os DAC
jkoritzinsky f931142
Update installers package.
jkoritzinsky 7be2e3b
Update Installers package from arcade
ViktorHofer e36639b
Update installers package.
jkoritzinsky 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
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.
I'd say before getting this in we need to merge mmitche's changes and verify. His changes remove the target and a bunch of these operations won't play well.
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’ve already validated that these changes work and sign the files correctly. This can replace the SignBinaries group in mmitche’s PR.
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 this work outside a target though?
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.
It could if L55 would use the WithMetadataValue item function to check on filename but I doubt it's worth changing it now that Jeremy already did the validation.
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.
As discussed during standup, since we're just using globs and not using the version variables that Arcade updates, we can use this inside of an InitialTargets target if need be to ensure that we correctly sign the short name DAC, long name DAC, and DBI.