[cmake-developers] [CMake 0014152]: Comment change in flags.make triggers unnecessary rebuilds

[Mantis Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://public.kitware.com/Bug/view.php?id=14152 
====================================================================== 
Reported By:                Craig Scott
Assigned To:                
====================================================================== 
Project:                    CMake
Issue ID:                   14152
Category:                   CMake
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2013-05-17 00:33 EDT
Last Modified:              2013-05-17 00:33 EDT
====================================================================== 
Summary:                    Comment change in flags.make triggers unnecessary
rebuilds
Description: 
Let's say you have a set of source files and you use
set_source_files_properties() to set COMPILER_DEFINITIONS on just one of those
files. If you change that definition, it results in *all* the files getting
rebuilt. This is because the makefile generator adds a comment to the flags.make
file stating the custom define. The actual flags are unchanged, but because the
comment changes, make rebuilds all the sources because those sources depend on
the contents/timestamp of flags.make. I would recommend that the custom defines
NOT be included in a comment within the flags.make file to avoid this. Maybe
those comments can be sent to a separate file which has no effect on the build,
since they are just informational anyway?
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2013-05-17 00:33 Craig Scott    New Issue                                    
======================================================================