Skip to content

Initial addition of tvOS Mono - #34475

Merged
akoeplinger merged 40 commits into
dotnet:masterfrom
directhex:add-tvos
Apr 8, 2020
Merged

Initial addition of tvOS Mono#34475
akoeplinger merged 40 commits into
dotnet:masterfrom
directhex:add-tvos

Conversation

@directhex

Copy link
Copy Markdown
Contributor

Let's see if this works

Comment thread src/mono/mono.proj Outdated
Comment thread src/mono/mono.proj Outdated
Comment thread eng/native/configureplatform.cmake
Comment thread eng/native/configureplatform.cmake Outdated
Comment thread src/libraries/Directory.Build.props Outdated
Comment thread src/libraries/OSGroups.json
Comment thread eng/native/configureplatform.cmake Outdated
Comment thread src/mono/mono.proj Outdated
Comment thread src/mono/mono.proj
Comment thread src/libraries/Native/build-native.sh Outdated
@directhex
directhex marked this pull request as ready for review April 6, 2020 20:10
Comment thread eng/Subsets.props

@akoeplinger akoeplinger left a comment

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.

Looks very good, only a few changes to address :)

Comment thread eng/Subsets.props
Comment thread src/libraries/Native/Unix/CMakeLists.txt
Comment thread src/libraries/Native/Unix/configure.cmake Outdated
Comment thread src/libraries/Native/build-native.sh Outdated
Comment thread src/mono/mono.proj
Comment thread src/mono/mono.proj
Comment thread src/mono/mono.proj Outdated

@akoeplinger akoeplinger left a comment

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.

One nit, looks good otherwise!

Comment thread src/mono/mono.proj Outdated
@akoeplinger
akoeplinger merged commit 952eeca into dotnet:master Apr 8, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants