Port the #![windows_subsystem] attribute to the new attribute system by scrabsha · Pull Request #149363 · rust-lang/rust (original) (raw)

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 andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation12 Commits1 Checks12 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

@scrabsha

Part of #131229.

r? @jdonszelmann

I think it's worth running the Windows test suite before merging that (I don't have the rights for this).

@rustbot

Some changes occurred in compiler/rustc_hir/src/attrs

cc @jdonszelmann

Some changes occurred in compiler/rustc_passes/src/check_attr.rs

cc @jdonszelmann

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

@rustbot rustbot added A-attributes

Area: Attributes (`#[…]`, `#![…]`)

S-waiting-on-review

Status: Awaiting review from the assignee but also interested parties.

T-compiler

Relevant to the compiler team, which will review and decide on the PR/issue.

labels

Nov 26, 2025

@rustbot

This comment has been minimized.

@scrabsha

@jdonszelmann

uh, I think this should be the right job:
@bors try jobs=x86_64-msvc

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request

Nov 26, 2025

@rust-bors

Port the #![windows_subsystem] attribute to the new attribute system

try-job: x86_64-msvc

@rust-bors

This comment was marked as off-topic.

@rust-log-analyzer

This comment was marked as off-topic.

@jieyouxu

Close, missing a -1.
@bors try jobs=x86_64-msvc-1

@rust-bors

This comment has been minimized.

rust-bors bot added a commit that referenced this pull request

Nov 27, 2025

@rust-bors

Port the #![windows_subsystem] attribute to the new attribute system

try-job: x86_64-msvc-1

@jdonszelmann

@rust-bors

☀️ Try build successful (CI)
Build commit: a16778f (a16778f2dc05e6f49d23e40e50cb67acf80b0f3e, parent: a4175159da185130408f71d7aa7e9bff0d455c24)

@jdonszelmann

@bors

📌 Commit 2ab2090 has been approved by jdonszelmann

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors

Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

and removed S-waiting-on-review

Status: Awaiting review from the assignee but also interested parties.

labels

Nov 27, 2025

bors added a commit that referenced this pull request

Nov 27, 2025

@bors

Rollup of 8 pull requests

Successful merges:

r? @ghost @rustbot modify labels: rollup

rust-timer added a commit that referenced this pull request

Nov 27, 2025

@rust-timer

Rollup merge of #149363 - scrabsha:rust/sasha/vkknqylzvzlu, r=jdonszelmann

Port the #![windows_subsystem] attribute to the new attribute system

Part of #131229.

I think it's worth running the Windows test suite before merging that (I don't have the rights for this).

@scrabsha scrabsha deleted the rust/sasha/vkknqylzvzlu branch

November 28, 2025 17:09

github-actions bot pushed a commit to rust-lang/rust-analyzer that referenced this pull request

Dec 1, 2025

@bors

Labels

A-attributes

Area: Attributes (`#[…]`, `#![…]`)

S-waiting-on-bors

Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

T-compiler

Relevant to the compiler team, which will review and decide on the PR/issue.