Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package kdelibs4support for openSUSE:Factory 
checked in at 2022-03-09 18:46:44
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/kdelibs4support (Old)
 and      /work/SRC/openSUSE:Factory/.kdelibs4support.new.2349 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "kdelibs4support"

Wed Mar  9 18:46:44 2022 rev:101 rq:960081 version:5.91.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/kdelibs4support/kdelibs4support.changes  
2022-02-24 18:21:36.362687914 +0100
+++ 
/work/SRC/openSUSE:Factory/.kdelibs4support.new.2349/kdelibs4support.changes    
    2022-03-11 11:45:07.974867797 +0100
@@ -1,0 +2,6 @@
+Mon Mar  7 11:58:31 UTC 2022 - Christophe Giboudeaux <[email protected]>
+
+- Add patch to fix installation:
+  * 0001-Use-KDE_INSTALL_FULL_-variables-where-needed.patch
+
+-------------------------------------------------------------------

New:
----
  0001-Use-KDE_INSTALL_FULL_-variables-where-needed.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ kdelibs4support.spec ++++++
--- /var/tmp/diff_new_pack.pSwe6B/_old  2022-03-11 11:45:08.518868407 +0100
+++ /var/tmp/diff_new_pack.pSwe6B/_new  2022-03-11 11:45:08.522868412 +0100
@@ -35,6 +35,8 @@
 Source1:        %{name}-%{version}.tar.xz.sig
 Source2:        frameworks.keyring
 %endif
+# PATCH-FIX-UPSTREAM
+Patch0:         0001-Use-KDE_INSTALL_FULL_-variables-where-needed.patch
 BuildRequires:  NetworkManager-devel
 BuildRequires:  extra-cmake-modules >= %{_kf5_bugfix_version}
 BuildRequires:  fdupes

++++++ 0001-Use-KDE_INSTALL_FULL_-variables-where-needed.patch ++++++
>From 5924302486a72041519f05aa3bccc3a0e7d7ca3e Mon Sep 17 00:00:00 2001
From: Christophe Giboudeaux <[email protected]>
Date: Mon, 7 Mar 2022 12:57:35 +0100
Subject: [PATCH] Use KDE_INSTALL_FULL_* variables where needed

There's no guarantee KDE_INSTALL_* variables are relative. This can cause 
errors when creating symlinks after installation.
---
 src/CMakeLists.txt | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index c5abb2a6..2096b355 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -595,12 +595,12 @@ if (UNIX)
    install(PROGRAMS kio/fileshareset DESTINATION ${KDE_INSTALL_LIBEXECDIR_KF5} 
)
 
    install(CODE "
-     set(FILESHARESET_PATH 
\"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${KDE_INSTALL_LIBEXECDIR_KF5}/fileshareset\")
+     set(FILESHARESET_PATH 
\"\$ENV{DESTDIR}/${KDE_INSTALL_FULL_LIBEXECDIR_KF5}/fileshareset\")
         EXECUTE_PROCESS(COMMAND sh -c \"chown 0 '\${FILESHARESET_PATH}' && 
chmod u+s '\${FILESHARESET_PATH}'\")
    ")
 
    # write a cmake script file which creates the symlink
-   file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/make_fileshare_symlink.cmake 
"exec_program(${CMAKE_COMMAND} ARGS -E create_symlink fileshareset 
\"\$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}/${KDE_INSTALL_LIBEXECDIR_KF5}/filesharelist\")\n")
+   file(WRITE ${CMAKE_CURRENT_BINARY_DIR}/make_fileshare_symlink.cmake 
"exec_program(${CMAKE_COMMAND} ARGS -E create_symlink fileshareset 
\"\$ENV{DESTDIR}/${KDE_INSTALL_FULL_LIBEXECDIR_KF5}/filesharelist\")\n")
    # and add it as post-install script to any of the installed targets, so it 
will be executed during "make install"
    set_target_properties(KF5KDELibs4Support PROPERTIES POST_INSTALL_SCRIPT 
${CMAKE_CURRENT_BINARY_DIR}/make_fileshare_symlink.cmake)
 endif ()
@@ -645,7 +645,7 @@ if(NOT WIN32)
     install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${_custom_dtd_kdex}
         DESTINATION ${KDE_INSTALL_DATADIR_KF5}/kdoctools/customization/dtd )
 else()
-    get_filename_component(NORMALIZED_DTD_INSTALL_PATH 
"${CMAKE_INSTALL_PREFIX}/${KDE_INSTALL_DATADIR_KF5}/kdoctools/customization/dtd"
 ABSOLUTE)
+    set(NORMALIZED_DTD_INSTALL_PATH 
"${KDE_INSTALL_FULL_DATADIR_KF5}/kdoctools/customization/dtd")
     # file(RELATIVE_PATH ...) returns wrong values for paths containing /../ 
(it doesn't normalize the path)
     # To work around this we make sure the variable passed has all /../ 
elements
     # removed using get_filename_component(... ABSOLUTE)
-- 
2.35.1

Reply via email to