intel / wds

Wireless Display Software For Linux OS (WDS)
https://01.org/wds
GNU Lesser General Public License v2.1
511 stars 104 forks source link

rename the generated header to parser.hpp #149

Open michaelolbrich opened 8 years ago

michaelolbrich commented 8 years ago

CMake expects this name an generated the corresponding dependencies. Without this the files are always regenerated.

alexshalamov commented 8 years ago

We've changed parser to be pure C parser. I'm not sure why cmake expects .hpp, maybe because of https://github.com/01org/wds/blob/master/CMakeLists.txt#L3 ?

I think it would be better to tell to cmake to treat generated parser as C code.

michaelolbrich commented 6 years ago

I think the header file name is derived from the source file name: cpp -> hpp. Anyways, since cmake 3.4 there is an option to set the filename. Patch updated.