Skip to content

missing -Zjson-target-spec error: mention that this is a cargo flag#16793

Merged
weihanglo merged 1 commit intorust-lang:masterfrom
RalfJung:json-target-spec-flag
Mar 26, 2026
Merged

missing -Zjson-target-spec error: mention that this is a cargo flag#16793
weihanglo merged 1 commit intorust-lang:masterfrom
RalfJung:json-target-spec-flag

Conversation

@RalfJung
Copy link
Copy Markdown
Member

@RalfJung RalfJung commented Mar 26, 2026

Thanks for the pull request 🎉!
Please read the contribution guide: https://doc.crates.io/contrib/.

What does this PR try to resolve?

When I first saw this error, I thought it was a rustc error, so I figured out how to patch cargo-miri to add this to RUSTFLAGS -- only to then be told that the flag doesn't exist. Only then I realized that it's actually cargo emitting the error. Something similar just happened in rust-lang/rust#154340.

I think the error would be more helpful if it gave an indication of where that flag it mentions should be added.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 26, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Mar 26, 2026

r? @ehuss

rustbot has assigned @ehuss.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from ehuss, epage, weihanglo

@RalfJung RalfJung force-pushed the json-target-spec-flag branch from 95b663e to e08c0ce Compare March 26, 2026 15:09
Copy link
Copy Markdown
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like we should have a way to distinguish unstable flags from different tools. Another example is that -Zallow-features and you have no way to allow only certain tool's feature.

Anyway, looks good to me.

View changes since this review

@weihanglo weihanglo enabled auto-merge March 26, 2026 15:23
@weihanglo weihanglo added this pull request to the merge queue Mar 26, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 26, 2026
@weihanglo weihanglo added this pull request to the merge queue Mar 26, 2026
Merged via the queue into rust-lang:master with commit 99bf958 Mar 26, 2026
29 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 26, 2026
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Mar 28, 2026
Update cargo submodule

10 commits in e84cb639edfea2c42efd563b72a9be0cc5de6523..ddb37b06aba4dcf791fde56dd9fcf73ae7d05f7f
2026-03-21 01:27:07 +0000 to 2026-03-27 15:01:07 +0000
- fix(resolver): Better match rustc in error styling (rust-lang/cargo#16795)
- missing -Zjson-target-spec error: mention that this is a cargo flag (rust-lang/cargo#16793)
- Update changelog for 1.94.1 (rust-lang/cargo#16789)
- test(build): Make it easier to review `build.warnings` behavior (rust-lang/cargo#16788)
- test(build): Cover more behavior of `build.warnings` (rust-lang/cargo#16785)
- chore(deps): bump curl-sys from 0.4.83 to 0.4.86 (rust-lang/cargo#16786)
- Warn when installing with a non-default toolchain (rust-lang/cargo#16131)
- feat(cli): Add support for completing `--config` argument values with `native-completions` (rust-lang/cargo#16249)
- feat(cli): complete `--config` and `--color` before command (rust-lang/cargo#16780)
- chore(deps): bump rustls-webpki from 0.103.9 to 0.103.10 (rust-lang/cargo#16778)

r? ghost
rust-bors Bot pushed a commit to rust-lang/rust that referenced this pull request Mar 31, 2026
Update cargo submodule



11 commits in e84cb639edfea2c42efd563b72a9be0cc5de6523..888f675344eb1cf2308fd53183e667bdd2c58e51
2026-03-21 01:27:07 +0000 to 2026-03-30 16:59:25 +0000
- chore: bump to [email protected] (rust-lang/cargo#16808)
- fix(resolver): Better match rustc in error styling (rust-lang/cargo#16795)
- missing -Zjson-target-spec error: mention that this is a cargo flag (rust-lang/cargo#16793)
- Update changelog for 1.94.1 (rust-lang/cargo#16789)
- test(build): Make it easier to review `build.warnings` behavior (rust-lang/cargo#16788)
- test(build): Cover more behavior of `build.warnings` (rust-lang/cargo#16785)
- chore(deps): bump curl-sys from 0.4.83 to 0.4.86 (rust-lang/cargo#16786)
- Warn when installing with a non-default toolchain (rust-lang/cargo#16131)
- feat(cli): Add support for completing `--config` argument values with `native-completions` (rust-lang/cargo#16249)
- feat(cli): complete `--config` and `--color` before command (rust-lang/cargo#16780)
- chore(deps): bump rustls-webpki from 0.103.9 to 0.103.10 (rust-lang/cargo#16778)
@rustbot rustbot added this to the 1.96.0 milestone Mar 31, 2026
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.

4 participants