artivis / manif

A small C++11 header-only library for Lie theory.

https://artivis.github.io/manif

MIT License

1.47k stars 242 forks source link

Add installation of pip metadata files for when manifpy python bindings are installed only via CMake #243

Closed traversaro closed 2 years ago

traversaro commented 2 years ago

Hi @artivis !

This PR adds a bit of CMake code to ensure that even if you just install manifpy Python bindings via CMake and adding the correct installation directory to PYTHONPATH, the resulting installation is correctly listed by pip list.

codecov[bot] commented 2 years ago

Codecov Report

Merging #243 (d9e438a) into devel (ab560a3) will decrease coverage by 0.00%. The diff coverage is n/a.

@@            Coverage Diff             @@
##            devel     #243      +/-   ##
==========================================
- Coverage   98.17%   98.17%   -0.01%     
==========================================
  Files          55       55              
  Lines        1754     1753       -1     
==========================================
- Hits         1722     1721       -1     
  Misses         32       32

artivis commented 2 years ago

lgtm, thanks @traversaro !

traversaro commented 2 years ago

Thanks @artivis !