Module: kamailio
Branch: master
Commit: 58bd1b995a1bf36eb1efebbd2e091e0dc9ca716c
URL: https://github.com/kamailio/kamailio/commit/58bd1b995a1bf36eb1efebbd2e091e0…
Author: Xenofon Karamanos <xk(a)gilawa.com>
Committer: Xenofon Karamanos <xk(a)gilawa.com>
Date: 2024-12-19T17:49:13Z
cmake: Fix modules suffix to .so for all OS
---
Modified: src/modules/CMakeLists.txt
---
Diff: https://github.com/kamailio/kamailio/commit/58bd1b995a1bf36eb1efebbd2e091e0…
Patch: https://github.com/kamailio/kamailio/commit/58bd1b995a1bf36eb1efebbd2e091e0…
---
diff --git a/src/modules/CMakeLists.txt b/src/modules/CMakeLists.txt
index fe13a88f537..a1b474a4502 100644
--- a/src/modules/CMakeLists.txt
+++ b/src/modules/CMakeLists.txt
@@ -65,6 +65,8 @@ function(add_module_group group_modules)
add_subdirectory(${module_path})
# Remove the 'lib' prefix from the module name
set_target_properties(${module_name} PROPERTIES PREFIX "")
+ # Set suffix to .so always
+ set_target_properties(${module_name} PROPERTIES SUFFIX ".so")
# Add this policy to new so we can use target_link_libraries(... PRIVATE
# ...) to link against targets defined in children directories
It is about the comment:
- https://github.com/kamailio/kamailio/commit/28f4089efcc9c2ccf8c79fc9ecc509a…
Besides the first line there, the rest is common and can be put in a file (template) and included. But as I wrote there, it might not be the best option. I am fine to go for alternatives considered better.
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/commit/4bf2c9634ab19ac76c12c33ddc3d422…
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/commit/4bf2c9634ab19ac76c12c33ddc3d4220631d38ef/150535853(a)github.com>
Do we expect a local kamailio to use /var/run/kamailio as it's `run` directory or use `build_folder/run`? if then, then kamailio must be run as `sudo` since it affects root files and folders.
If you do not provide it the default prefix is `/usr/local`, so even if its not installed it will use that when running it, so it must therefore have sudo access and the directory already created (I think).
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/commit/1860420fea7ed57d25bc7d735b46c11…
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/commit/1860420fea7ed57d25bc7d735b46c11c739aa0c4/150535640(a)github.com>