Skip to content
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

Multitarget Views.WPF #1069

Merged
merged 4 commits into from
Dec 21, 2019
Merged

Multitarget Views.WPF #1069

merged 4 commits into from
Dec 21, 2019

Conversation

TysonMN
Copy link
Contributor

@TysonMN TysonMN commented Dec 19, 2019

Description of Change

SkiaSharp.View.WPF now targets .NET Core 3.0 in addition to .NET Framework 4.5.1.

Bugs Fixed

Related to issue #1029, which is about getting both SkiaSharp.View.WPF as well as SkiaSharp.View.WindowsForms to target .Net Core.

API Changes

None

PR Checklist

  • Has tests (if omitted, state reason in description)
  • Rebased on top of master at time of PR
  • NA
  • Changes adhere to coding standard
  • Updated documentation

@mattleibow
Copy link
Contributor

We can also update the nuspecs with the new netcoreapp3.0:
https://github.com/mono/SkiaSharp/blob/master/nuget/SkiaSharp.Views.Desktop.Common.nuspec#L39

And:
https://github.com/mono/SkiaSharp/blob/master/nuget/SkiaSharp.Views.WPF.nuspec#L38

Just note, the nuspecs are pseudo-nuspec and are manipulated in the build. That is why they are a bit different.

@TysonMN
Copy link
Contributor Author

TysonMN commented Dec 19, 2019

@mattleibow
Copy link
Contributor

The platform that builds it. It looks to build on all the platforms, so leave it out.

@TysonMN
Copy link
Contributor Author

TysonMN commented Dec 19, 2019

@mattleibow mattleibow merged commit bc3d08f into mono:dev/vs-2019 Dec 21, 2019
@mattleibow
Copy link
Contributor

Thanks for the PR! Merged!

@TysonMN
Copy link
Contributor Author

TysonMN commented Dec 21, 2019

My pleasure :)

Is there a NuGet package (possibly a pre-release) with these changes?

I would test it by adding it to a project we are developing at work.

@mattleibow
Copy link
Contributor

We can! Just noticed that this branch failed to build because DevOps got disconnected from all its agents... Rebuilding and then I'll let you know when you can get your hands on the build.

I did start a build on our public CI:
https://dev.azure.com/xamarin/public/_build/results?buildId=11744&view=results

That should work for testing, but I am also going to push out the PR build to the preview feed:

https://aka.ms/skiasharp-eap/index.json

The reason for this is that CI builds both the preview and the stable packages to make sure everything is working. Then when I release I first push the preview packages for a few weeks, and then go stable - no need to rebuild. But, sometimes you may be tempted to select the stable packages from the CI server - and then it gets cached... I can control what gets to the preview feed.

@mattleibow
Copy link
Contributor

Hehe, seems my certificates to build the samples expired today... I'm going to have to upload them to CI, but I need to get a sleep action going here. When I wake up in a few hours, I'll rebuild with the v1.68.2 and push that after merging the VS2019 build to master.

@TysonMN TysonMN deleted the dev/vs-2019_1029_WPF_target_.NET_Core branch December 21, 2019 12:55
@mattleibow mattleibow added this to the v1.68.2 milestone Apr 29, 2020
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.

2 participants