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.

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

Update E0025 to new error format #36757

Merged
merged 1 commit into from
Sep 29, 2016
Merged

Conversation

KiChjang
Copy link
Member

Part of #35233.
Fixes #35198.

r? @jonathandturner

@sophiajt
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Sep 26, 2016

📌 Commit 396e45d has been approved by jonathandturner

@sophiajt
Copy link
Contributor

Looks like one of the unit test needs updating:

failures:

---- [compile-fail] compile-fail/issue-15260.rs stdout ----

error: /build/src/test/compile-fail/issue-15260.rs:18: unexpected "note": 'multiple uses of `a` in pattern'

error: /build/src/test/compile-fail/issue-15260.rs:17: unexpected "note": 'first use of `a`'

error: /build/src/test/compile-fail/issue-15260.rs:23: unexpected "note": 'multiple uses of `a` in pattern'

error: /build/src/test/compile-fail/issue-15260.rs:22: unexpected "note": 'first use of `a`'

error: /build/src/test/compile-fail/issue-15260.rs:29: unexpected "note": 'multiple uses of `a` in pattern'

error: /build/src/test/compile-fail/issue-15260.rs:27: unexpected "note": 'first use of `a`'

error: /build/src/test/compile-fail/issue-15260.rs:30: unexpected "note": 'multiple uses of `a` in pattern'

error: /build/src/test/compile-fail/issue-15260.rs:27: unexpected "note": 'first use of `a`'

error: /build/src/test/compile-fail/issue-15260.rs:17: expected note not found: field `a` previously bound here

error: /build/src/test/compile-fail/issue-15260.rs:22: expected note not found: field `a` previously bound here

error: /build/src/test/compile-fail/issue-15260.rs:27: expected note not found: field `a` previously bound here

error: /build/src/test/compile-fail/issue-15260.rs:27: expected note not found: field `a` previously bound here

error: 8 unexpected errors found, 4 expected errors not found
status: exit code: 101
command: /build/build/x86_64-unknown-linux-gnu/stage2/bin/rustc /build/src/test/compile-fail/issue-15260.rs -L /build/build/x86_64-unknown-linux-gnu/test/compile-fail --target=x86_64-unknown-linux-gnu --error-format json -L /build/build/x86_64-unknown-linux-gnu/test/compile-fail/issue-15260.stage2-x86_64-unknown-linux-gnu.compile-fail.libaux -C prefer-dynamic -o /build/build/x86_64-unknown-linux-gnu/test/compile-fail/issue-15260.stage2-x86_64-unknown-linux-gnu -Crpath -O -Lnative=/build/build/x86_64-unknown-linux-gnu/rust-test-helpers
unexpected errors (from JSON output): [
    Error {
        line_num: 18,
        kind: Some(
            Note
        ),
        msg: "multiple uses of `a` in pattern"
    },
    Error {
        line_num: 17,
        kind: Some(
            Note
        ),
        msg: "first use of `a`"
    },
    Error {
        line_num: 23,
        kind: Some(
            Note
        ),
        msg: "multiple uses of `a` in pattern"
    },
    Error {
        line_num: 22,
        kind: Some(
            Note
        ),
        msg: "first use of `a`"
    },
    Error {
        line_num: 29,
        kind: Some(
            Note
        ),
        msg: "multiple uses of `a` in pattern"
    },
    Error {
        line_num: 27,
        kind: Some(
            Note
        ),
        msg: "first use of `a`"
    },
    Error {
        line_num: 30,
        kind: Some(
            Note
        ),
        msg: "multiple uses of `a` in pattern"
    },
    Error {
        line_num: 27,
        kind: Some(
            Note
        ),
        msg: "first use of `a`"
    }
]

not found errors (from test file): [
    Error {
        line_num: 17,
        kind: Some(
            Note
        ),
        msg: "field `a` previously bound here"
    },
    Error {
        line_num: 22,
        kind: Some(
            Note
        ),
        msg: "field `a` previously bound here"
    },
    Error {
        line_num: 27,
        kind: Some(
            Note
        ),
        msg: "field `a` previously bound here"
    },
    Error {
        line_num: 27,
        kind: Some(
            Note
        ),
        msg: "field `a` previously bound here"
    }
]

thread '[compile-fail] compile-fail/issue-15260.rs' panicked at 'explicit panic', src/tools/compiletest/src/runtest.rs:1097
note: Run with `RUST_BACKTRACE=1` for a backtrace.


failures:
    [compile-fail] compile-fail/issue-15260.rs

test result: FAILED. 2575 passed; 1 failed; 12 ignored; 0 measured

@sophiajt
Copy link
Contributor

@bors r-

(pulling it out so we don't accidentally roll it up)

@KiChjang
Copy link
Member Author

Alright, updated PR, should be ok now.

@sophiajt
Copy link
Contributor

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Sep 26, 2016

📌 Commit fd7314f has been approved by jonathandturner

sophiajt pushed a commit to sophiajt/rust that referenced this pull request Sep 28, 2016
bors added a commit that referenced this pull request Sep 28, 2016
Rollup of 11 pull requests

- Successful merges: #36376, #36672, #36740, #36757, #36765, #36769, #36782, #36783, #36784, #36795, #36796
- Failed merges:
@bors bors merged commit fd7314f into rust-lang:master Sep 29, 2016
@KiChjang KiChjang deleted the E0025-format branch September 29, 2016 01:41
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.

3 participants