search

linebender / piet

An abstraction for 2D graphics.
Apache License 2.0
1.23k stars 95 forks source link

feat: Add a feature that allows piet to be used in no_std environments #557

Open notgull opened 1 year ago

notgull commented 1 year ago

As below, so above.

The main hiccup here is that I can't use Box<dyn std::error::Error> on no_std, so I just make it so the BackendError variant of Error isn't there on no_std. There is probably a more elegant solution to this problem.

xStrom commented 1 year ago

I haven't looked at the changes in any depth, but the CI failures should be resolved.

notgull commented 1 year ago

I can't replicate the failures locally

xStrom commented 11 months ago

Is this still something you would like to see merged?

If so, then the current problem is that the CI seems to be stuck in 'waiting'. I think the best approach here is for you to rebase on master and force push to your branch to re-trigger the CI for this PR. Thanks!

notgull commented 11 months ago

I still want this, it's just that I've been busy with more pressing issues.

notgull commented 8 months ago

I have no idea why the piet-core-graphics implementation is failing

ratmice commented 7 months ago

@notgull #562 has a fix for the piet-coregraphics failures, but also bumps some dependencies.

I got that other failure once about local connections, but another time it went green so I'm not sure if that check is spurious or why exactly it finally went green for me.