Fix instructions and dependencies on macos

weld-project / weld

High-performance runtime for data analytics applications

https://www.weld.rs

BSD 3-Clause "New" or "Revised" License

2.99k stars 258 forks source link

Closed winding-lines closed 5 years ago

winding-lines commented 5 years ago

Use brew --prefix instead of hardcoded path, it does not work on my system. Fix the flags for the c++ compiler

Ran cargo build --release on macos mojave and ubuntu 18.04

sppalkia commented 5 years ago

Thanks for the fixes! It looks good to me -- is it ready to merge?

winding-lines commented 5 years ago

Ready!