Skip to content

musl: powerpc(64): Decommonize termios definitions#4518

Merged
tgross35 merged 1 commit intorust-lang:mainfrom
Gelbpunkt:musl-powerpc-termios
Jul 3, 2025
Merged

musl: powerpc(64): Decommonize termios definitions#4518
tgross35 merged 1 commit intorust-lang:mainfrom
Gelbpunkt:musl-powerpc-termios

Conversation

@Gelbpunkt
Copy link
Copy Markdown
Contributor

Description

PowerPC targets use their own, separate definitions of termios that change the order of the c_line and c_cc fields.

Sources

https://git.musl-libc.org/cgit/musl/tree/arch/powerpc/bits/termios.h#n3
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc64/bits/termios.h#n3

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

PowerPC targets use their own, separate definitions of termios that
change the order of the c_line and c_cc fields.

See:
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc/bits/termios.h#n3
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc64/bits/termios.h#n3

Signed-off-by: Jens Reidel <adrian@travitia.xyz>
@tgross35 tgross35 added the stable-nominated This PR should be considered for cherry-pick to libc's stable release branch label Jul 3, 2025
@tgross35 tgross35 enabled auto-merge July 3, 2025 02:19
@Gelbpunkt
Copy link
Copy Markdown
Contributor Author

Not sure why the Android CI job got cancelled. Could you please retry it?

@tgross35 tgross35 added this pull request to the merge queue Jul 3, 2025
@tgross35
Copy link
Copy Markdown
Contributor

tgross35 commented Jul 3, 2025

Android is just super flaky unfortunately, looks like it made it to the queue now

Merged via the queue into rust-lang:main with commit 5698215 Jul 3, 2025
93 of 98 checks passed
tgross35 pushed a commit to tgross35/rust-libc that referenced this pull request Jul 29, 2025
PowerPC targets use their own, separate definitions of termios that
change the order of the c_line and c_cc fields.

See:
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc/bits/termios.h#n3
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc64/bits/termios.h#n3

Signed-off-by: Jens Reidel <adrian@travitia.xyz>

(backport <rust-lang#4518>)
(cherry picked from commit 0dfd754)
@tgross35 tgross35 mentioned this pull request Jul 29, 2025
github-merge-queue Bot pushed a commit that referenced this pull request Jul 29, 2025
PowerPC targets use their own, separate definitions of termios that
change the order of the c_line and c_cc fields.

See:
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc/bits/termios.h#n3
https://git.musl-libc.org/cgit/musl/tree/arch/powerpc64/bits/termios.h#n3

Signed-off-by: Jens Reidel <adrian@travitia.xyz>

(backport <#4518>)
(cherry picked from commit 0dfd754)
@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 Jul 29, 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 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