llm icon indicating copy to clipboard operation
llm copied to clipboard

:memo: add build instructions for fedora

Open sylvain-reynaud opened this issue 1 year ago • 2 comments

Closes #22

How to test it ?

docker run -it fedora bash

Then

ls /usr/lib/        #no gcc
dnf groupinstall "Development Tools" "Development Libraries"
ls /usr/lib/gcc/x86_64-redhat-linux/12/include/        #installed
git clone https://github.com/sylvain-reynaud/llama-rs
cd llama-rs/
curl https://sh.rustup.rs -sSf | sh
source "$HOME/.cargo/env"
cargo build --release        #still fails
CPATH="/usr/lib/gcc/x86_64-redhat-linux/12/include/" cargo build --release        #success

sylvain-reynaud avatar Mar 16 '23 08:03 sylvain-reynaud

Is this still necessary after #28 ?

philpax avatar Mar 22 '23 08:03 philpax

I made a quick test. Installing a C compiler is still necessary, but none of the additional steps apply anymore since we got rid of bindgen (specifically, setting CPATH).

I would say including instructions in the README to make people aware they need a C compiler should be enough. But we can also provide some distro-specific commands. Similar to dnf groupinstall "Development Tools" "Development Libraries", ubuntu has the build-essential package for example.

Doing something for #20 would also help a lot in making this less compilicated, since people can just download the binary then.

setzer22 avatar Mar 22 '23 10:03 setzer22

Closing this as #73 should address it. (Please let me know/send a PR if it doesn't!)

philpax avatar Mar 26 '23 13:03 philpax