Fixed FTBFS caused by usage of dynamic exception specification, no longer accepted in c++17. #13
Merged
MicheleC
merged 1 commits from fix/FTBFS-with-c++17 into master 3 years ago
SlavekB
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'fix/FTBFS-with-c++17'
Deleting a branch is permanent. It CANNOT be undone. Continue?
When building with clang in bullseye, c++17 is used. There was FTBFS due to two problems:
Could you please test it still builds fine in the various distros/archs before we merge?
I didn't do a test, for now, but there is one comment.
OPENEXR_CXXFLAGS="-std=c++$CXX_STD_TEST"fiCXXFLAGS="$SAVED_CXXFLAGS $OPENEXR_CFLAGS $OPENEXR_CXXFLAGS"CXXFLAGS="$SAVED_CXXFLAGS -fexceptions $OPENEXR_CFLAGS $OPENEXR_CXXFLAGS"Here it seems to be a good idea to use the variable:
$(USE_EXCEPTIONS)which is set according to whether the compiler provides such a flag.Amended
48e40fb6bbtoaa1e0c376e3 years agoaa1e0c376eto085b8b6c983 years agoSuccessfully tested building on Debian 7.x (Wheezy) and Ubuntu 21.10 (Impish).
Builds without problems on Slackware64-current.
085b8b6c98into master 3 years agoThanks both for testing!
Reviewers
085b8b6c98.