[SeExpr] [Bug 427651] Static linking of LLVM libs breaks krita

2020-11-12 Thread amyspark
https://bugs.kde.org/show_bug.cgi?id=427651

amyspark  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/grap
   ||hics/kseexpr/-/commit/b57c3
   ||236f3c540b81d44c7ac77d77869
   ||f35d7c1a
 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are watching all bug changes.

[SeExpr] [Bug 427651] Static linking of LLVM libs breaks krita

2020-10-14 Thread amyspark
https://bugs.kde.org/show_bug.cgi?id=427651

--- Comment #3 from amyspark  ---
FTR, linking against shared LLVM makes it impossible to use in systems that
provide only static LLVM (like the Academy Software Foundation's Docker images,
which I use for testing my changes in CI). 
I'm checking if the CMake modules provides any tooling to dynamically detect
and use the shared lib if available (otherwise, I'll have to use
llvm-config...)

-- 
You are receiving this mail because:
You are watching all bug changes.

[SeExpr] [Bug 427651] Static linking of LLVM libs breaks krita

2020-10-14 Thread amyspark
https://bugs.kde.org/show_bug.cgi?id=427651

--- Comment #2 from amyspark  ---
This is correct. The SeExpr fork we use in Krita does not involve LLVM at all.

In the next point release, I will make it build without LLVM support by
default.

In any case, I will have a look at this and make sure it works correctly.

-- 
You are receiving this mail because:
You are watching all bug changes.

[SeExpr] [Bug 427651] Static linking of LLVM libs breaks krita

2020-10-14 Thread Boudewijn Rempt
https://bugs.kde.org/show_bug.cgi?id=427651

Boudewijn Rempt  changed:

   What|Removed |Added

 CC||b...@valdyas.org
 Status|REPORTED|ASSIGNED
   Assignee|krita-bugs-n...@kde.org |a...@amyspark.me
 Ever confirmed|0   |1

--- Comment #1 from Boudewijn Rempt  ---
Afaik, seexpr for Krita should be built with the following flags:
-DENABLE_LLVM_BACKEND=OFF -DENABLE_SSE4=OFF -DBUILD_DOC=OFF -DBUILD_TESTS=OFF
-DBUILD_UTILS=OFF -DUSE_PYTHON=OFF

so, that disables llvm and python.

Amyspark, is that correct?

-- 
You are receiving this mail because:
You are watching all bug changes.