Skip to content

mk: Add a printer helper to the make setup #24315

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 13, 2015

Conversation

richo
Copy link
Contributor

@richo richo commented Apr 11, 2015

Also add docs because not being able to discover these things is the pits.

This was lifted from http://blog.jgc.org/2015/04/the-one-line-you-should-add-to-every.html , and solves a problem I've solved myself a lot of times, often in massively inferior ways.

Super open to documenting it somewhere other than nitty-gritty, which isn't immediately obvious.

Also add docs because not being able to discover these things is the
pits.
@rust-highfive
Copy link
Contributor

r? @pcwalton

(rust_highfive has picked a reviewer for you, use r? to override)

@pnkfelix
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Apr 12, 2015

📌 Commit 7a1d726 has been approved by pnkfelix

@bors
Copy link
Collaborator

bors commented Apr 12, 2015

⌛ Testing commit 7a1d726 with merge e7e320e...

@bors
Copy link
Collaborator

bors commented Apr 12, 2015

💔 Test failed - auto-linux-64-x-android-t

@richo
Copy link
Contributor Author

richo commented Apr 12, 2015

I think this is spurious, retry?

(Can people without an r-bit ask bors to retry?)

On Sun, Apr 12, 2015 at 6:04 AM, bors [email protected] wrote:

[image: 💔] Test failed - auto-linux-64-x-android-t
http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/4400


Reply to this email directly or view it on GitHub
#24315 (comment).

@pnkfelix
Copy link
Member

@bors retry

@pnkfelix
Copy link
Member

@bors rollup

@bors
Copy link
Collaborator

bors commented Apr 13, 2015

⌛ Testing commit 7a1d726 with merge 0cf99c3...

bors added a commit that referenced this pull request Apr 13, 2015
Also add docs because not being able to discover these things is the pits.

This was lifted from http://blog.jgc.org/2015/04/the-one-line-you-should-add-to-every.html , and solves a problem I've solved myself a *lot* of times, often in massively inferior ways.

Super open to documenting it somewhere other than nitty-gritty, which isn't immediately obvious.
@bors
Copy link
Collaborator

bors commented Apr 13, 2015

@bors bors merged commit 7a1d726 into rust-lang:master Apr 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants