Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove System.Drawing #454

Merged
merged 5 commits into from
Oct 27, 2022
Merged

Conversation

linch90
Copy link
Contributor

@linch90 linch90 commented Oct 26, 2022

Hi, Magicodes teams, I see request from #440 issue that remove the System.Drawing as it will be disabled in Linux after Net7.
After some research, I think SixLabors.ImageSharp would be a good choice.
So I do some work to change to SixLabors.ImageSharp, and remove the System.Drawing package and related codes.
This pull request is a break change, and need for additional review.
Lastly, this pull request is for review, not for accept, if it is acceptable for your team, there are additional codes to commit for this pull request.

@xl-wenqiang
Copy link
Collaborator

Thank you very much for your contribution! Due to SixLabors ImageSharp has changed its license, and commercial authorization is required for for-profit organizations with an annual total income of more than $1 million, so we need to consider carefully.

非常感谢您的贡献!由于 SixLabors.ImageSharp变更了授权许可证,对于年总收入高于100万美元的营利性组织需要进行商业授权,因此我们需要慎重考虑。

@xl-wenqiang
Copy link
Collaborator

Q. What do I need to do as an Open Source or Source Available consumer following these changes?

A. Absolutely nothing. No changes are required in any form. The license will continue to be granted to you under the Apache License 2.0. You and any consumers of your work continue to enjoy exactly the same freedom as before.

SixLabors/ImageSharp#2151 (reply in thread)
@hueifeng

@xl-wenqiang xl-wenqiang merged commit e125a19 into dotnetcore:develop Oct 27, 2022
@xl-wenqiang xl-wenqiang added this to the 2.7.0 milestone Nov 1, 2022
@linch90 linch90 deleted the br-linch-sixlabors branch January 20, 2023 13:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

后续对.net7的支持会移除 System.Drawing.Common/libgdiplus的依赖吗
3 participants