Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't print opt fuel messages to stdout because it breaks Rustbuild #55495

Merged
merged 2 commits into from
Nov 7, 2018
Merged

Don't print opt fuel messages to stdout because it breaks Rustbuild #55495

merged 2 commits into from
Nov 7, 2018

Commits on Oct 30, 2018

  1. Don't print opt fuel messages to stdout because it breaks Rustbuild 

    Rustbuild passes `--message-format json` to the compiler invocations
which causes JSON to be emitted on stdout. Printing optimization fuel
messages to stdout breaks the json and causes Rustbuild to fail.

Work around this by emitting optimization fuel related messages on
stderr instead.
    @wesleywiser
    wesleywiser committed Oct 30, 2018
    Configuration menu
    Copy the full SHA
    a7bea73 View commit details
    Browse the repository at this point in the history

Commits on Nov 6, 2018

  1. Add explcit --error-format options to tests of print-fuel to sidest… 

    …ep compiletests presumption of JSON.

As a driveby, rename the stdout files to stderr, which should have been part of the earlier commit.
    @pnkfelix
    pnkfelix committed Nov 6, 2018
    Configuration menu
    Copy the full SHA
    22b571d View commit details
    Browse the repository at this point in the history