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

Fix "thirdparty" folder #95

Closed GPMueller closed 4 years ago

GPMueller commented 5 years ago

Closes #79.


Some thoughts about the thirdparty folder:

It would be nice to automatically treat "thirdparty" folders, removing the need for a build config for many projects, if the dependencies inside also adhere to a reasonable folder structure. In any case, this should only be done if no build configuration was given.

This could be done by assuming a folder structure "thirdparty/targetname" etc., but this might be too unstable with respect to minor alterations of the folder structure. More cases could be covered without build config if this were also done for the root folder, which could contain e.g. "myexe" and "mylib" folders corresponding to two respective targets.

codecov[bot] commented 5 years ago

Codecov Report

Merging #95 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #95   +/-   ##
=======================================
  Coverage   80.91%   80.91%           
=======================================
  Files          12       12           
  Lines         985      985           
=======================================
  Hits          797      797           
  Misses        188      188
Impacted Files Coverage Δ
clang_build/io_tools.py 100% <100%> (ø) :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 762e6b8...840891c. Read the comment docs.