<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div>I'd like to express my concerns about the proposed change. CMake strongly discourages using `file(GLOB)` to find source files, since file additions then do not automatically trigger a buildsystem regeneration.</div></div></div></blockquote><div><br></div><div>I second this.  The intent of the patch is to address an issue that is only present when using a practice explicitly not recommended in CMake documentation:<br><br><a href="https://cmake.org/cmake/help/v3.5/command/file.html">https://cmake.org/cmake/help/v3.5/command/file.html</a><br><div style="margin-left:40px" class="">Note: We do not recommend using GLOB to collect a list of source files from
your source tree.  If no CMakeLists.txt file changes when a source is
added or removed then the generated build system cannot know when to
ask CMake to regenerate.
</div><br></div></div><br></div></div>