View Issue Details

IDProjectCategoryView StatusLast Update
0001129bareos-coreGeneralpublic2023-07-20 17:23
Reporterrjung Assigned Tobruno-at-bareos  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
PlatformSolaris10OSSolarisOS Version10
Product Version18.2.6 
Summary0001129: OpenSSL and readline include path missing in cmake build
DescriptionActually version is 18.4.1, but that is missing in the mantis version dropdown.

The new cmake based compilation correctly detects non-system OpenSSl and readline, when run e.g. with flags

      -DOPENSSL_ROOT_DIR=/path/to/openssl
      -DReadline_ROOT_DIR=/path/to/readline

But the when running maake, it can't find the header files, because the respective include paths never get added.

As a workaround I was using the follwoing patch:

--- core/CMakeLists.txt Fri Sep 28 10:30:36 2018
+++ core/CMakeLists.txt Sun Nov 3 14:55:00 2019
@@ -435,6 +435,10 @@
    set(HAVE_TLS "1")
 ENDIF()

+IF( "${HAVE_OPENSSL}")
+include_directories(${OPENSSL_INCLUDE_DIR})
+ENDIF()
+
 IF(NOT openssl)
    unset(HAVE_OPENSSL)
    unset(HAVE_TLS)
@@ -446,6 +450,7 @@
 set(got_readline "${READLINE_FOUND}" )
 if ("${READLINE_FOUND}")
    set(HAVE_READLINE 1)
+ include_directories(${Readline_INCLUDE_DIR})
 endif()

 if ("${PAM_FOUND}")

Steps To ReproduceRun cmake against a non-system OpenSSl and readline and make sure, the OpenSSl and readline header files are not installed in teh default system header file locations. The run make.
Tagscompile cmake headers openssl readline

Activities

bruno-at-bareos

bruno-at-bareos

2023-07-20 17:23

manager   ~0005243

Seems that actual version have those options available.
Please test with fresh code and report if not working.

Issue History

Date Modified Username Field Change
2019-11-03 18:57 rjung New Issue
2019-11-03 18:57 rjung Tag Attached: compile cmake headers openssl readline
2023-07-20 17:23 bruno-at-bareos Assigned To => bruno-at-bareos
2023-07-20 17:23 bruno-at-bareos Status new => closed
2023-07-20 17:23 bruno-at-bareos Resolution open => fixed
2023-07-20 17:23 bruno-at-bareos Note Added: 0005243