[CDRIVER-3495] Split option to BUILD / INSTALL static libraries Created: 18/Jan/20 Updated: 28/Oct/23 Resolved: 29/Feb/20 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | libmongoc |
| Affects Version/s: | 1.16.0 |
| Fix Version/s: | 1.17.0-beta, 1.17.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Remi Collet | Assignee: | Roberto Sanchez |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Fedora / RHEL / CentOS |
||
| Comments |
| Comment by Githook User [ 29/Feb/20 ] |
|
Author: {'username': 'rcsanchez97', 'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com'}Message: |
| Comment by Remi Collet [ 18/Jan/20 ] |
|
Context: during RPM build we may need to build examples, tests and thus static libraries, but don't want to install these static libraries as not allowed (with exception) for downstream distribution.
For now, the ENABLE_STATIC is mandatory with ENABLE_TEST
Will be nice to have 2 options, ENABLE_BUILD_STATIC and ENABLE_INSTALL_STATIC (only add a condition for the "install" directive)
More, when built / installed, the cmake files contains references to the static libraries, and thus is unusable if not available.
Also see https://github.com/mongodb/libmongocrypt/pull/87 where the second commit add such an option for libmongocrypt
P.S. test suite is disabled in RPM build as the mongo server is no more available, so this is not a blocker for us, so only a idea for improvement. |