search

SyneRBI / SIRF-SuperBuild

SIRF CMake SuperBuild
http://www.ccpsynerbi.ac.uk
Apache License 2.0
15 stars 17 forks source link

should remove forcing STIR to C++-11 #852

Closed KrisThielemans closed 4 months ago

KrisThielemans commented 5 months ago

Not sure why we put this in, possibly when STIR didn't set minimum C++ version yet, but it will be soon too old. https://github.com/SyneRBI/SIRF-SuperBuild/blob/929fa6052d14d2c5f99d8917f5cd0b27c13bece8/SuperBuild/External_STIR.cmake#L103

Best thing is to let STIR take care of it, or to use same as SIRF.

@evgueni-ovtchinnikov I believe you force to C++-14 in some cases? Can you explain how (and maybe why)?

KrisThielemans commented 5 months ago

@evgueni-ovtchinnikov please change the line to use 14

KrisThielemans commented 4 months ago

@evgueni-ovtchinnikov can this be done ASAP?