Zandronum Chat @ irc.zandronum.com
#zandronum Get the latest version: 3.0
Source Code
My View | View Issues | Change Log | Roadmap | Doomseeker Issue Support Ranking | Rules | My Account
  

View Issue Details [ Jump to Notes ] [ Issue History ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0003542Doomseeker[All Projects] Bugpublic2018-10-07 08:372018-12-11 13:49
ReporterWubTheCaptain 
Assigned ToWubTheCaptain 
PrioritylowSeverityfeatureReproducibilityalways
StatusconfirmedResolutionopen 
PlatformOSOS Version
Product Version1.1 
Target Version1.3Fixed in Version 
Summary0003542: No CXX_STANDARD defined in CMake build files
DescriptionWe could get more precise warnings from the C++ compiler, if it knew the standards version to use. There's a macro CMAKE_CXX_STANDARD for that, and CXX_STANDARD for build files.
Additional Informationhttps://cmake.org/cmake/help/latest/variable/CMAKE_CXX_STANDARD.html [^]
https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD.html [^]

https://cmake.org/cmake/help/latest/variable/CMAKE_CXX_STANDARD_REQUIRED.html [^]
https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD_REQUIRED.html [^]

As far as I'm aware, Doomseeker conforms to C++03 standard.
Attached Filespatch file icon 0001-CMake-Enforce-C++98-standard-_without-extensions_.patch [^] (961 bytes) 2018-12-10 14:08 [Show Content]

- Relationships

-  Notes
User avatar (0020113)
WubTheCaptain (developer)
2018-10-13 18:45
edited on: 2018-10-13 18:48

 Is it enough to set this once in /CMakeLists.txt, or do I need to also do the same for these?

  • src/core/CMakeLists.txt
  • src/plugins/CMakeLists.txt
  • src/plugins/*/CMakeLists.txt
  • src/wadseeker/CMakeLists.txt
User avatar (0020114)
Zalewa (developer)
2018-10-13 19:28

 Off the top of my head, setting a variable in the top-level CMakeLists.txt should propagate its value to the whole solution. Unless there are some caveats I'm not aware of, setting the CMAKE_* variables to proper values should do the job.
User avatar (0020238)
WubTheCaptain (developer)
2018-12-10 14:08

 Should be easy enough, patch attached. Needs testing?
User avatar (0020241)
Blzut3 (administrator)
2018-12-11 05:01

 Turning off CMAKE_CXX_EXTENSIONS will break compiling on some platforms which use those extensions in their system headers (i.e. some Mac SDKs).

It would probably be a good idea to consider src/wadseeker/CMakeLists.txt a top level to preempt adding work to 0003280.

Issue Community Support
Only registered users can voice their support. Click here to register, or here to log in.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2018-10-07 08:37 WubTheCaptain New Issue
2018-10-07 08:37 WubTheCaptain Status new => assigned
2018-10-07 08:37 WubTheCaptain Assigned To => WubTheCaptain
2018-10-07 08:38 WubTheCaptain Status assigned => acknowledged
2018-10-07 08:39 WubTheCaptain Description Updated View Revisions
2018-10-07 08:39 WubTheCaptain Additional Information Updated View Revisions
2018-10-07 09:17 WubTheCaptain Additional Information Updated View Revisions
2018-10-07 11:31 WubTheCaptain Priority normal => low
2018-10-13 18:28 WubTheCaptain Status acknowledged => assigned
2018-10-13 18:45 WubTheCaptain Note Added: 0020113
2018-10-13 18:45 WubTheCaptain Status assigned => feedback
2018-10-13 18:48 WubTheCaptain Note Edited: 0020113 View Revisions
2018-10-13 19:28 Zalewa Note Added: 0020114
2018-10-14 12:48 WubTheCaptain Status feedback => confirmed
2018-12-10 13:59 WubTheCaptain Status confirmed => assigned
2018-12-10 14:08 WubTheCaptain File Added: 0001-CMake-Enforce-C++98-standard-_without-extensions_.patch
2018-12-10 14:08 WubTheCaptain Note Added: 0020238
2018-12-10 14:08 WubTheCaptain Status assigned => needs review
2018-12-11 05:01 Blzut3 Note Added: 0020241
2018-12-11 13:49 WubTheCaptain Status needs review => confirmed




Questions or other issues? Contact Us.

Links

Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker