Results 1568 comments of John Cupitt

libvips doesn't have anything to download from a URL, but you can link the load/save operations up to any byte source or destination. See eg.: https://github.com/libvips/pyvips/blob/master/examples/connection.py So just swap the...

Quick note: c2pa has been rewritten in rust and is now at: https://github.com/contentauth/c2pa-rs As far as I can see, this package is not available in current Debian or Fedora, and...

Building the rust library makes what looks like a nice C interface too. It's HUGE! Five times larger than libvips. ``` $ ls -l libc2pa_c.so -rwxrwxr-x 2 john john 21650088...

Let's punt this to the 8.19 cycle. I asked on the c2pa discord and they are looking into linux packaging. They are not yet on 1.0 and the API is...

This is great @lxsameer, thank you for doing this work! It seems to work OK for me, here's nip4 loading a cr2 sample: ![image](https://github.com/user-attachments/assets/311e28c9-11b9-4701-9893-a92fe29a07a5) I made some (mostly) general comments....

Now I think again, I think you can just not set priority. You define a set of suffixes and no `_is_a()` method, so your loader will be picked based JUST...

I had another thought last night -- the `librawload` name is a bit awkward, we don't use the lib prefix anywhere else. How about `dcrawload`, since libraw is a wrapper...

I think this looks great. Thank you again for doing all this work @lxsameer. I suppose we should have some tests, but they can come in a followup PR. This...

OK, let's merge and fix anything else later. Nice job!

I bumped the version in master to 8.18 ready for the next dev cycle, and credited you in the changelog @lxsameer, I hope that's OK. https://github.com/libvips/libvips/commit/8c5c65fddc782dc5914caf52ea4c17de61f9cb73