On Sun, May 10, 2015 at 2:24 PM James Peach <jpe...@apache.org> wrote:
> > > On Mar 31, 2015, at 11:33 AM, Leif Hedstrom <zw...@apache.org> wrote: > > > > Hi all, > > > > I’ve updated the tar-ball with the clang-format tools: > > > > http://people.apache.org/~zwoop/clang-format-03312015.tar.bz2 > > > > > > The MD5 of the tar-ball is > > > > minotaur (18:28) 266/0 $ md5 clang-format-03312015.tar.bz2 > > MD5 (clang-format-03312015.tar.bz2) = > 6b3e7933f3e93b52361c88970bc0d8fe > > > > > > Please use this exact version of clang format. Currently, it has a > static binary for Linux, which I hope should work on most platforms (let me > know if it doesn’t). In addition, it also has a binary for OS X (Yosemite, > I have not tested on Maverick). It’s important we all use the same version, > so going forward, I’m versioning the packages as well, with the date. If > you are developing on a platform other then Linux and OS X, let me know and > I’ll try to make a version for that as well. > > > > Question: Should we consider incorporating these binaries and scripts > into our source tree? > > Yes, definitely! > I am -1 on binaries in the repo, but if we wanted to put the code and build it, I'd be ok with that. I assume the license is compatible? > > FWIW, here’s a script that formats the top commit. It’s quite useful in > conjunction with "git rebase —exec” ... > > #! /bin/bash > > case $(uname -s) in > Darwin) OS=osx;; > *) OS=linux;; > esac > > TOPDIR=$(git rev-parse --show-toplevel) > FORMAT=$TOPDIR/.git/clang-format/clang-format.$OS > > exec $FORMAT -i $(git show --pretty=format: --name-only)
