Details
-
Bug
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
10.2(EOL)
-
None
Description
Building the server package with cmake parameter -DINSTALL_LAYOUT=DEB will result in wrong installation directories for Connector/C.
How to fix:
--- a/cmake/mariadb_connector_c.cmake
|
+++ b/cmake/mariadb_connector_c.cmake
|
@@ -24,6 +24,8 @@ SET(CONC_WITH_MYSQLCOMPAT ON)
|
|
IF (INSTALL_LAYOUT STREQUAL "RPM")
|
SET(CONC_INSTALL_LAYOUT "RPM")
|
+ELSEIF (INSTALL_LAYOUT STREQUAL "DEB")
|
+ SET(CONC_INSTALL_LAYOUT "DEB")
|
ELSE()
|
SET(CONC_INSTALL_LAYOUT "DEFAULT")
|
ENDIF()
|
Attachments
Issue Links
Activity
It does not work because in libmariadb/install.cmake there is at the bottom the section that starts like this:
#
|
# Overwrite defaults
|
#
|
...
|
IF(INSTALL_PLUGINDIR)
|
SET(INSTALL_PLUGINDIR_${INSTALL_LAYOUT} ${INSTALL_PLUGINDIR})
|
ENDIF()
|
The server passes a value for INSTALL_PLUGINDIR, so the INSTALL_PLUGINDIR_DEB has no function.
Fixing PR: https://github.com/MariaDB/server/pull/1468/commits
Follow-up PR: https://github.com/MariaDB/server/pull/1478
This was closed via https://github.com/MariaDB/server/commit/4bc31a904f22290801075deed949733ba2133974
You can't change this on 10.2, that would change the file paths and break packaging in stable releases 10.2, 10.3 and 10.4. This is OK to do in 10.5 and I am already working on it. However, this change does not seem to be enough, I'll post you more details later.. Assigned to myself, no point in Sergei putting time on this now when he did not have time to do this in the past month. I have enough info from Georg to complete this.