symforce-org / symforce

Fast symbolic computation, code generation, and nonlinear optimization for robotics
https://symforce.org
Apache License 2.0
1.44k stars 147 forks source link

Fix install paths #149

Closed aaron-skydio closed 2 years ago

aaron-skydio commented 2 years ago

Previously we were installing all our headers into include, instead of preserving the directory structure in there. This breaks lots of things, for instance we create an assert.h that'll break things attempting to include the stdlib assert.h.

This preserves directory structure correctly when running cmake --build --target install (and therefore also pip install ., which calls cmake). Tested locally in a conda environment.