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

Added net6.0-windows to TFMs. #1019

Closed
wants to merge 2 commits into from
Closed

Conversation

AraHaan
Copy link

@AraHaan AraHaan commented Oct 19, 2022

Due to the fact that in .NET 6 S.D.C is Windows only, I decided to make a Windows only target for .NET 6 which removes a few annoying package refs that get placed into my build outputs in my .NET 6 WinForms applications that use Svg. If the application is windows only, I see no need to include the osx System.Drawing runtime reference either.

Reference Issue

Contributes to: #939

What does this implement/fix? Explain your changes.

This adds a net6.0-windows target to Svg which allows it to be used in .NET 6 WindowsDesktop applications without annoyingly unneeded dependencies polluting the output directory in them.

Any other comments?

None.

Due to the fact that in .NET 6 S.D.C is Windows only, I decided to make a Windows only target for .NET 6 which removes a few annoying package refs that get placed into my build outputs in my .NET 6 WinForms applications that use Svg. If the application is windows only, I see no need to include the ``osx`` System.Drawing runtime reference either.
Source/Svg.csproj Outdated Show resolved Hide resolved
@AraHaan AraHaan marked this pull request as ready for review October 19, 2022 09:07
@mrbean-bremen
Copy link
Member

@wieslawsoltes - can you have a look at this? I'm traveling...

@paulushub paulushub self-requested a review December 28, 2023 10:35
@AraHaan
Copy link
Author

AraHaan commented Dec 29, 2023

I might have to rebase this after all this time.

@paulushub
Copy link
Contributor

paulushub commented Dec 29, 2023

I might have to rebase this after all this time.

Thanks, also consider adding .NET 8 (we could skip the 7), net472, net481. update the samples.

@AraHaan
Copy link
Author

AraHaan commented Dec 30, 2023

I would say that would be the best thing to do as well.

@paulushub paulushub mentioned this pull request Jan 9, 2024
@paulushub paulushub self-assigned this Jan 14, 2024
@paulushub
Copy link
Contributor

After the analysis, package like Open-XML-SDK uses SVG-NET and targets and .NET 6.

Moving to the Windows only net6-windows will not help the situation.

The package libgdiplus-packaging is actually marked as
This repository has been archived by the owner on Dec 31, 2022. It is now read-only.

Since this is only targeting net-standard we could restrict the support to that target, as it has no future.

@paulushub paulushub mentioned this pull request Jan 14, 2024
13 tasks
@paulushub
Copy link
Contributor

@AraHaan The main issues are addressed in #1123, please post any remaining issue and I will address it.
We need to clear the PR workload, sorry for any inconvenience.

@paulushub paulushub closed this Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants