Skip to content

Fix vtk CMake error#158

Merged
FilipovicLado merged 2 commits into
masterfrom
fix-vtk-targets
Jun 24, 2026
Merged

Fix vtk CMake error#158
FilipovicLado merged 2 commits into
masterfrom
fix-vtk-targets

Conversation

@FilipovicLado

Copy link
Copy Markdown
Member
  • Fixes a duplicate-target CMake error that occurred when ViennaLS was
    pulled in by a parent project (e.g. ViennaPS) that had already loaded
    VTK
  • If VTK::CommonCore already exists in scope, ViennaLS now skips VTK
    discovery entirely and reuses the existing targets
  • Bumps version to 5.8.3

Test plan

  • CI passes on all platforms (Linux, Windows, macOS x86_64 + arm64)
  • ViennaPS python bindings build successfully against this branch
    (validated via temporary commit pin in ViennaPS)

@FilipovicLado FilipovicLado merged commit c3a1a2a into master Jun 24, 2026
18 checks passed
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.

1 participant