Skip to content

libc v0.2 cherry-pick: Enable libc-test and fix definitions/declarations for AIX #4450 #4455

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

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

xingxue-ibm
Copy link
Contributor

@xingxue-ibm xingxue-ibm commented May 16, 2025

Description

This PR enables libc-test on AIX. With the fixes to definitions and declarations, libc-test now runs successfully on AIX.

(cherry picked from commit c192a5cae31dd84ad17dc3c2dd0a3deaacae452a)

In addition to cherry-picking from the main branch, src/unix/mod.rs is modified to use to *const *mut in execv* functions for AIX.

Sources

Checklist

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

@rustbot rustbot added the O-unix label May 16, 2025
@xingxue-ibm xingxue-ibm changed the title libc v2.0 cherry-pick: Enable libc-test and fix definitions/declarations for AIX #4450 libc v0.2 cherry-pick: Enable libc-test and fix definitions/declarations for AIX #4450 May 16, 2025
@tgross35
Copy link
Contributor

tgross35 commented May 16, 2025

Mentioned at #4450, there isn't any need to open PRs since I do the picks in order. I will pick up 6f4ad51 though since that is a needed fix.

However, thank you for putting in the effort here!

@tgross35 tgross35 closed this May 16, 2025
@xingxue-ibm
Copy link
Contributor Author

Thanks so much, @tgross35!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants