niklas-heer / speed-comparison

A repo which compares the speed of different programming languages.
https://niklas-heer.github.io/speed-comparison
MIT License
475 stars 76 forks source link

Updated Earthfile to use latest nightly rust #87

Closed eternalfrustation closed 1 year ago

eternalfrustation commented 1 year ago

Vectorization is only available on nightly rust, as far as i can tell

giordano commented 1 year ago

Seems to work for me with Rust 1.64, without using nightly.

niklas-heer commented 1 year ago

@eternalfrustation it also seems that there is no sh installed by default on alpine.

https://github.com/niklas-heer/speed-comparison/actions/runs/3346502675/jobs/5543390049#step:6:2576

niklas-heer commented 1 year ago

@giordano that is a good point. The current version on alpine 3.16 seems to be 1.60. https://pkgs.alpinelinux.org/package/v3.16/community/x86_64/rust

@eternalfrustation maybe it would be better to use the rust:1.64-alpine image instead. (see the official rust Docker images)

eternalfrustation commented 1 year ago

I have updated Earthfile to install rust:1.64-alpine. Unfortunately, i don't have much experience with docker and earthly, so i don't know if this will work The workflow on my forked repo ran successfully, and rust's mean time was less than C, so i guess vectorization is also working

niklas-heer commented 1 year ago

It worked 👍 combined_results

niklas-heer commented 1 year ago

@eternalfrustation thank you for your contribution! 👍