OlafvdSpek / ctemplate

BSD 3-Clause "New" or "Revised" License
353 stars 93 forks source link

CMake find module for ctemplate #91

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
A CMake module that allows the usage of find_package in a CMakeLists.txt

Example:
find_package(Ctemplate)

This could be added to a contrib folder.

Original issue reported on code.google.com by pasci.bach@gmail.com on 4 Jun 2012 at 9:02

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks. Looks very complex though. 

> Macro that wraps templates int varnames.h

What does this mean / do?

Original comment by olafvdspek on 11 Jun 2012 at 2:58

GoogleCodeExporter commented 9 years ago
Sorry for the late response.

Some of the code is borrowed from the FindQt4.cmake module. Most of the 
complexity (I assume you refer to the CTEMPLATE_MAKE_OUTPUT_file macro) is 
needed to make it work on windows too.

The macro: CTEMPLATE_VARNAMES(CT_HEADER ${CT_FILES})

Basically takes all the template files defined in CT_FILES and creates the 
corrensponding header files. The name of this header files are then saved in 
the variable CT_HEADERS. Internally the macros simply calls make_tpl_varnames_h 
for every  template file.

Would you suggest to better document this function in the code?

Original comment by pasci.bach@gmail.com on 14 Jun 2012 at 5:44

GoogleCodeExporter commented 9 years ago
I guess that function is an implementation detail, so it shouldn't really 
matter. 

> Macro that wraps templates int varnames.h

Is "int" a typo here?

Original comment by olafvdspek on 18 Jun 2012 at 2:12

GoogleCodeExporter commented 9 years ago
Sorry for my delayed response.

Yes int is a typo. It should say:
Macro that wraps templates into varnames.h

Original comment by pasci.bach@gmail.com on 2 Aug 2012 at 7:20

GoogleCodeExporter commented 9 years ago
What do you think? Can this be included in the source?

Original comment by pasci.bach@gmail.com on 21 Nov 2012 at 10:05

GoogleCodeExporter commented 9 years ago
I fixed the typo you mentioned above.

Original comment by pasci.bach@gmail.com on 21 Nov 2012 at 10:10

Attachments:

GoogleCodeExporter commented 9 years ago
Yes, I think this can be included in the next release.

Original comment by olafvdspek on 22 Nov 2012 at 5:50