-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
linux/arm64 support (including a Rust rewrite) #2261
base: master
Are you sure you want to change the base?
Conversation
@RedstoneWizard08 rewriting node-canvas in Rust is most likely out of scope for this repository. There's also a Rust-based Skia binding already that you might be interested in: https://www.npmjs.com/package/@napi-rs/canvas. |
64ed3d8
to
ff0f2ab
Compare
@zbjornson Yes, I am aware that that exists, but the idea was to create a portable drop-in replacement for the existing C++ version, allowing the existing API to be supported on many more platforms. Since C++ was giving me a lot of issues (100+ commits worth of band-aid fixes and issues), I decided to rewrite it with a pure-rust implementation so it would be easier to cross-compile. |
The skia version also is not pure-rust, meaning it is harder to compile for other systems. |
Make it, ignore that guy, it's still cool. Ask ChatGPT if you get stuck on something math related, it might help. |
Yeah, I definitely will. |
Just ran into this while trying to build a Rust re-write project on arm64 architecture. I come from a long history of run-anywhere programming languages and am now discovering what pains may have led early developers to go take such an approach as the concept of distributed contributions (multi-arch) grew. Would hate to have to suggest migrating over to Skia as a patch! Happy to see that at least the request is active. |
@elysiumplain thanks! I did consider using Skia, but I decided not to, since Skia is still C with Rust, and I want this to be pure rust for maximum compatibility. I do know that |
Also I've been really busy with school and I have finals in 2 weeks so I haven't had a lot of time, I'm trying to keep working on it. |
Also @7f8ddd ChatGPT wasn't too helpful xD |
1 similar comment
Also @7f8ddd ChatGPT wasn't too helpful xD |
Might be able to get help in a graphics programming Discord. There are servers all about this kinda stuff with people who probably understand it. |
Any you recommend? |
HOLY CRAP I JUST FIXED THE MATRIX LETS GOOOO |
The issue was my bad logic in assuming that overwriting variable names wouldn't change anything. I was wrong. |
btw all tests are passing (not all are implemented yet tho) |
Thanks for contributing!
Would likely fix: #2074 #1662
Milestones:
Implementation Checklist