Problem 2002 - Geant4Config.cmake should not set CMAKE_CXX_EXTENSIONS
Summary: Geant4Config.cmake should not set CMAKE_CXX_EXTENSIONS
Status: CLOSED FIXED
Alias: None
Product: Geant4
Classification: Unclassified
Component: cmake (show other problems)
Version: 10.3
Hardware: All All
: P4 normal
Assignee: Ben Morgan
URL:
Depends on:
Blocks:
 
Reported: 2017-09-06 12:24 CEST by Dennis Klein
Modified: 2019-07-08 11:18 CEST (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this problem.
Description Dennis Klein 2017-09-06 12:24:38 CEST 
Dear Geant4 team,

when integrating Geant4 via its Geant4Config.cmake into our project, it sets the CMAKE_CXX_EXTENSIONS option to OFF. This is a global cmake option and should in my opinion not be set by an included dependency. I propose to prefix the variable with Geant4_ as done with all the other variables. So, the user can decide whether to apply this setting.

Best regards,
Dennis
Comment 1 Ben Morgan 2018-11-06 10:06:40 CET 
Apologies for the delay in this. We're looking at this in relation to a larger review of compiler flags required (rather than recommended) for use when compiling/linking Geant4 libraries.
Comment 2 Ben Morgan 2019-07-08 11:18:47 CEST 
This is now fixed on master, so will be in the next 10.6 release, plus patches.