Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fedora 38 compile error #118

Open
ZeddieXX opened this issue Jun 3, 2023 · 4 comments
Open

Fedora 38 compile error #118

ZeddieXX opened this issue Jun 3, 2023 · 4 comments

Comments

@ZeddieXX
Copy link

ZeddieXX commented Jun 3, 2023

Getting an error when compiling based on your instructions for building from source (on Fedora):
[ 21%] Building CXX object CMakeFiles/xmrig.dir/src/crypto/common/LinuxMemory.cpp.o
[ 21%] Building C object CMakeFiles/xmrig.dir/src/crypto/cn/c_blake256.c.o
[ 22%] Building C object CMakeFiles/xmrig.dir/src/crypto/cn/c_groestl.c.o
/home/zeddie/tmp/xmrig/src/backend/opencl/runners/OclRyoRunner.cpp: In member function ‘virtual void xmrig::OclRyoRunner::set(const xmrig::Job&, uint8_t*)’:
/home/zeddie/tmp/xmrig/src/backend/opencl/runners/OclRyoRunner.cpp:92:20: error: ‘length_error’ is not a member of ‘std’
92 | throw std::length_error("job size too big");
| ^~~~~~~~~~~~
/home/zeddie/tmp/xmrig/src/backend/opencl/runners/OclRyoRunner.cpp:37:1: note: ‘std::length_error’ is defined in header ‘’; did you forget to ‘#include ’?
36 | #include "crypto/cn/CnAlgo.h"
+++ |+#include
37 |
[ 22%] Building C object CMakeFiles/xmrig.dir/src/crypto/cn/c_jh.c.o
make[2]: *** [CMakeFiles/xmrig.dir/build.make:2008: CMakeFiles/xmrig.dir/src/backend/opencl/runners/OclRyoRunner.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:182: CMakeFiles/xmrig.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

@jordan89m
Copy link

Same here on Archlinux.

@jordan89m
Copy link

Found the fix. Open up src/backend/opencl/runners/OclRyoRunner.cpp and add the following:

#include <stdexcept>

just beneath the other #includes.

@realsetvin
Copy link

thank you so much for creating the pull request! should we close this issue now?

@Spudz76
Copy link

Spudz76 commented May 29, 2024

Was fixed at commit 7445870 (May 2) and released in v6.21.3-mo2. Please close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants