clang-format not installed by packages.sh

cmu-db / peloton

The Self-Driving Database Management System

http://pelotondb.io

Apache License 2.0

2.04k stars 623 forks source link

clang-format not installed by packages.sh #1261

Open pervazea opened 6 years ago

pervazea commented 6 years ago

clang-format-3.6 or as per current requirements

mbutrovich commented 6 years ago

This might be tough to easily fix for macOS since LLVM@3.6 is no longer offered as a Homebrew package. I had to download the full LLVM 3.6 archive and manually extract clang-format, rename it to clang-format-3.6, and stick it in the appropriate folder. I'm wondering if there's somewhere safe we could just pull that binary from without grabbing the entire LLVM 3.6 archive.

tcm-marcel commented 6 years ago

We had a longer discussion with the version of clang-format in #1015 and it was changed back in #1050.