New user self-registration is currently disabled. Please email eigen-core-team @ lists.tuxfamily.org if you need an account.
Bug 1167 - Use CMake INSTALL(DIRECTORY ... ) instead of manual, recursive INSTALL(FILES ...)
Use CMake INSTALL(DIRECTORY ... ) instead of manual, recursive INSTALL(FILES ...
Status: RESOLVED FIXED
Product: Eigen
Classification: Unclassified
Component: Core - general
3.3 (current stable)
All All
: Normal Internal Design
Assigned To: Nobody
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2016-02-15 14:44 UTC by Christoph Hertzberg
Modified: 2016-08-29 09:02 UTC (History)
4 users (show)



Attachments

Description Christoph Hertzberg 2016-02-15 14:44:39 UTC
Installation of the Eigen source tree is currently done manually/recursively instead of by using the CMake feature INSTALL(DIRECTORY ...)
Is there a (historic) reason for that? It seems this is supported at least since CMake 2.6?
Comment 1 Gael Guennebaud 2016-02-15 20:50:30 UTC
Good point, I don't see any reason to motivate the current state, except that the feature was probably missing when we started. So I'm OK to simplify that. This simplified solution is also more future proof.
Comment 2 Gael Guennebaud 2016-08-29 09:02:39 UTC
https://bitbucket.org/eigen/eigen/commits/289a5c2d13d4/

Note You need to log in before you can comment on or make changes to this bug.