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

unknown unstable option: ignore-directory-in-diagnostics-source-blocks #115977

Closed
m-ou-se opened this issue Sep 19, 2023 · 7 comments · Fixed by #116009
Closed

unknown unstable option: ignore-directory-in-diagnostics-source-blocks #115977

m-ou-se opened this issue Sep 19, 2023 · 7 comments · Fixed by #116009
Labels
C-bug Category: This is a bug.

Comments

@m-ou-se
Copy link
Member

m-ou-se commented Sep 19, 2023

When I run ./x.py test locally, I get errors:

---- [ui] tests/ui-fulldeps/session-diagnostic/invalid-variable.rs stdout ----

error: test compilation failed although it shouldn't!
[..]
--- stderr -------------------------------
error: unknown unstable option: `ignore-directory-in-diagnostics-source-blocks`
------------------------------------------

(Same for 27 ui-fulldeps tests in total.)

I assume this happened after #115869. cc @pietroalbini

@m-ou-se m-ou-se added the C-bug Category: This is a bug. label Sep 19, 2023
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Sep 19, 2023
@m-ou-se
Copy link
Member Author

m-ou-se commented Sep 19, 2023

Since CI isn't running into these issues, I'm guessing it only happens with --stage=1 and works fine with --stage=2.

@m-ou-se
Copy link
Member Author

m-ou-se commented Sep 19, 2023

Yup, it works fine with ./x.py test --stage=2 here.

@compiler-errors
Copy link
Member

compiler-errors commented Sep 19, 2023

Ugh, this keeps happening to ui-fulldeps when we add new -Z flags.

@compiler-errors
Copy link
Member

#114482 #114102

I think we should just test ui-fulldeps --stage=1 in CI to make sure this doesn't keep happening. See the description of #114102 to understand why ui-fulldeps actually builds with stage0, which means that modifications to the UI test code adding new flags need to make sure they're properly gating their code.

@compiler-errors
Copy link
Member

Either that, or we need something like -Zignore-missing-flags and pass that just to the ui-fulldeps suite on stage1. That seems more like a hack though :s

@saethlin saethlin removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Sep 19, 2023
@Mark-Simulacrum
Copy link
Member

@compiler-errors https://github.com/rust-lang/rust/blob/master/src/ci/docker/host-x86_64/x86_64-gnu-llvm-15/script.sh#L6 should be pretty easy to extend with a --stage=1 for ui-fulldeps.

@compiler-errors
Copy link
Member

Cool, will do when I get home.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants