[cmake-developers] Additional properties for PCH support

2013-11-26 Thread Peter Kuemmel
There is this 9 year old ticket about support for precompiled headers, http://www.cmake.org/Bug/view.php?id=1260 And I wanna bring up a patch for discussion, http://cmake.org/gitweb?p=stage/cmake.git;a=commitdiff;h=d207f1bf8fd697ae6521a1336dd27222168f548e With the two new source file

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Brad King
On 11/26/2013 12:32 PM, Peter Kuemmel wrote: So, is this the right way to improve the pch situation? Please summarize the situation here so we don't have to read the entire history of issue 1260 and all the proposed approaches. IIUC the modules in issue 1260 are all a workaround for lack for

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Stephen Kelly
Peter Kuemmel wrote: So, is this the right way to improve the pch situation? I don't know anything about PCH, but Daniel Pfeifer has a branch here: https://github.com/purpleKarrot/CMake/tree/WIP-PCH-Support Thanks, Steve. -- Powered by www.kitware.com Visit other Kitware open-source

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Peter Kuemmel
On 11/26/2013 12:32 PM, Peter Kuemmel wrote: So, is this the right way to improve the pch situation? Please summarize the situation here so we don't have to read the entire history of issue 1260 and all the proposed approaches. As you already said, many attempts to solve the problem with

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Brad King
On 11/26/2013 02:23 PM, Peter Kuemmel wrote: I'm not sure if it is a good idea to implement the pch support completely on the C++ side It could be a combination of C++-coded logic plus lookups of platform information variables set in Modules/Platform/*.cmake and Modules/Compiler/*.cmake files.

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Peter Kuemmel
It could be a combination of C++-coded logic plus lookups of You mean lookups only form C++ the side? platform information variables set in Modules/Platform/*.cmake and Modules/Compiler/*.cmake files. Just about everything CMake generates works this way. With this property we have

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Peter Kuemmel
I don't know anything about PCH So PCHs were never relevant in your real world projects? I also don't use them very often because of the non-existing support of CMake. And too often you break the build when someone builds without using PCHs.

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Brad King
On 11/26/2013 03:20 PM, Peter Kuemmel wrote: That doesn't matter because IMO the properties are not suitable for addition because they step on the toes of internal implementation What properties would be needed for the IDEs? And couldn't we make some properties internal only? We can't

Re: [cmake-developers] Additional properties for PCH support

2013-11-26 Thread Daniel Pfeifer
2013/11/26 Stephen Kelly steve...@gmail.com: Peter Kuemmel wrote: So, is this the right way to improve the pch situation? I don't know anything about PCH, but Daniel Pfeifer has a branch here: https://github.com/purpleKarrot/CMake/tree/WIP-PCH-Support Thanks for pointing at this branch. I