Bug 475617

Summary: Fixed needed to compile krita under Fedora
Product: [Applications] krita Reporter: Dov Grobgeld <dov.grobgeld>
Component: GeneralAssignee: Krita Bugs <krita-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: dimula73, halla
Priority: NOR    
Version First Reported In: git master (please specify the git hash!)   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Dov Grobgeld 2023-10-14 17:37:52 UTC
SUMMARY

When trying to compile krita commit f35274b26c1eeab410ba7b82d62d156107f6f46f under Fedora 37, I had to apply the following patch to compile:

```
diff --git a/3rdparty_vendor/raqm/CMakeLists.txt b/3rdparty_vendor/raqm/CMakeLists.txt
index 3d42fe186b..6518c83b36 100644
--- a/3rdparty_vendor/raqm/CMakeLists.txt
+++ b/3rdparty_vendor/raqm/CMakeLists.txt
@@ -86,6 +86,7 @@ set(libraqm_SRC
 
 add_library(libraqm STATIC ${libraqm_SRC})
 set_property(TARGET libraqm PROPERTY C_STANDARD 99)
+set_property(TARGET libraqm PROPERTY POSITION_INDEPENDENT_CODE ON)
 
 target_include_directories(libraqm INTERFACE ${libraqm_src_SOURCE_DIR}/src)
 target_include_directories(libraqm PUBLIC ${CMAKE_CURRENT_BINARY_DIR})
```

Meta question:

- Should I have opened a merge request?
- Should I attach the patch as an attachment?
Comment 1 Halla Rempt 2023-10-15 10:38:33 UTC
A merge request makes things really easy :-)
Comment 2 Dmitry Kazakov 2025-04-22 16:35:35 UTC
This patch is not needed anymore since we anavbled PIC in the entire Krita lately:
https://invent.kde.org/graphics/krita/-/commit/0d508c09637c361aa16658f371961e0bb48aeec2
Comment 3 Dmitry Kazakov 2025-04-22 16:38:41 UTC
Git commit e62c63696b3967e5f390592fa3c7a6064ffee344 by Dmitry Kazakov.
Committed on 22/04/2025 at 16:38.
Pushed by dkazakov into branch 'krita/5.2'.

Enable PIC for our local raqm builds

M  +1    -0    3rdparty_vendor/raqm/CMakeLists.txt

https://invent.kde.org/graphics/krita/-/commit/e62c63696b3967e5f390592fa3c7a6064ffee344