Skip to content

Upgrade to libvips v8.17.0#275

Merged
lovell merged 1 commit intolovell:mainfrom
kleisauke:vips-8.17.0
Jun 6, 2025
Merged

Upgrade to libvips v8.17.0#275
lovell merged 1 commit intolovell:mainfrom
kleisauke:vips-8.17.0

Conversation

@kleisauke
Copy link
Collaborator

No description provided.

@lovell lovell merged commit 1b3bdab into lovell:main Jun 6, 2025
14 checks passed
@kleisauke kleisauke deleted the vips-8.17.0 branch June 6, 2025 11:57
@lovell
Copy link
Owner

lovell commented Jun 6, 2025

Thanks Kleis, I think I'm going to refactor out the version number checking logic from the build script (and merge the value of the libvips version into this) so it's quicker to check locally if there's a newer version of any dependency without running an entire build.

@kleisauke
Copy link
Collaborator Author

Great, sounds good to me.

I'm currently investigating this on the darwin-x64 runner:

Checking for function "rsvg_handle_set_stylesheet" with dependency librsvg-2.0: NO 

https://github.com/lovell/sharp-libvips/actions/runs/15489943190/job/43612935538#step:4:9155

Curiously, this is not an issue on the darwin-arm64v8 runner.
https://github.com/lovell/sharp-libvips/actions/runs/15489943190/job/43612935518#step:4:9010

(this was originally found in kleisauke/net-vips@2253e0a)

@lovell
Copy link
Owner

lovell commented Jun 6, 2025

Perhaps prefix: '#include <librsvg/rsvg.h>' is required here?

https://github.com/libvips/libvips/blob/5da295f49fe27a33a79100cd332fe401d6835264/meson.build#L392

@kleisauke
Copy link
Collaborator Author

Good catch. I'll open a draft PR on this repository to see if it fixes the issue.

(sounds very similar to the issues linked to mesonbuild/meson#11011)

@lovell
Copy link
Owner

lovell commented Jun 7, 2025

See commit be68966 for the CI improvements to version number checking (plus takes advantage of new ARM64 runners).

megantaite pushed a commit to animoto/sharp-libvips that referenced this pull request Jul 29, 2025
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

Successfully merging this pull request may close these issues.

2 participants