Skip to content

Remove flag '-std=gnu++11'

Andrei Pavel requested to merge remove-cpp11 into master

Packaging failed after a recent merge: https://jenkins.aws.isc.org/job/kea-dev/job/pkg/1686/. One of the causes was the -std=gnu++11 flag. It is only passed for rpm. This also contradicts the docs which say that C++14 is required.

I suggest we remove the flag, and run with the compiler default, which, for our systems, should be either C++14, C++17, or C++20, based on how new the default compiler is on that system.

There is also -Wno-deprecated-declarations passed which hides some warnings. There is no reason to do that as far as I can see. For debian, we go the other way, and provide -Wall -pedantic for more warnings.

Ran here: https://jenkins.aws.isc.org/job/kea-manual/job/pkg/443/.

Merge request reports