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
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"
fi
CXXFLAGS="$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
48e40fb6bb
toaa1e0c376e
3 years agoaa1e0c376e
to085b8b6c98
3 years agoSuccessfully tested building on Debian 7.x (Wheezy) and Ubuntu 21.10 (Impish).
Builds without problems on Slackware64-current.
085b8b6c98
into master 3 years agoThanks both for testing!
Reviewers
085b8b6c98
.