search

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

Made contradicting cla's mutually exclusive. #103

Closed NOhs closed 4 years ago

NOhs commented 4 years ago

For the command-line arguments of clang-build one can specify to build "ALL" targets, meaning those targets not needed by the top-level project are still selected to be build. Or one can select those targets one wishes to build (and all their dependencies). It doesn't make sense to provide both. With this patch, the user will be informed that they provided contradicting arguments.

codecov[bot] commented 4 years ago

Codecov Report

Merging #103 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #103      +/-   ##
==========================================
+ Coverage   80.94%   80.96%   +0.01%     
==========================================
  Files          12       12              
  Lines         971      972       +1     
==========================================
+ Hits          786      787       +1     
  Misses        185      185
Impacted Files Coverage Δ
clang_build/clang_build.py 69.31% <100%> (+0.16%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 12ea8c2...c17ce0a. Read the comment docs.