Skip to content

fix: remove SaFlags workaround for libc issue#2762

Merged
SteveLauC merged 1 commit into
nix-rust:masterfrom
xtqqczze:GH2751
May 11, 2026
Merged

fix: remove SaFlags workaround for libc issue#2762
SteveLauC merged 1 commit into
nix-rust:masterfrom
xtqqczze:GH2751

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented Apr 1, 2026

Remove workaround for libc issue introduced in 0708c8c

Related: #2751 (comment)

@xtqqczze

This comment was marked as resolved.

@xtqqczze xtqqczze marked this pull request as ready for review April 1, 2026 13:27
Copy link
Copy Markdown
Member

@asomers asomers left a comment

Choose a reason for hiding this comment

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

In your commit message, could you please tag the commit whose workaround you're removing?

@xtqqczze

This comment was marked as outdated.

@asomers asomers added this pull request to the merge queue Apr 16, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Apr 17, 2026
@SteveLauC SteveLauC self-requested a review May 11, 2026 07:59
@SteveLauC
Copy link
Copy Markdown
Member

SteveLauC commented May 11, 2026

Hi @xtqqczze, you can rebase your branch to resolve the CI failures and a conflict.

Update: I took the liberty of rebasing it locally and pushing the updated branch for you.

Remove workaround for libc issue introduced in 0708c8c
@SteveLauC SteveLauC enabled auto-merge May 11, 2026 08:11
@SteveLauC SteveLauC added this pull request to the merge queue May 11, 2026
Merged via the queue into nix-rust:master with commit e615180 May 11, 2026
45 checks passed
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.

3 participants