Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Add Tizen 5.0 RID for NETCoreApp#30875

Merged
weshaggard merged 1 commit into
dotnet:masterfrom
gbalykov:update-tizen-rid-for-netcore-app
Jul 6, 2018
Merged

Add Tizen 5.0 RID for NETCoreApp#30875
weshaggard merged 1 commit into
dotnet:masterfrom
gbalykov:update-tizen-rid-for-netcore-app

Conversation

@gbalykov
Copy link
Copy Markdown
Member

@gbalykov gbalykov commented Jul 6, 2018

@danmoseley danmoseley requested a review from weshaggard July 6, 2018 16:39
<OfficialBuildRID Include="tizen.4.0.0-armel">
<Platform>armel</Platform>
</OfficialBuildRID>
<OfficialBuildRID Include="tizen.5.0.0-armel">
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.

Should we just replace the tizen.4.0.0-armel with tizen.5.0.0-armel?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@weshaggard both RIDs are required

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.

Sounds good. That implies you still support both 4.0.0 and 5.0.0 with the current master builds. Are there plans to support the latest .NET Core on 5.0.0 only in the future?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@weshaggard until the official release of Tizen 5.0 in autumn we have to support both. After that it is yet to be decided.

@weshaggard weshaggard merged commit e38aaa5 into dotnet:master Jul 6, 2018
@karelz karelz modified the milestones: Future, 3.0 Jul 8, 2018
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…-for-netcore-app

Add Tizen 5.0 RID for NETCoreApp

Commit migrated from dotnet/corefx@e38aaa5
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants