View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0014690

CMake

public

2014-01-09 08:45

2014-06-02 08:38

Reporter

Arunmozhi

Assigned To

Daniele E. Domenichelli

Priority

normal

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

CMake 3.0

Fixed in Version

CMake 3.0

Summary

0014690: CheckTypeSize doesn't behave as documented in daily build

Description

According to the documentation of CheckTypeSize ...

If the BUILTIN_TYPES_ONLY option is not given, the macro checks for headers <sys/types.h>, <stdint.h>, and <stddef.h>, and saves results in HAVE_SYS_TYPES_H, HAVE_STDINT_H, and HAVE_STDDEF_H. The type size check automatically includes the available headers, thus supporting checks of types defined in the headers.

...

The above behavior works in all versions of cmake including the current daily build 2.8.12.20140108-gc5c24.

However if there is a CHECK_TYPE_SIZE invocation with BUILTIN_TYPES_ONLY option, further CHECK_TYPE_SIZE without BUILTIN_TYPES_ONLY fails (does not detect/include the required headers <sys/types.h>, <stdint.h>, and <stddef.h>). This is observed in the daily builds (tested on 2.8.12.20140108-gc5c24).

Steps To Reproduce

An example CMakeLists.txt attached.

Tags

No tags attached.

Attached Files

CMakeLists.txt [^] (321 bytes) 2014-01-09 08:45 [Show Content] [Hide Content]

CMAKE_MINIMUM_REQUIRED(VERSION 2.8.12)

PROJECT(check_type_size C)

INCLUDE(CheckTypeSize)

# Commenting the following 2 lines fixes the issue
CHECK_TYPE_SIZE("int" INT_SIZE BUILTIN_TYPES_ONLY)
message("HAVE_INT_SIZE:${HAVE_INT_SIZE}")

CHECK_TYPE_SIZE("size_t" SIZET_SIZE)
message("HAVE_SIZET_SIZE:${HAVE_SIZET_SIZE}")

Relationships

Notes
(0034919) Brad King (manager) 2014-01-09 09:34	This commit is the likely culprit: CheckTypeSize: Add support for C++ http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=07a2342f [^] because it changed the argument parsing.

(0034922) Brad King (manager) 2014-01-10 09:20	CheckTypeSize: Unset temporary variables when done http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=8252406c [^]

(0036073) Robert Maynard (manager) 2014-06-02 08:38	Closing resolved issues that have not been updated in more than 4 months.

Issue History
Date Modified	Username	Field	Change
2014-01-09 08:45	Arunmozhi	New Issue
2014-01-09 08:45	Arunmozhi	File Added: CMakeLists.txt
2014-01-09 09:34	Brad King	Assigned To	=> Daniele E. Domenichelli
2014-01-09 09:34	Brad King	Status	new => assigned
2014-01-09 09:34	Brad King	Target Version	=> CMake 3.0
2014-01-09 09:34	Brad King	Note Added: 0034919
2014-01-10 09:20	Brad King	Note Added: 0034922
2014-01-10 09:20	Brad King	Status	assigned => resolved
2014-01-10 09:20	Brad King	Resolution	open => fixed
2014-01-10 09:20	Brad King	Fixed in Version	=> CMake 3.0
2014-06-02 08:38	Robert Maynard	Note Added: 0036073
2014-06-02 08:38	Robert Maynard	Status	resolved => closed