[SofaKernel/CMakeLists.txt] Set the default compilation mode to c++17.

[damienmarchal]

Because most compiler handle it properly and we are in 2020.

---

This PR:

• builds with SUCCESS for all platforms on the CI.
• does not generate new warnings.
• does not generate new unit test failures.
• does not generate new scene test failures.
• does not break API compatibility.
• is more than 1 week old (or has fast-merge label).

Reviewers will merge only if all these checks are true.

[fredroy]

Err... Should we not wait for the CI to support c++17 (aka VS2017/2019) ?

[damienmarchal]

No, we should upgrade the CI to support this PR :)

[hugtalbot]

Update documentation (supported version) : see pages
and make sure that the CI is following these (Guillaume will work on this)

[damienmarchal]

Hi all,

What's is the progress in this ?

[hugtalbot]

We wait for CI environment, and Guillaume is actively working on it

[hugtalbot]

FYI external plugins can already be compiled with compilation mode c++17

[damienmarchal]

Thanks for the answer,

Actually we are using c++17 in our plugins since years.

The problem is that not having sofa supporting c+17 means that when we want to make PRs to integrate some features into the sofa master we have to convert all the code to c++11. As it was agreed to allow Sofa to support c++17 a year ago I think it would be nice to have it asap, as it would free the contributors from reverting their codebase to c++11 to make PRs.

So good luck to Guillaume.

[guparan]

[ci-build][force-full-build][with-all-tests]

[guparan]

CI is clear now.
This PR triggers following error:

applications/plugins/SofaSphFluid/src/SofaSphFluid/SPHFluidForceField.inl:34:10: fatal error: execution: No such file or directory
 #include <execution>
          ^~~~~~~~~~~
compilation terminated.

see https://ci.inria.fr/sofa-ci-dev/job/sofa-framework/job/PR-1249/CI_CONFIG=ubuntu_gcc-5.4,CI_PLUGINS=options,CI_TYPE=release/7/console

[damienmarchal]

Hi,
What about the link error to ARTrack on windows ?

[guparan]

I can't push on your branch. Did you forbid it?
Removing using namespace std; in applications/plugins/ARTrack/extlibs/ARTrackLib/mainTracker.cpp does fix the error on Windows.

[guparan]

@damienmarchal Did you see my comment?

[damienmarchal]

I didn't do anything special on the branch

[damienmarchal]

@guparan Any idea of the problem in windows ?

[damienmarchal]

[ci-build][with-regression-tests][with-scene-tests]

[hugtalbot]

[ci-build][with-all-tests]