System.Drawing.Common dependency / .Net 6.0 support

[alex-jitbit]

You rely on System.Drawing.Common (via QRCoder) which has known issues on Linux and will be removed (marked as platform-specific) in .NET 6.0 here the official announcement https://docs.microsoft.com/en-us/dotnet/core/compatibility/core-libraries/6.0/system-drawing-common-windows-only

[alex-jitbit]

Also see this codebude/QRCoder#315

[ahwm]

Thanks for the information! We'll keep an eye on it and when (if) QRCoder releases a new version that addresses that then we'll update that.

[flytzen]

So it looks like #102 solves this - at least for now - and makes it run on .Net 6 on both Windows and Linux.
I am not really sure as to how. Maybe @TristanBoland's change to use PngByteQRCode uses a different code path in QR coder that does not rely on System.Drawing.
@ahwm If correct, then we can probably close this as well once we release the fix in #102? Or do we need to dig into this a bit more to make sure?

[ahwm]

So it looks like #102 solves this - at least for now - and makes it run on .Net 6 on both Windows and Linux. I am not really sure as to how. Maybe @TristanBoland's change to use PngByteQRCode uses a different code path in QR coder that does not rely on System.Drawing. @ahwm If correct, then we can probably close this as well once we release the fix in #102? Or do we need to dig into this a bit more to make sure?

Looking at the PngBytQRCode.cs file it looks like it manages not to depend on System.Drawing by writing predefined byte values directly to a stream. I could potentially see that working for all frameworks.

https://github.com/codebude/QRCoder/blob/v1.4.3/QRCoder/PngByteQRCode.cs (starting line 99)

[flytzen]

Fixed in #102.
Will be released in 2.5 - see #104