Skip to content

BUG: VTK Bridge publicly depends on ITKCommon#5852

Merged
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix-vtk-bridge-dependency
Mar 2, 2026
Merged

BUG: VTK Bridge publicly depends on ITKCommon#5852
blowekamp merged 1 commit intoInsightSoftwareConsortium:mainfrom
blowekamp:fix-vtk-bridge-dependency

Conversation

@blowekamp
Copy link
Copy Markdown
Member

Changes VTK Bridge module dependency from PRIVATE_DEPENDS to DEPENDS for ITKCommon, making it a public dependency as required.

The VTK Bridge module's headers include ITKCommon headers, which means ITKCommon must be a public dependency, not a private one. This ensures that consuming the ITKVTK module properly propagates the ITKCommon dependency.

Related to PR #5842

Changes:

  • Modified Modules/Bridge/VTK/itk-module.cmake to change PRIVATE_DEPENDS to DEPENDS for ITKCommon

Changes VTK Bridge module dependency from PRIVATE_DEPENDS to DEPENDS for ITKCommon, making it a public dependency as required.

Related to PR InsightSoftwareConsortium#5842
@github-actions github-actions Bot added type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances area:Bridge Issues affecting the Bridge module labels Mar 2, 2026
@dzenanz
Copy link
Copy Markdown
Member

dzenanz commented Mar 2, 2026

/azp run ITK.Windows

@blowekamp blowekamp merged commit a3a8973 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Bridge Issues affecting the Bridge module type:Bug Inconsistencies or issues which will cause an incorrect result under some or all circumstances

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants