Skip to content

Switch to an official release of buildifier.#7801

Merged
maciekcc merged 2 commits into
tensorflow:masterfrom
jhseu:buildifier
Feb 23, 2017
Merged

Switch to an official release of buildifier.#7801
maciekcc merged 2 commits into
tensorflow:masterfrom
jhseu:buildifier

Conversation

@jhseu
Copy link
Copy Markdown
Contributor

@jhseu jhseu commented Feb 22, 2017

The current git master of buildifier has a broken build.

git clone https://github.com/bazelbuild/buildifier.git ${BUILDIFIER_DIR}
mkdir ${BUILDIFIER_DIR}
curl -Ls https://github.com/bazelbuild/buildifier/archive/0.4.3.tar.gz | \
tar -C "${BUILDIFIER_DIR}" --strip-components=1 -xz
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A leading space before the "t". The horror! The horror!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's actually a continuation of the last line :P I'll indent it two more spaces.

@maciekcc maciekcc merged commit b86522c into tensorflow:master Feb 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants