@simonmichal requested changes on this pull request.
> @@ -29,6 +29,7 @@ include( XrdXml ) include( XrdPosix ) include( XrdFfs ) include( XrdPlugins ) +include( XrdSsi )
Do we want to include XrdSsi in 4.7.0? If not this needs to be optional (similarly as for http and cepth).
> @@ -9,6 +9,8 @@ set( LIB_XRD_N2NO2P XrdN2No2p-${PLUGIN_VERSION} ) set( LIB_XRD_PSS XrdPss-${PLUGIN_VERSION} ) set( LIB_XRD_GPFS XrdOssSIgpfsT-${PLUGIN_VERSION} ) set( LIB_XRD_ZCRC32 XrdCksCalczcrc32-${PLUGIN_VERSION} ) +set( LIB_XRD_SSI XrdSsi-${PLUGIN_VERSION} )
IMHO it would be better to keep all XrdSsi cmake staff in XrdSsi.cmake. Adding it partially to XrdPlugins.cmake is misleading and clutters the file.
> + XrdSsi/XrdSsiDir.cc XrdSsi/XrdSsiDir.hh + XrdSsi/XrdSsiFile.cc XrdSsi/XrdSsiFile.hh + XrdSsi/XrdSsiFileReq.cc XrdSsi/XrdSsiFileReq.hh + XrdSsi/XrdSsiFileSess.cc XrdSsi/XrdSsiFileSess.hh + XrdSsi/XrdSsiSfs.cc XrdSsi/XrdSsiSfs.hh + XrdSsi/XrdSsiSfsConfig.cc XrdSsi/XrdSsiSfsConfig.hh + XrdSsi/XrdSsiStat.cc +) + +target_link_libraries( + ${LIB_XRD_SSI} + XrdSsiLib + XrdUtils + XrdServer ) + +set_target_properties(
The library is not versioned!
> +#------------------------------------------------------------------------------- +# The XrdSsiLog lib +#------------------------------------------------------------------------------- +add_library( + ${LIB_XRD_SSILOG} + SHARED + XrdSsi/XrdSsiLogging.cc +) + +target_link_libraries( + ${LIB_XRD_SSILOG} + XrdSsiLib + XrdUtils + XrdServer ) + +set_target_properties(
The library is not versioned!
> + XrdSsi/XrdSsiDir.cc XrdSsi/XrdSsiDir.hh + XrdSsi/XrdSsiFile.cc XrdSsi/XrdSsiFile.hh + XrdSsi/XrdSsiFileReq.cc XrdSsi/XrdSsiFileReq.hh + XrdSsi/XrdSsiFileSess.cc XrdSsi/XrdSsiFileSess.hh + XrdSsi/XrdSsiSfs.cc XrdSsi/XrdSsiSfs.hh + XrdSsi/XrdSsiSfsConfig.cc XrdSsi/XrdSsiSfsConfig.hh + XrdSsi/XrdSsiStat.cc +) + +target_link_libraries( + ${LIB_XRD_SSI} + XrdSsiLib + XrdUtils + XrdServer ) + +set_target_properties(
Also, the library is installed but is not packaged.
Note this creates libXrdSsi.so which is installed but not packaged. libXrdSsi.so.* are, in turn not created but they are packaged.
In packaging/rhel/xrootd.spec.in:
> @@ -733,6 +733,10 @@ fi %{_libdir}/libXrdN2No2p-4.so %{_libdir}/libXrdOssSIgpfsT-4.so %{_libdir}/libXrdServer.so.* +%{_libdir}/libXrdSsi-4.so.*
Are we sure this should go to server-libs, and not to a new package, .e.g. ssi-libs ?
Actually, since libXrdSsi is not versioned this fails as there are no libXrdSsi-4.si.*, there is only the libXrdSsi-4.so, which in turn is installed but not packaged.
In packaging/rhel/xrootd.spec.in:
> @@ -733,6 +733,10 @@ fi %{_libdir}/libXrdN2No2p-4.so %{_libdir}/libXrdOssSIgpfsT-4.so %{_libdir}/libXrdServer.so.* +%{_libdir}/libXrdSsi-4.so.* +%{_libdir}/libXrdSsiLib.so.* +%{_libdir}/libXrdSsiLog-4.so.*
Same here, since libXrdSsiLog is not versioned this fails as there are no libXrdSsiLog-4.si.*, there is only the libXrdSsiLog-4.so, which in turn is installed but not packaged.
> @@ -125,6 +125,18 @@ set( XROOTD_PRIVATE_HEADERS XrdOfs/XrdOfsHandle.hh XrdOfs/XrdOfsTrace.hh XrdOfs/XrdOfsTPCInfo.hh + XrdSsi/XrdSsiAtomics.hh
Are we sure we don't want this in a separate package, say ssi-devel (together with *.so files) ?
> +#------------------------------------------------------------------------------- +# The XrdSsiLog lib +#------------------------------------------------------------------------------- +add_library( + ${LIB_XRD_SSILOG} + SHARED + XrdSsi/XrdSsiLogging.cc +) + +target_link_libraries( + ${LIB_XRD_SSILOG} + XrdSsiLib + XrdUtils + XrdServer ) + +set_target_properties(
Also, the library is installed but is not packaged.
Note this creates libXrdSsiLog.so which is installed but not packaged. libXrdSsiLog.so.* are, in turn not created but they are packaged.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1