Trick-17 / clang-build

Clang-based cross platform build system written in Python
https://clang-build.readthedocs.io
MIT License
8 stars 3 forks source link

Build Type #92

Closed NOhs closed 4 years ago

NOhs commented 5 years ago

So we have the option with clang-build to select the build type. But I don't think it has to be a target attribute. As the build type is kind of "global" per run as you do not compile a project in parts with debug and in parts with release options.

GPMueller commented 5 years ago

Agreed, this should simply be an environment option.

GPMueller commented 5 years ago

I just realised that this is already a property of the Environment class, and the same goes for clang, clangpp and force_build.

Also, instead of passing files['headers'] etc. to the targets, we could simply pass the files dict (note the HeaderOnly targets currently don't get the 'sourcefiles' part of the dict). These changes might shorten a lot of code.