Re: [cmake-developers] FindGTest modification

2016-03-15 Thread Brad King
On 03/15/2016 02:57 PM, Zak Eckert wrote: > I updated to using the CMAKE_CONFIGURE_DEPENDS Thanks. I squashed the changes and applied them: FindGTest: Automatically re-run cmake when tests change https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=a5d3d003 Please verify that version works

Re: [cmake-developers] FindGTest modification

2016-03-15 Thread Zak Eckert
I updated to using the CMAKE_CONFIGURE_DEPENDS. I have attached a patch that has both commits in it. Let me know if there are any other issues. Zak On Mon, Mar 14, 2016 at 10:58 AM, Brad King wrote: > On 03/11/2016 12:00 PM, Zak Eckert wrote: > > +

Re: [cmake-developers] FindGTest modification

2016-03-14 Thread Brad King
On 03/11/2016 12:00 PM, Zak Eckert wrote: > + configure_file(${source} ${source}.out COPYONLY) Thanks. Please try using CMAKE_CONFIGURE_DEPENDS instead: https://cmake.org/cmake/help/v3.5/prop_dir/CMAKE_CONFIGURE_DEPENDS.html It should be something like: set_property(DIRECTORY APPEND

[cmake-developers] FindGTest modification

2016-03-11 Thread Zak Eckert
I have made a small edit to the FindGTest module that eliminates the need to manually rerun cmake when new tests are added to existing test files when using the gtest_add_tests function. There are false positives when the test files are modified without adding/removing tests. Additionally it