Skip to content

update 'unsupported' message #3558

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
May 4, 2024
Merged

update 'unsupported' message #3558

merged 1 commit into from
May 4, 2024

Conversation

RalfJung
Copy link
Member

@RalfJung RalfJung commented May 4, 2024

Instead of "the interpreter", just say Miri.

Also be a more more clear about what is expected to be supported and what not (Cc #2325).

@RalfJung
Copy link
Member Author

RalfJung commented May 4, 2024

@bors r+

@bors
Copy link
Contributor

bors commented May 4, 2024

📌 Commit 23be20b has been approved by RalfJung

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented May 4, 2024

⌛ Testing commit 23be20b with merge b25a719...

@bors
Copy link
Contributor

bors commented May 4, 2024

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing b25a719 to master...

@bors bors merged commit b25a719 into rust-lang:master May 4, 2024
8 checks passed
@RalfJung RalfJung deleted the unsupported branch May 4, 2024 12:47
bors added a commit that referenced this pull request May 4, 2024
only show the 'basic API common for this target' message when this is a missing foreign function

Follow-up to #3558
bors added a commit that referenced this pull request May 4, 2024
only show the 'basic API common for this target' message when this is a missing foreign function

Follow-up to #3558
RalfJung pushed a commit to RalfJung/rust that referenced this pull request May 4, 2024
only show the 'basic API common for this target' message when this is a missing foreign function

Follow-up to rust-lang/miri#3558
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants