Skip to content

Conversation

heiher
Copy link
Contributor

@heiher heiher commented Apr 8, 2025

Description

Remove the __pad2 field from struct ipc_perm.

Refer: https://git.musl-libc.org/cgit/musl/tree/arch/generic/bits/ipc.h?h=v1.2.5

struct ipc_perm {
	key_t __ipc_perm_key;
	uid_t uid;
	gid_t gid;
	uid_t cuid;
	gid_t cgid;
	mode_t mode;
	int __ipc_perm_seq;
	long __pad1;
	long __pad2;
};

Sources

  • src/unix/linux_like/linux/musl/b64/loongarch64/mod.rs

Checklist

  • Relevant tests in libc-test/semver have been updated
  • No placeholder or unstable values like *LAST or *MAX are
    included (see #3131)
  • Tested locally (cd libc-test && cargo test --target mytarget);
    especially relevant for platforms that may not be checked in CI

@rustbot
Copy link
Collaborator

rustbot commented Apr 8, 2025

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link
Contributor

@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

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

LGTM, merging is blocked on a CI bug

@tgross35 tgross35 enabled auto-merge April 9, 2025 07:26
@tgross35 tgross35 added this pull request to the merge queue Apr 9, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 9, 2025
@tgross35 tgross35 enabled auto-merge April 9, 2025 17:47
@tgross35 tgross35 added this pull request to the merge queue Apr 9, 2025
Merged via the queue into rust-lang:main with commit 2d82b0e Apr 9, 2025
44 of 45 checks passed
@heiher
Copy link
Contributor Author

heiher commented May 7, 2025

@rustbot label +stable-nominated

@rustbot rustbot added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label May 7, 2025
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request May 28, 2025
@tgross35 tgross35 mentioned this pull request May 28, 2025
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request May 28, 2025
@tgross35 tgross35 added stable-applied This PR has been cherry-picked to libc's stable release branch and removed stable-nominated This PR should be considered for cherry-pick to libc's stable release branch labels Jun 1, 2025
DorianNiemiecSVRJS pushed a commit to DorianNiemiecSVRJS/rust-libc that referenced this pull request Jun 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-linux O-musl O-unix S-waiting-on-review stable-applied This PR has been cherry-picked to libc's stable release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants