Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 915471 - app-editor/imhex-1.30.1 : build failed with llvm17
Summary: app-editor/imhex-1.30.1 : build failed with llvm17
Status: RESOLVED FIXED
Alias: None
Product: GURU
Classification: Unclassified
Component: Package issues (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gasc Henri
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-09 11:28 UTC by xiaoming
Modified: 2023-10-10 06:32 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,370.36 KB, text/x-log)
2023-10-09 11:28 UTC, xiaoming
Details
patch to use llvm16 (use_llvm16.patch,525 bytes, patch)
2023-10-09 12:05 UTC, xiaoming
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description xiaoming 2023-10-09 11:28:06 UTC
Created attachment 872372 [details]
build.log

app-editor/imhex-1.30.1 build failed with llvm17.
the author says it's not yet support llvm17(https://github.com/WerWolv/ImHex/issues/1356), so perhaps we should patch the cmake file to use llvm16 or provide a system-llvm use flag.
related error:
cd /var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex_build/plugins/builtin && ccache /usr/bin/x86_64-pc-linux-gnu-g++ -DARCH_64_BIT -DBACKTRACE_HEADER=\"execinfo.h\" -DFMT_SHARED -DHEX_HAS_EXECINFO -DIMGUI_IMPL_OPENGL_LOADER_GLAD -DIMGUI_USER_CONFIG=\"imgui_config.h\" -DIMHEX_PROJECT_NAME=\"builtin\" -DIMHEX_VERSION=\"1.30.1\" -DLIBROMFS_PROJECT_NAME=builtin -DNDEBUG -DOS_LINUX -DSYSTEM_PLUGINS_LOCATION=\"/usr/lib64/imhex\" -Dbuiltin_EXPORTS -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/plugins/builtin/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/libimhex/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/xdgpp -I/usr/include/capstone -I/usr/lib/llvm/17/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/imgui/include -I/usr/lib64/libffi/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/microtar/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/pattern_language/lib/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/io/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/types/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/utils/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/hash/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/containers/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/pattern_language/generators/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/miniaudio/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libwolv/libs/net/include -I/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/lib/external/libromfs/lib/include -isystem /usr/include/freetype2  -fuse-ld=lld -march=native -O2 -pipe  -fexceptions -frtti   -Wall -Wextra -Wpedantic -Werror -Wno-array-bounds -Wno-restrict -Wno-stringop-overread -Wno-stringop-overflow -Wno-dangling-reference -std=gnu++23 -fPIC -MD -MT plugins/builtin/CMakeFiles/builtin.dir/source/content/file_handlers.cpp.o -MF CMakeFiles/builtin.dir/source/content/file_handlers.cpp.o.d -o CMakeFiles/builtin.dir/source/content/file_handlers.cpp.o -c /var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/plugins/builtin/source/content/file_handlers.cpp
/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/main/source/crash_handlers.cpp: In lambda function:
/var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/main/source/crash_handlers.cpp:120:99: error: too many arguments to function ‘char* llvm::itaniumDemangle(std::string_view)’
  120 | tr = hex::format("{}()::what() -> {}", llvm::itaniumDemangle(typeid(ex).name(), nullptr, nullptr, nullptr), ex.what());
      |                                        ~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

In file included from /var/tmp/portage/app-editors/imhex-1.30.1/work/ImHex/main/source/crash_handlers.cpp:15:
/usr/lib/llvm/17/include/llvm/Demangle/Demangle.h:35:7: note: declared here
   35 | char *itaniumDemangle(std::string_view mangled_name);
Comment 1 xiaoming 2023-10-09 12:05:33 UTC
Created attachment 872373 [details, diff]
patch to use llvm16
Comment 2 Gasc Henri 2023-10-09 15:38:37 UTC
Thanks for that. I updated the ebuild and put it on the last version
Comment 3 Larry the Git Cow gentoo-dev 2023-10-10 06:32:10 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/proj/guru.git/commit/?id=c169e2e8f87bb192603e2cbee910290d81caae26

commit c169e2e8f87bb192603e2cbee910290d81caae26
Author:     Henri Gasc <gasc@eurecom.fr>
AuthorDate: 2023-10-09 15:37:25 +0000
Commit:     Henri Gasc <gasc@eurecom.fr>
CommitDate: 2023-10-09 15:38:55 +0000

    app-editors/imhex: add 1.31.0, drop 1.30.1
    
    Closes: https://bugs.gentoo.org/915471
    Closes: https://bugs.gentoo.org/909290
    Closes: https://bugs.gentoo.org/909291
    Signed-off-by: Henri Gasc <gasc@eurecom.fr>

 app-editors/imhex/Manifest                         |  4 +--
 app-editors/imhex/files/llvm-16.patch              | 13 ++++++++++
 .../{imhex-1.30.1.ebuild => imhex-1.31.0.ebuild}   | 29 +++++++++-------------
 app-editors/imhex/metadata.xml                     |  3 +++
 4 files changed, 30 insertions(+), 19 deletions(-)