Skip to content

internal: Use a process group for flycheck#13552

Merged
bors merged 1 commit into
rust-lang:masterfrom
Veykril:flycheck-process-group
Nov 7, 2022
Merged

internal: Use a process group for flycheck#13552
bors merged 1 commit into
rust-lang:masterfrom
Veykril:flycheck-process-group

Conversation

@Veykril

@Veykril Veykril commented Nov 5, 2022

Copy link
Copy Markdown
Member

Should fix #13348

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 5, 2022
@Veykril

Veykril commented Nov 7, 2022

Copy link
Copy Markdown
Member Author

@bors r+

@bors

bors commented Nov 7, 2022

Copy link
Copy Markdown
Contributor

📌 Commit 1dcc25a has been approved by Veykril

It is now in the queue for this repository.

@bors

bors commented Nov 7, 2022

Copy link
Copy Markdown
Contributor

⌛ Testing commit 1dcc25a with merge 7742077...

@bors

bors commented Nov 7, 2022

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 7742077 to master...

@bors bors merged commit 7742077 into rust-lang:master Nov 7, 2022
@Veykril Veykril deleted the flycheck-process-group branch November 7, 2022 10:39
}
}

struct JodChild(GroupChild);

@Veykril Veykril Nov 14, 2022

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I just realized I forgot to implement Drop for this 😶 (shouldn't really matter fortunately)

nielx added a commit to nielx/rust-analyzer that referenced this pull request Jan 15, 2023
With rust-lang#13552 the depencency of on the command-group crate was introduced, which also
introduced a dependency on nix. That version of nix does not build on Haiku. This
change introduces a newer version of command-group, which also updates nix from
0.22.3 to 0.26.1, which is compatible on Haiku.
bors added a commit that referenced this pull request Jan 15, 2023
Update command-group from 1.0.8 to 2.0.1

With #13552 the depencency of on the command-group crate was introduced, which also introduced a dependency on nix. That version of nix does not build on Haiku. This change introduces a newer version of command-group, which also updates nix from 0.22.3 to 0.26.1, which is compatible on Haiku.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Canceling initial check build does not work

4 participants