Skip to content

target: add -Zunstable-options when querying cfg#537

Open
DeepeshWR wants to merge 1 commit intolu-zero:masterfrom
DeepeshWR:fix/custom-target-cfg-unstable-options
Open

target: add -Zunstable-options when querying cfg#537
DeepeshWR wants to merge 1 commit intolu-zero:masterfrom
DeepeshWR:fix/custom-target-cfg-unstable-options

Conversation

@DeepeshWR
Copy link
Copy Markdown

Since Rust 1.95,invoking rustc with --target requires -Zunstable-options when used with custom JSON targets. Without this flag, the command fails.

Reference:
rust-lang/rust@b4e645f rust-lang/rust@8563312

@lu-zero
Copy link
Copy Markdown
Owner

lu-zero commented Apr 21, 2026

Can you please also add a test for it?

@lu-zero
Copy link
Copy Markdown
Owner

lu-zero commented Apr 21, 2026

adding it unconditionally breaks everything though :)

@DeepeshWR DeepeshWR force-pushed the fix/custom-target-cfg-unstable-options branch from 4de8265 to f18c870 Compare April 21, 2026 07:32
@lu-zero
Copy link
Copy Markdown
Owner

lu-zero commented Apr 21, 2026

Please make rustfmt and clippy happy

@DeepeshWR DeepeshWR force-pushed the fix/custom-target-cfg-unstable-options branch from f18c870 to bb234d1 Compare April 22, 2026 09:52
@lu-zero
Copy link
Copy Markdown
Owner

lu-zero commented Apr 22, 2026

it is still unhappy :/

@DeepeshWR DeepeshWR force-pushed the fix/custom-target-cfg-unstable-options branch from bb234d1 to bce4107 Compare April 22, 2026 10:52
@DeepeshWR
Copy link
Copy Markdown
Author

it is still unhappy :/

Fixed now :)

@lu-zero
Copy link
Copy Markdown
Owner

lu-zero commented Apr 24, 2026

sadly not yet...

Since Rust 1.95,invoking rustc with `--target` requires
`-Zunstable-options` when used with custom JSON targets.
Without this flag, the command fails.

Reference:
rust-lang/rust@b4e645f
rust-lang/rust@8563312

Signed-off-by: Deepesh Varatharajan <Deepesh.Varatharajan@windriver.com>
@lu-zero lu-zero force-pushed the fix/custom-target-cfg-unstable-options branch from bce4107 to b8049a4 Compare April 28, 2026 06:02
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.

2 participants