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

Python platform support #112

Closed GPMueller closed 4 years ago

GPMueller commented 4 years ago

This PR implements some additional platform-parameters that can be used to locate the Python headers and library, as well as building binary extension modules. A test using pybind11 is included and integrated in the test-suite.

Closes #43.

GPMueller commented 4 years ago

Remaining problems running test_pybind11 on CI:

Note the include and library directories on the Ubuntu CI:

Python headers in:   /home/travis/virtualenv/python3.8.1/include/python3.8
Python library in:   /home/travis/virtualenv/python3.8.1/lib
GPMueller commented 4 years ago

I'm not particularly happy with the solution for the AppVeyor problem, but since it's only in the context of test cleanup I suppose it's fine for now. This PR should therefore be ready for merging now.

codecov[bot] commented 4 years ago

Codecov Report

Merging #112 into master will increase coverage by 1.51%. The diff coverage is 97.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
+ Coverage   79.03%   80.54%   +1.51%     
==========================================
  Files          17       17              
  Lines        1197     1218      +21     
==========================================
+ Hits          946      981      +35     
+ Misses        251      237      -14
Impacted Files Coverage Δ
clang_build/platform.py 98.41% <96.66%> (+0.54%) :arrow_up:
clang_build/compiler.py 69.81% <100.00%> (+1.81%) :arrow_up:
clang_build/target.py 81.84% <100.00%> (+1.96%) :arrow_up:
clang_build/single_source.py 61.90% <0.00%> (+12.69%) :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 88350d2...4d842c2. Read the comment docs.