# New warnings from master CMake

**URL:** https://discourse.itk.org/t/new-warnings-from-master-cmake/1575
**Category:** Engineering
**Tags:** cmake
**Created:** [February 5, 2019, 10:24pm UTC](https://discourse.itk.org/t/new-warnings-from-master-cmake/1575 "2019-02-05T22:24:05Z")
**Posts on this page:** 2
**Page:** 1

<div class="post-metadata">

### Author: ![dzenanz](https://discourse.itk.org/user_avatar/discourse.itk.org/dzenanz/32/1093_2.png) [@dzenanz](https://discourse.itk.org/u/dzenanz)
#### Post date: [February 5, 2019, 10:24pm UTC](https://discourse.itk.org/t/new-warnings-from-master-cmake/1575/1 "2019-02-05T22:24:05Z")

</div>

Using the master branch of CMake, and VS2019 Preview2 as the generator, these warnings appear:

```
...
Filter ZLIB is ON

CMake Warning (dev) at Modules/ThirdParty/HDF5/src/itkhdf5/CMakeLists.txt:733 (set):
implicitly converting 'INTEGER' to 'STRING' type.
This warning is for project developers. Use -Wno-dev to suppress it.

Configuring done

WARNING: Target "gtest" has EXCLUDE_FROM_ALL set and will not be built by default but an install rule has been provided for it. CMake does not define behavior for this case.

WARNING: Target "gtest_main" has EXCLUDE_FROM_ALL set and will not be built by default but an install rule has been provided for it. CMake does not define behavior for this case.

Generating done

```

@blowekamp what should be done about GTest `EXCLUDE_FROM_ALL` issue?

As these warnings are coming from a not-yet-released CMake, there is no urgency to address them.

Hopefully, HDF5 people will fix the warning coming from there once the new logic makes its way into a CMake release.

---

<div class="post-metadata">

### Author: ![blowekamp](https://discourse.itk.org/user_avatar/discourse.itk.org/blowekamp/32/79_2.png) [@blowekamp](https://discourse.itk.org/u/blowekamp)
#### Post date: [February 5, 2019, 10:31pm UTC](https://discourse.itk.org/t/new-warnings-from-master-cmake/1575/2 "2019-02-05T22:31:25Z")

</div>

For the google test the current intention is that those targets don’t get installed:

> <https://github.com/InsightSoftwareConsortium/ITK/blob/master/Modules/ThirdParty/GoogleTest/src/CMakeLists.txt#L40-L41>

So investigation into the itk\_module\_target is needed, to determine if it implement NO\_INSTALL in a way CMAKE may now expect.
