-
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
Enable Mono build in CI #1934
Merged
Merged
Enable Mono build in CI #1934
Changes from all commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
dda0e24
Enable Mono in default build
akoeplinger 9dd624a
Install Mono build dependencies
akoeplinger b38c606
Integrate Mono into the build system
ViktorHofer b71b56d
Fix yml syntax
ViktorHofer 3fc55c6
Fix yml #2
ViktorHofer 8c1e766
Try another syntax
ViktorHofer fc15f62
Revert "Try another syntax"
ViktorHofer 516c74e
RuntimeFlavor and yml
ViktorHofer b299e14
Merge branch 'master' into enable-mono-ci
akoeplinger 20eca7c
Update build container images
ViktorHofer 1cda2e3
Merge branch 'enable-mono-ci' of https://github.com/dotnet/runtime in…
ViktorHofer 4e7474e
Fix yml
ViktorHofer d3622c4
Add libraries release test run on mono
ViktorHofer 876a802
Installer yml fixes
ViktorHofer 579cac8
Fix yml
ViktorHofer d38acee
Move yml block
ViktorHofer 87dc59e
Yml fixes
ViktorHofer 7c49749
Yml fix
ViktorHofer 3e02bac
yml fix juhu
ViktorHofer 00d3770
Yml fixes
ViktorHofer d05dd74
yml fixes
ViktorHofer 015f2dc
yml again
ViktorHofer 6b83c47
Build release only
ViktorHofer df16fe3
Fix yml
akoeplinger ec4268e
Revert "Build release only"
akoeplinger 79a41ad
Merge remote-tracking branch 'upstream/master' into enable-mono-ci
akoeplinger 4451143
Fix script args
akoeplinger 23437a8
Remove corecrl specific args
akoeplinger d362d64
configure.ac fix
akoeplinger f5ad7f5
Disable not needed Windows builds
akoeplinger f2efbac
Update linux musl x64 container
ViktorHofer 380795e
Change container back
ViktorHofer bc95ce3
Merge remote-tracking branch 'upstream/master' into enable-mono-ci
akoeplinger 86419c6
Fix System.Private.CoreLib build in Mono
akoeplinger a70cb85
Fix mono configuration properties
ViktorHofer 9fe3655
Merge branch 'enable-mono-ci' of https://github.com/dotnet/runtime in…
ViktorHofer ebb5853
Update container for linux arm64
ViktorHofer 695feb4
Disable failing mono configurations in yml
ViktorHofer be29630
Revert alpine image update
ViktorHofer 77207be
Foce clang for mono unix build
akoeplinger fd6d599
Use alpine image based on runtime flavor
ViktorHofer 691837d
Merge branch 'enable-mono-ci' of https://github.com/dotnet/runtime in…
ViktorHofer b453b07
Fix yml
ViktorHofer e46af53
Yml updates
ViktorHofer 614de00
Another yml display name approach
ViktorHofer 1e97f40
Honor RuntimeArtifactsPath
ViktorHofer 8374713
Update MonoConfiguration in Subsets
ViktorHofer 7beff64
Fix installer build
ViktorHofer 5d0de8d
Fix installer RuntimeArtifactsPath
ViktorHofer 58fe4e7
Use clang differently
akoeplinger 90788fc
Disable Windows runs
akoeplinger cfd3e79
Disable more tests
akoeplinger 73bae92
Disable mono windows test runs
ViktorHofer b6df298
Fix doc about RuntimeFlavor
akoeplinger 480e7cb
Disable more tests
akoeplinger 060c3d8
Fix Windows build
akoeplinger 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 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 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -50,6 +50,13 @@ jobs: | |
- name: ROOTFS_DIR | ||
value: ${{ parameters.jobParameters.crossrootfsDir }} | ||
|
||
- ${{ if ne(parameters.jobParameters.runtimeFlavor, '') }}: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. runtimeFlavor should always be passed down as a |
||
- name: runtimeFlavorName | ||
${{ if eq(parameters.jobParameters.runtimeFlavor, 'mono') }}: | ||
value: Mono | ||
${{ if eq(parameters.jobParameters.runtimeFlavor, 'coreclr') }}: | ||
value: CoreCLR | ||
|
||
osGroup: ${{ parameters.osGroup }} | ||
osSubgroup: ${{ parameters.osSubgroup }} | ||
archType: ${{ parameters.archType }} | ||
|
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 open an issue for this and investigate? We should try to be consistent with the docker images we build with.
Also, instead of using
!= mono
I would change this to== coreclr
so that it is easier to read.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.
Yes I'll file an issue for that as it's highly concerning. Just hadn't had time yet.