Repository navigation
CMakeLists.txt: purpose of the CPACK_STRIP_FILES #433
Description
The CMakeLists.txt contains the line:
set(CPACK_STRIP_FILES "${CMAKE_INSTALL_LIBDIR}/libadldap.so;${CMAKE_INSTALL_BINDIR}/admc")
From the description of the option https://cmake.org/cmake/help/latest/module/CPack.html#variable:CPACK_STRIP_FILES
it looks like it removes debug symbols from the so and executable.
In the description it's said that the option now can be just a boolean TRUE.
But do we really need the option at all?
Activity
artyom-poptsov commentedon Jun 23, 2026
Hello @yurtpage !
Why not? It is a common practice to strip extra information from binary files during installation, to make them smaller. Smaller binaries make for less memory footprint both in the storage in RAM.
In some distributions you can install debug information as separate packages. ALT Linux distribution is one of such distributions:
https://www.altlinux.org/Debuginfo
yurtpage commentedon Jun 23, 2026
Then it can be changed to
TRUEfor simplicity.I don't remember if I had some warnings on this or any problems. But usually packages don't use the variable because the package builder strips everything itself
https://chatgpt.com/s/t_6a3aa7514cfc81918c662a43e0e8bb37
artyom-poptsov commentedon Jun 24, 2026
Hello @yurtpage !
Good point, I checked the CMake/CPack documentation and indeed since CMake version 2.6 we can set
CPACK_STRIP_FILEStoTRUE:https://cmake.org/cmake/help/latest/module/CPack.html
Since the minimum required version of CMake for ADMC is 3.16, we can safely replace the list of files with
TRUEin the setting ofCPACK_STRIP_FILESvariable.artyom-poptsov commentedon Jun 24, 2026
I've made a PR that fixes this: https://altlinux.space/alt-domain/admc/pulls/8
yurtpage commentedon Jun 24, 2026
Thank you!
build(CMakeLists.txt): Set "CPACK_STRIP_FILES" to "TRUE"