search

MRChemSoft / mrchem

MultiResolution Chemistry
GNU Lesser General Public License v3.0
27 stars 21 forks source link

mrchem 1.1.x do not install libraries any more #448

Open susilehtola opened 1 year ago

susilehtola commented 1 year ago

The newer versions of mrchem install development headers, but have added OBJECT in 1a47fe1 which prevents the creation of distributable libraries. This is an issue for Linux distributions.

Moreover, it would be nice to have a standard sonamed shared library.

stigrj commented 1 year ago

Any thoughts @robertodr ?

robertodr commented 1 year ago

The thinking was that the library was only used to link to mrchem.x and nowhere else. Plus, there's no real API for MRChem and distributing a DSO for it didn't/doesn't really make sense. The development headers should not be installed, that CMake code is a leftover.