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

ZXing.Net Bindings for SkiaSharp Package missing Linux #544

Open
ahmed-abdelrazek opened this issue Jan 17, 2024 · 4 comments
Open

ZXing.Net Bindings for SkiaSharp Package missing Linux #544

ahmed-abdelrazek opened this issue Jan 17, 2024 · 4 comments

Comments

@ahmed-abdelrazek
Copy link

it comes with
image
but not
image

@mathy-plutoflume
Copy link

Version 2.88.3 has vulnerabilities (#545) can the package also be upgraded when above issue is worked on.

@ahmed-abdelrazek
Copy link
Author

Version 2.88.3 has vulnerabilities (#545) can the package also be upgraded when above issue is worked on.

they already updated it to 2.88.6 but didn't push a new package yet 6d11086

@micjahn
Copy link
Owner

micjahn commented Jan 18, 2024

The SkiaSharp.NativeAssets.* dependencies are added transitively by the SkiaSharp package.
I have no idea how to add the package SkiaSharp.NativeAssets.Linux correctly to the SkiaSharp binding package. And in my opinion, it should not be necessary.
What do you think about adding the package manually to your project which uses the SkiaSharp binding package?

@ahmed-abdelrazek
Copy link
Author

i did add it manually which fixed my problem (running asp core web api on linux with skiasharp renderer) but the point is i shouldn't have to because i expect my cross platforms library to by cross platforms
in hindsight it's on SkiaSharp project including just windows and mac by default and not linux as well they have whole section on their readme about this issue https://github.com/mono/SkiaSharp#using-skiasharp mono/SkiaSharp#453

i guess you can also add the SkiaSharp.NativeAssets.Linux library manually just like SkiaSharp itself until they fix their package as temporary (with fanky support) fix for anyone that except it to work out of the box like me
or leave this issue open for any one that is asking

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

No branches or pull requests

3 participants