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
Fix Xamarin project compiles #725
Conversation
Maybe we can add these back in later, when we have compiling support for these platforms.
src/OpenTK/OpenTK.iOS.csproj
Outdated
</Policies> | ||
</Properties> | ||
</MonoDevelop> | ||
</ProjectExtensions> |
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.
The <ProjectExtensions>
block was probably included on accident - remove?
src/OpenTK/OpenTK.iOS.csproj
Outdated
</Target> | ||
<ProjectExtensions> |
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.
No need to include this. Remove.
@@ -53,7 +53,14 @@ public AssemblyDefinition Resolve(AssemblyNameReference name, ReaderParameters p | |||
|
|||
var framework_dir = Path.GetDirectoryName(typeof(object).Module.FullyQualifiedName); | |||
var framework_dirs = on_mono | |||
? new[] { framework_dir, Path.Combine(framework_dir, "Facades") } | |||
? new[] |
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.
Looks good. I worry that we'll have to maintain our own version of the resolver in the future because of this, though.
The project analysis simply takes a long time due to the size of the project. |
* Fix Generator.Rewrite failing on iOS builds * Fix incorrect path * Directly link to Xamarin.iOS facades * Fix Xamarin.Mac framework directory * Add TVOS/WatchOS framework directories * Add Android framework directory * Remove unused framework dirs for now Maybe we can add these back in later, when we have compiling support for these platforms. * Remove ProjectExtensions
Purpose of this PR
OpenTK.csproj
.OpenTK.csproj
.Testing status
Comments