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

Upgrade to .net6 #42

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Upgrade to .net6 #42

wants to merge 1 commit into from

Conversation

yangyiyi
Copy link

@yangyiyi yangyiyi commented Oct 11, 2023

Hi,

Due to IronPDF version 2023.10.3 is using SixLabors version 1.0.0 and above, caused conflict with NetBarcode version and throwed exception of Method missing.

  1. Update Target Framework to .net6.
  2. Update dependency package.
  3. Update the method which is no longer in new version of the package.

1. Update Target Framework to .net6.
2. Update dependency package.
3. Update the method which is no longer in new version of the package.
<PackageReference Include="SixLabors.ImageSharp" Version="2.1.1" />
<PackageReference Include="SixLabors.ImageSharp.Drawing" Version="1.0.0-beta14" />
<PackageReference Include="SixLabors.Fonts" Version="1.0.1" />
<PackageReference Include="SixLabors.ImageSharp" Version="2.1.6" />

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conflict with #44 changes

@@ -1,6 +1,6 @@
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<TargetFramework>netstandard2.0</TargetFramework>
<TargetFrameworks>net6.0</TargetFrameworks>
Copy link

@Bogatinov Bogatinov Apr 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this breaks .NET Framework compatibility
https://learn.microsoft.com/en-us/dotnet/standard/net-standard?tabs=net-standard-2-0

Possible to use netstandard2.0 while also supporting the new SixLabors libs?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Bogatinov , Thanks for comment, I will check netstandard2.0 is supporting the new SixLabors libs or not and do a necessary update on it.

Copy link

@Bogatinov Bogatinov Apr 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tommyliew-machines I checked it out

  • SixLabors.Fonts 1.0.1 works on netstandard2.0, but 2.0.0 is net6.0 dependent
  • SixLabors.ImageSharp.Drawing 1.0.0 works on netstandard2.0, but 2.0.0 is net6.0 dependent
  • SixLabors.ImageSharp 2.1.8 is the last version that allows netstandard2.0

I can open a PR and update the libraries

  • SixLabors.Fonts 1.0.1
  • SixLabors.ImageSharp.Drawing 1.0.0
  • SixLabors.ImageSharp 2.1.8
  • and any methods that are different between Beta version

Then we can do 2 changes

  • my PR to offer a small bump in v1.7.1.1, which is still compliant with netstandard2.0
  • this PR to offer update to v2.0.0 which will break net48 compatibility = if Author is okay with that decision

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.

3 participants