tmpim / kristforge

The fastest krist miner in existence, with both GPU and CPU support
Apache License 2.0
19 stars 9 forks source link

Update README.md #14

Closed fishermedders closed 3 years ago

fishermedders commented 3 years ago

I've added a bit in the README.md, pertaining to the (potential) absence of libOpenCL.so on Linux systems. Tested on Ubuntu 20.04 (Debian), so I am not sure of other distros.

https://gyazo.com/ffbedc0584638b5e55a733312a932cd4 <- Console log in question.

Of course, edit the line to your liking, but I think that this information could be very useful for many users. I wasn't sure if I should have written the package manager format, e.g. 'sudo apt-get install ocl-icd-opencl-dev', but perhaps this would be a good addition.

Please forgive me if I missed a pull request format, this is my first pull request; regardless, I feel like this is pertinent information. :) Thanks for making kristforge, and for your contribution to open source software, Fisher.

dmarcuse commented 3 years ago

Hmm, this is actually a bug - it should gracefully disable GPU mining and continue when there's no OpenCL libraries available. Could you re-run the program with RUST_BACKTRACE=1 set, to get a trace of where the crash is happening?

Edit: managed to find it already, will publish a release fixing this soon

dmarcuse commented 3 years ago

I've released version 3.1.4 to fix the crash, and updated the README to indicate that OpenCL packages usually need to be installed separately on Linux systems. I didn't use your PR in the end because the specific package name and command vary between distributions, but thanks for opening this anyways!