[CMake] Help with libintl/configure tests

Bill Hoffman bill.hoffman at kitware.com
Fri Apr 25 09:09:47 EDT 2008


Bill Hoffman wrote:

> 1. call these directly in check_type_size:
>  check_include_file(sys/types.h HAVE_SYS_TYPES_H)
>  check_include_file(stdint.h HAVE_STDINT_H)
>  check_include_file(stddef.h HAVE_STDDEF_H)
> 
> 2. Update the documentation for check_type_size to tell people that they 
> need those checks.
> 
> 3. Try to see if those have been called and issue a warning if they have 
> not.
> 
> 
> I am thinking 1 is the best solution and will avoid confusion.  I don't 
> think it will hurt because duplicate calls will be ignored (the test 
> won't be done multiple times unless a different variable name is used, 
> but in that case the check type size will not have worked anyway.
> 

I went with 1 and did a commit.
$ cvs commit -m "ENH: make sure all required headers are checked before 
checking type size" CheckTypeSize.cmake
/cvsroot/CMake/CMake/Modules/CheckTypeSize.cmake,v  <--  CheckTypeSize.cmake
new revision: 1.31; previous revision: 1.30

-Bill



More information about the CMake mailing list