Code Merge Request #4: Fiy CMake Python detection (open)

This fixes the Python detection if CMake is configured for MSVC.

[Mathias Fröhlich]

Thanks so far.

But, I am not going to merge that as is. There are lots of unrelated changes packed into this commit. I have already picked the typos. Also the upcase/downcase change that I pused some time ago is partially undone.
If I merge something I want to be able to review the change without being distracted to unrelated changes. So, having commits that focus on exactly one topic would be great. That also opens the opportunity to revert that specific topic commit in case it turns out problematic without sacreficing may be usefull changes within the same munged together commit.

I am not exactly sure what you are fixing. Is this more related to the version check or more related that the wrong python dll is liked to the rti python binding module?

best
Mathias

[tbeu]

Thanks for the feedback. Glad to see that this repo is alive. I will provide a new PR with dedicated CMake fixes (there is one more linker issue in MSVC when building the import libs for the python libs).

[tbeu]

With the new master I now get the message

Could NOT find Python: Found unsuitable major version "3", but required major version is exact version "2"

Python 3.7 is what I have installed and what is detected.

[Mathias Fröhlich]

With the current git master, there is an other change that should improve the selection of the python version to be used for the binding.