Skip to content
This repository was archived by the owner on May 17, 2026. It is now read-only.

ENH: Use modern ITK interface library for example#40

Closed
blowekamp wants to merge 1 commit into
mainfrom
modernize-cmake-interface-libraries
Closed

ENH: Use modern ITK interface library for example#40
blowekamp wants to merge 1 commit into
mainfrom
modernize-cmake-interface-libraries

Conversation

@blowekamp

Copy link
Copy Markdown
Member

Replace ITK_LIBRARIES with ITK::PrincipalComponentsAnalysisModule interface library.

  • Remove explicit include_directories as the interface library handles this automatically
  • Ensures proper dependency linking with modern CMake target interface libraries

Replace ITK_LIBRARIES with ITK::PrincipalComponentsAnalysisModule interface library.
Remove explicit include_directories as the interface library handles this automatically.
@hjmjohnson

Copy link
Copy Markdown
Member

The PrincipalComponentsAnalysis module has been ingested into ITK proper; this remote-module repository is archived and unmaintained. This PR added a modern ITK interface library for the standalone example build, which is obsolete by ingestion (the module's example now builds within ITK proper). Closing as part of draining the archived remote-module backlog. See the in-tree module at https://github.com/InsightSoftwareConsortium/ITK

@hjmjohnson hjmjohnson closed this May 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants