-
Notifications
You must be signed in to change notification settings - Fork 1.9k
[WPF] Use project style SDK; target netcoreapp3.0 #7063
Conversation
Hmm build errors, but I'm guessing we need to update the tooling on our side @rmarinho ?
|
Looks like you are not building with .net core 3? I used the latest preview (7) and enabled preview support in VS 2019. |
@krdmllr this would need to install net core 3.0 preview on the bits, not sure if you are familiar with AzurePipelines but basically we need in the windows bits to use the botnet installer task to install .net core 3, and then try the build, you can edit our yaml file to do that. |
@rmarinho i'm familiar with azure pipelines :) I would add the following task
|
/azp run |
Pull request contains merge conflicts. |
6e30852
to
818b323
Compare
Can you see if the install .net core 3 task is successful? We could create a global.json in the solution root directory during the build where we can set the .net core version https://docs.microsoft.com/de-de/dotnet/core/tools/global-json
|
The build is currently failing because the installed VS/MSbuild version is too old for dotnetcore 3 |
Besides the build pipelin issues I currently have the issue that the OpenTK lib does not target .net standard. Their update is on the way but I think it will take some time opentk/opentk#823 Since this only affects the OpenGLViewRenderer, I excluded the file and dependency when building for .net core 3 so this version currently does not support the OpenGLView. The .net461 version is unaffected. |
I've been working with @krdmllr to get the build working. For now I have been manually triggering builds for the YAML file and we've made some progress. The one thing that seems to stop us now is that there is no Java SDK installed on the hosted Win2019/VS2019 agents. |
Just revisiting this one. @rmarinho can we simply switch to VS2019 build agents? Is that planned already somewhere? |
Last time i checked the vs2019 wasn't building our UWP projects out of the box |
@rmarinho is there anything planned on your site to migrate to VS2019 build servers? |
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.
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.
I just got the master branch and reapplied all changes that this branch has and the Gallery project start working again. At least on .NET Framework. I was running some additional tests, and have you realized that OpenTK is built only for .NET Framework? An 4.0 version is being developed with support of .Net Standard/Core. |
061938e
to
5ded028
Compare
Finally fixed it, the generics.xaml wasnt loading because this got kicked out while merging @trturino the OpenGL was discussed above (feb 22) |
What about other projects? Maps, Control Gallery? they should also be update to restore with netcoreapp |
@VladislavAntonyuk PR's are welcome :) Some stuff like iOS and Android don't support sdk style projects right now irc. |
I have build errors with blank WPF Core app and 4.8.0.780-nightly XF build. Should I add Xamarin.Forms package reference to the project and right away there're errors:
|
@maxim-saplin , it is my favorite issue) Add |
@VladislavAntonyuk that worked, thanks! |
can you check this issue, please? During your csproj style change the application info data are lost, now version is 0.0.0.0 |
This adds .netcoreapp3.0 besides the existing net 4.6.1 framework target for the WPF backend.
Description of Change
Add target framework .netcoreapp3.0.
Replaces the old Platform.WPF project file with an project sdk style one.
Update the OpenTK.GLControl package to 3.0.1.
Issues Resolved
fixes #5092
API Changes
None
Platforms Affected
WPF
Behavioral/Visual Changes
None
Before/After Screenshots
Not applicable
Testing Procedure
PR Checklist