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

Borrowck: diag migrate #101276

Closed
wants to merge 25 commits into from
Closed

Conversation

AndyJado
Copy link
Contributor

@AndyJado AndyJado commented Sep 1, 2022

No description provided.

@rustbot rustbot added A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 1, 2022
@rust-highfive
Copy link
Collaborator

r? @petrochenkov

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 1, 2022
@petrochenkov petrochenkov marked this pull request as ready for review September 1, 2022 14:28
@rustbot
Copy link
Collaborator

rustbot commented Sep 1, 2022

rustc_error_messages was changed

cc @davidtwco, @compiler-errors, @JohnTitor, @estebank, @TaKO8Ki

@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 1, 2022
@petrochenkov
Copy link
Contributor

Since this was submitted as a draft, changing it to S-waiting-on-author.

@davidtwco
Copy link
Member

r? @davidtwco

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@AndyJado
Copy link
Contributor Author

AndyJado commented Sep 3, 2022

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 3, 2022
@AndyJado AndyJado force-pushed the borrowck-diag-mig branch 2 times, most recently from 3f40403 to 5d2428f Compare September 4, 2022 08:31
@rust-log-analyzer

This comment has been minimized.

@AndyJado AndyJado force-pushed the borrowck-diag-mig branch 4 times, most recently from dc89bd1 to 2448a00 Compare September 5, 2022 08:16
@rust-log-analyzer
Copy link
Collaborator

The job mingw-check failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
    Checking rustc_codegen_ssa v0.0.0 (/checkout/compiler/rustc_codegen_ssa)
    Checking rustc_borrowck v0.0.0 (/checkout/compiler/rustc_borrowck)
    Checking rustc_mir_transform v0.0.0 (/checkout/compiler/rustc_mir_transform)
    Checking rustc_codegen_llvm v0.0.0 (/checkout/compiler/rustc_codegen_llvm)
error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:50:23
    |
50  |           let mut err = struct_span_err!(
50  |           let mut err = struct_span_err!(
    |  _______________________-
51  | |             self,
52  | |             new_loan_span,
53  | |             E0499,
...   |
56  | |             via(opt_via),
    | |_________- in this macro invocation
    |
note: the lint level is defined here
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:2:9
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:2:9
    |
2   | #![deny(rustc::diagnostic_outside_of_impl)]


error: diagnostics should be created using translatable messages
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:50:23
    |
50  |           let mut err = struct_span_err!(
50  |           let mut err = struct_span_err!(
    |  _______________________-
51  | |             self,
52  | |             new_loan_span,
53  | |             E0499,
...   |
56  | |             via(opt_via),
    | |_________- in this macro invocation
    |
note: the lint level is defined here
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:1:9
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:1:9
    |
1   | #![deny(rustc::untranslatable_diagnostic)]


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:177:23
    |
177 |           let mut err = struct_span_err!(
177 |           let mut err = struct_span_err!(
    |  _______________________-
178 | |             self,
179 | |             span,
180 | |             E0502,
...   |
187 | |             via(msg_old),
    | |_________- in this macro invocation


error: diagnostics should be created using translatable messages
    |
608 |   macro_rules! struct_span_err {
    |   ---------------------------- in this expansion of `struct_span_err!`
    |   ---------------------------- in this expansion of `struct_span_err!`
609 |       ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |           $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:177:23
    |
177 |           let mut err = struct_span_err!(
177 |           let mut err = struct_span_err!(
    |  _______________________-
178 | |             self,
179 | |             span,
180 | |             E0502,
...   |
187 | |             via(msg_old),
    | |_________- in this macro invocation


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
    |
608 | macro_rules! struct_span_err {
    | ---------------------------- in this expansion of `struct_span_err!`
    | ---------------------------- in this expansion of `struct_span_err!`
609 |     ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |         $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:301:9
    |
    |
301 |         struct_span_err!(self, span, E0596, "cannot borrow {} as mutable{}", path, reason,)


error: diagnostics should be created using translatable messages
    |
608 | macro_rules! struct_span_err {
    | ---------------------------- in this expansion of `struct_span_err!`
    | ---------------------------- in this expansion of `struct_span_err!`
609 |     ($session:expr, $span:expr, $code:ident, $($message:tt)*) => ({
610 |         $session.struct_span_err_with_code(
    |
   ::: compiler/rustc_borrowck/src/borrowck_errors.rs:301:9
    |
    |
301 |         struct_span_err!(self, span, E0596, "cannot borrow {} as mutable{}", path, reason,)


error: diagnostics should only be created in `SessionDiagnostic`/`AddSubdiagnostic` impls
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:395:29
    |
395 |         self.infcx.tcx.sess.struct_span_err_with_code(sp, msg, code)


error: diagnostics should be created using translatable messages
   --> compiler/rustc_borrowck/src/borrowck_errors.rs:395:29
    |
395 |         self.infcx.tcx.sess.struct_span_err_with_code(sp, msg, code)

error: could not compile `rustc_borrowck` due to 8 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `rustc_borrowck` due to 8 previous errors

@AndyJado
Copy link
Contributor Author

AndyJado commented Sep 9, 2022

@rustbot label +S-waiting-on-author -S-waiting-on-review

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 9, 2022
@AndyJado
Copy link
Contributor Author

moved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-translation Area: Translation infrastructure, and migrating existing diagnostics to SessionDiagnostic S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants