ncruces / RethinkRAW

RethinkRAW is an unpretentious, free RAW photo editor.
https://rethinkraw.com
MIT No Attribution
89 stars 8 forks source link

deps: updates wazero to 1.0.0-rc.1 #46

Closed evacchi closed 1 year ago

evacchi commented 1 year ago

This updates wazero to 1.0.0-rc.1. This version does not change any public API, but it delivers several bug fixes and improvements; including

Signed-off-by: Edoardo Vacchi evacchi@users.noreply.github.com

ncruces commented 1 year ago

This broke dcraw, again. I'll look into it.

Need to split it off into a module with its own integration tests.

evacchi commented 1 year ago

let me know how I can assist, what errors are you seeing?

ncruces commented 1 year ago

Thanks! No error messages. Simply, no thumbnails get extracted.

I'm pretty sure it's going to be because of changes to WASI, and/or WASI libc, that my custom FS doesn't like. But it passes fstest.TestFS now, so we'll see. It's a useful component, still, I think (it sandboxes dcraw to only be able to read that one file).

Don't worry. It'll be a fun debugging session. I'm sure it's my problem to fix.

evacchi commented 1 year ago

can you check if it's related to this? https://github.com/tetratelabs/wazero/pull/1182 in that case the workaround is easy, just check if fsconfig is nil and avoid passing it in that case (or pass a meaningful value instead)

ncruces commented 1 year ago

It was my issue all along! Hopefully, this time I'm covering all corner cases for fs.FS.