Summary: | Heaptrack 1.1.0 fails to build on 32bit | ||
---|---|---|---|
Product: | [Applications] Heaptrack | Reporter: | Wolfgang Bauer <wbauer1> |
Component: | general | Assignee: | Milian Wolff <mail> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | fabian, geiger.david68210 |
Priority: | NOR | ||
Version: | 1.1.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://commits.kde.org/heaptrack/76fd2e84ba133e96d2cfdf90cb715e66e923eb8f | Version Fixed In: | |
Sentry Crash Report: |
Description
Wolfgang Bauer
2018-05-16 14:07:45 UTC
Git commit 76fd2e84ba133e96d2cfdf90cb715e66e923eb8f by Milian Wolff. Committed on 17/05/2018 at 11:32. Pushed by mwolff into branch '1.1'. Fix compile on 32bit M +5 -0 src/util/linewriter.h https://commits.kde.org/heaptrack/76fd2e84ba133e96d2cfdf90cb715e66e923eb8f *** Bug 394613 has been marked as a duplicate of this bug. *** Here on Mageia Cauldron it still fails to build on 32bit even with the applied patch: [ 43%] Building CXX object tests/auto/CMakeFiles/tst_io.dir/tst_io.cpp.o cd /home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/build/tests/auto && /usr/bin/c++ -DCATCH_CONFIG_MAIN -I/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/build/tests/auto -I/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/../.. -I/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/../../src -I/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/build/tests/auto/../../src -I/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/../../src/track -O2 -g -pipe -Wformat -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -fstack-protector --param=ssp-buffer-size=4 -fomit-frame-pointer -march=i586 -mtune=generic -fasynchronous-unwind-tables -std=c++11 -Wall -Wpedantic -g -o CMakeFiles/tst_io.dir/tst_io.cpp.o -c /home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/tst_io.cpp /home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/tst_io.cpp: In function 'void ____C_A_T_C_H____T_E_S_T____112()': /home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/tst_io.cpp:127:86: error: unable to deduce 'std::initializer_list<_Tp>&&' from '{8746901101568, 0, 223316, 2320472, 9984}' for (uint64_t expected : {0x7f48beedc00ul, 0x0ul, 0x36854ul, 0x236858ul, 0x2700ul}) { ^ /home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/tests/auto/tst_io.cpp:127:86: note: deduced conflicting types for parameter '_Tp' ('long long unsigned int' and 'long unsigned int') make[2]: *** [tests/auto/CMakeFiles/tst_io.dir/build.make:66: tests/auto/CMakeFiles/tst_io.dir/tst_io.cpp.o] Error 1 make[2]: Leaving directory '/home/iurt/rpmbuild/BUILD/heaptrack-1.1.0/build' make[1]: *** [CMakeFiles/Makefile2:1373: tests/auto/CMakeFiles/tst_io.dir/all] Error 2 make[1]: *** Waiting for unfinished jobs.... commit c456f6a1575fb2834238a1f693e7c7787d768d42 Author: Milian Wolff <milian.wolff@kdab.com> Date: Tue May 29 10:45:53 2018 +0200 Fix compile warnings about format type mismatch on 32bit Use PRIu64 instead of hardcoding %lu for the std::chrono::milliseconds representation. CCBUG: 394330 commit 49577e019ea791ee63962cdfe7e9c0c5b5c6ea4b Author: Milian Wolff <milian.wolff@kdab.com> Date: Tue May 29 10:41:10 2018 +0200 Fix another compile error on 32bit Introduce a user-defined _u64 literal for uint64_t and use that instead of ul/ull. BUG: 394330 Confirmed! Now heaptrack built fine without issue! Thanks! |