lorenzo-rovigatti / oxDNA

A new version of the code to simulate the oxDNA/oxRNA models, now equipped with Python bindings
https://dna.physics.ox.ac.uk/
GNU General Public License v3.0
43 stars 28 forks source link

Clean up include paths for pybind11, python_defs. #37

Closed cgevans closed 2 years ago

cgevans commented 2 years ago

In BaseBox.h, the include path for python_defs.h actually goes up one directory too high; this is obscured by the way pybind11 is included, but fails if a system pybind11 is used.

In ConfigInfo.h, direct relative paths to pybind11 headers are used, but cmake already includes the pybind11 include path, so standard paths can be used instead. This simplifies use of system pybind11 headers (by a one-line cmake change) where necessary, for example, to fit Debian packaging policies.

Neither change should have any effect on the normal build process.