ITK icon indicating copy to clipboard operation
ITK copied to clipboard

BUG: Include find GTest CMake code when installed

Open blowekamp opened this issue 1 year ago • 4 comments

May address: #303

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.

blowekamp avatar Nov 20 '24 21:11 blowekamp

The GTest libraries should not be a build time dependency of an installed ITK, but not sure how to disable the GTest libraries in an installed ITK.

blowekamp avatar Nov 20 '24 21:11 blowekamp

not sure how to disable the GTest libraries in an installed ITK.

One needs to make sure nothing depends on them publicly. Then they shouldn't need to be installed.

bradking avatar Nov 20 '24 21:11 bradking

not sure how to disable the GTest libraries in an installed ITK.

One needs to make sure nothing depends on them publicly. Then they shouldn't need to be installed.

In all "itk-module.cmake" files, if used, it is under "TEST_DEPENDS".

blowekamp avatar Nov 20 '24 23:11 blowekamp

Off the top of my head I don't see why TEST_DEPENDS in the build tree should affect requirements in the install tree, at least conceptually.

bradking avatar Nov 21 '24 13:11 bradking

@blowekamp should this be integrated?

thewtex avatar Oct 27 '25 19:10 thewtex