Skip to content

Comments

AP-589: rebuild image based on iipsrv 1.3#7

Merged
anarchivist merged 4 commits intomainfrom
AP-589
Feb 18, 2026
Merged

AP-589: rebuild image based on iipsrv 1.3#7
anarchivist merged 4 commits intomainfrom
AP-589

Conversation

@anarchivist
Copy link
Member

@anarchivist anarchivist commented Feb 12, 2026

Ruven, the IIPImage maintainer, updates the iipsrv images upon new releases of iipsrv. This PR uses his (Alpine/lighttpd-based) image as a base image and update the package dependencies as part of the build process.

This upgrades us from IIPImage 1.1 to 1.3. One of the major changes in 1.2 is that iipsrv now supports the version 3.0 of the IIIF Image API, which necessitates some changes in the fixtures we're using. Otherwise, IIPImage 1.3 should be a drop-in replacement for the v1.1 image we have been using for UCBEARS.

@anarchivist anarchivist force-pushed the AP-589 branch 12 times, most recently from 048375c to d87b6ec Compare February 12, 2026 23:30
@anarchivist anarchivist force-pushed the AP-589 branch 2 times, most recently from ee56703 to e22f214 Compare February 18, 2026 00:38
@anarchivist anarchivist changed the title AP-589: rebuild image based on upstream iipsrv/iipsrv image AP-589: rebuild image based on iipsrv 1.3 Feb 18, 2026
@anarchivist
Copy link
Member Author

anarchivist commented Feb 18, 2026

@danschmidt5189 this is ready for re-review. tldr:

  • this updates the image to build iipsrv 1.3, and provides a parameterized build to allow us to compile other branches/tags as needed.
  • iipsrv 1.2+ returns iiif image api v3 responses by default. the tests and readme have been updated to reflect htis.
  • at some point debian libtiff-dev introduced libwebp-dev as a dependency; accordingly, autoconf detects it and builds webp support into iipsrv by default. this introduces a runtime dependency on libwebpmux3, which is added to the base image deps. we could certainly disable webp support, but i don't think it's worth it.
  • finally: UCBEARS tests are passing with the image built from this branch.

Copy link
Member

@awilfox awilfox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but I'm not sure if we want to document IIPSRV_VERSION specifically in the README.

r+ otherwise.

@anarchivist
Copy link
Member Author

@awilfox thanks; that's been fixed in 64ce4c1.

Copy link
Member

@danschmidt5189 danschmidt5189 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 🚢

@anarchivist anarchivist merged commit 4f547d8 into main Feb 18, 2026
5 checks passed
@anarchivist anarchivist deleted the AP-589 branch February 18, 2026 01:44
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.

3 participants