[zasexton]

Current situation

The unit tests link pthread by name, find_package(BLAS) and find_package(LAPACK) may select a library with 64-bit integers although the solver passes 32-bit int arguments, and find_package(VTK) links every installed VTK module. This PR links Threads::Threads, sets BLA_SIZEOF_INTEGER to 4, and requests only the VTK components the solver uses: CommonCore, CommonDataModel, CommonMisc, FiltersGeometry, and IOXML.

Fixes #705.

Release Notes

• Link only the VTK modules the solver uses, and require BLAS and LAPACK with 32-bit integers.

Documentation

No documentation changes are needed.

Testing

• Built the solver and passed the unit tests.
• Passed the fluid pipe_RCR_3d, struct block_compression, and electrophysiology cable_TTP_1d tests and the ustruct tests (36 runs).

Code of Conduct & Contributing Guidelines

• I agree to follow the Code of Conduct and Contributing Guidelines.

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[zasexton]

@ktbolt what do you think of explicitly linking only the needed components of VTK for solver builds?

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.45%. Comparing base (8df7565) to head (aa7bf08).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #712      +/-   ##
==========================================
+ Coverage   73.43%   73.45%   +0.01%     
==========================================
  Files         281      281              
  Lines       40720    40768      +48     
  Branches     6762     6770       +8     
==========================================
+ Hits        29903    29945      +42     
- Misses      10561    10567       +6     
  Partials      256      256              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ktbolt]

@zasexton Explicitly linking only the needed components of VTK for solver builds is the right thing to do; this is what I do when I build against my local VTK install that just builds VTK with these components that are needed just for reading/writing VTK files.

[bearsontheloose]

@claude review

[bearsontheloose]

This looks good.

[zasexton]

@ktbolt if you think these changes are good then I'll try to go ahead and merge this in today

[ktbolt]

Looks good!