Skip to content

COMP: VTKImageIO publicly depends on ImageIOBase#5850

Merged
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix_vtkimageio_deps
Mar 2, 2026
Merged

COMP: VTKImageIO publicly depends on ImageIOBase#5850
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix_vtkimageio_deps

Conversation

@blowekamp
Copy link
Copy Markdown
Member

Classes derived from ImageIOBase publicly depend on the interface.

Bug fix pulled from #5842

PR Checklist

  • No API changes were made (or the changes have been approved)
  • No major design changes were made (or the changes have been approved)
  • Added test (or behavior not changed)
  • Updated API documentation (or API not changed)
  • Added license to new files (if any)
  • Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
  • Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@github-actions github-actions Bot added type:Compiler Compiler support or related warnings area:IO Issues affecting the IO module labels Mar 2, 2026
@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Mar 2, 2026

/azp.run ITK.macOS.Python

@blowekamp blowekamp merged commit 1e4e9b8 into InsightSoftwareConsortium:main Mar 2, 2026
17 of 18 checks passed
@thewtex
Copy link
Copy Markdown
Member

thewtex commented Mar 6, 2026

Backported to release

hjmjohnson added a commit to hjmjohnson/ITK that referenced this pull request Apr 15, 2026
DCMTK, GE, LSM, and PhilipsREC have public headers that
#include itkImageIOBase.h or inherit from ImageIOBase, making
ITKIOImageBase a genuine public dependency. Move it from
PRIVATE_DEPENDS to DEPENDS so downstream consumers receive
the transitive include paths and link targets.

Follows the same pattern as the VTK fix in PR InsightSoftwareConsortium#5850.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:IO Issues affecting the IO module type:Compiler Compiler support or related warnings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants