Skip to content

Replacing Custom Oses with versionless rids - #254

Closed
Anipik wants to merge 4 commits into
dotnet:masterfrom
Anipik:Rid
Closed

Replacing Custom Oses with versionless rids#254
Anipik wants to merge 4 commits into
dotnet:masterfrom
Anipik:Rid

Conversation

@Anipik

@Anipik Anipik commented Mar 17, 2020

Copy link
Copy Markdown

Relate runtime PR dotnet/runtime#33315

This change is required to enable to pass the win formatting leg.

Comment thread doc/config.md Outdated
Comment thread doc/cijobs.md Outdated
Comment thread sample_config.json Outdated
@Anipik

Anipik commented Mar 17, 2020

Copy link
Copy Markdown
Author

@ericstj i suggest merging this after the runtime PR, bcoz otherwise it will lead to all PRs in the runtime having failed formatting job. or maybe we can merge both of them together.

@ViktorHofer

Copy link
Copy Markdown
Member

What about other release branches (5.0 and 3.x) which consume jitutils from master?

Comment thread doc/cijobs.md Outdated
Comment thread bootstrap.sh

SunOS)
__HostOS=SunOS
__HostOS=sunos

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haven't seen this RID before. Is it going to be a problem that we don't have a mapping for this in runtime.json?

@ericstj

ericstj commented Mar 17, 2020

Copy link
Copy Markdown
Member

The diff looks reasonable to me, but I can't really sign off on this as I don't know the purpose of most of these files.

Co-Authored-By: Eric StJohn <ericstj@microsoft.com>
@BruceForstall

Copy link
Copy Markdown
Contributor

Closing this since dotnet/runtime#33315 was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants