search

the-butterfly-effect / tbe

The Butterfly Effect, a realistic physics simulation game
http://the-butterfly-effect.org
GNU General Public License v2.0
96 stars 13 forks source link

Some improvements to the CMake build script #358

Closed ratijas closed 2 years ago

ratijas commented 3 years ago

Win/Apple/Linux if-else structure is partially based on Telegram scripts1. Also, duplicating opening argument inside closing else/endif/etc. is considered legacy by CMake documentation, and thus better be avoided.

Changed home page URL from multi-level escaped backslash insanity to "http://" with forward slashes.

Changed condition if ("${RPM}") to if (RPM), since later is a special form of a basic expression in CMake condition syntax2. This is pretty much what if (WIN32) et al. are all about.

Some install() commands with single source argument were inlined.

Some lines were indented with tabs. They were replaced with spaces.

ratijas commented 3 years ago

You may or may not like the change. I was just practicing my CMake skills. Anyway, hope you'll enjoy it!

ratijas commented 2 years ago

Closing due to inactivity and also to clean up my list of opened pull request.