Re: [CMake] Please update the documentation for execute_process

2017-08-25 Thread Andreas Naumann
Am 25.08.2017 um 16:12 schrieb Jeffrey Walton: Below is a typical case for us (https://github.com/weidai11/cryptopp/blob/master/CMakeLists.txt). If it looks pretty shitty, it probably is. That's the best we have been able to come up with based on the documentation. Below is the example code. We

Re: [CMake] Please update the documentation for execute_process

2017-08-25 Thread Jeffrey Walton
> Below is a typical case for us > (https://github.com/weidai11/cryptopp/blob/master/CMakeLists.txt). If > it looks pretty shitty, it probably is. That's the best we have been > able to come up with based on the documentation. Below is the example code. We don't know whether it should be: set(SHE

[CMake] Please update the documentation for execute_process

2017-08-25 Thread Jeffrey Walton
The documentation for execute_process has some room for improvement. We recently got burned by a problem that has existed since at least 2011: https://stackoverflow.com/q/6797395/608639 . Once we learned the problem we could research a bit. The problem cost us over 8 man hours when it should not h