Skip to content

AIX fails to build #141543

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
Thomasdezeeuw opened this issue May 25, 2025 · 5 comments · Fixed by #142091
Closed

AIX fails to build #141543

Thomasdezeeuw opened this issue May 25, 2025 · 5 comments · Fixed by #142091
Labels
C-bug Category: This is a bug. O-aix OS: Big Blue's Advanced Interactive eXecutive.. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Comments

@Thomasdezeeuw
Copy link
Contributor

Hit this in a CI run of Mio: https://github.com/tokio-rs/mio/actions/runs/15226132067/job/42828397158.

Relevant information:

rustc 1.89.0-nightly (3e674b06b 2025-05-23)
binary: rustc
commit-hash: 3e674b06b5c74adea662bd0b0b06450757994b16
commit-date: 2025-05-23
host: x86_64-unknown-linux-gnu
release: 1.89.0-nightly
LLVM version: 20.1.5

Build log:

  error[E0425]: cannot find function `getenv` in this scope
     --> /home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/std/src/sys/pal/unix/os.rs:285:22
      |
  285 |     if let Some(p) = getenv(OsStr::from_bytes("PATH".as_bytes())) {
      |                      ^^^^^^ not found in this scope
      |
  help: consider importing one of these functions
      |
  8   + use crate::sys::env::getenv;
      |
  8   + use libc::getenv;
      |
  
  For more information about this error, try `rustc --explain E0425`.
  error: could not compile `std` (lib) due to 1 previous error
error: process didn't exit successfully: `/home/runner/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/bin/cargo check -Z build-std=std,panic_abort --target powerpc64-ibm-aix --manifest-path Cargo.toml --all-features` (exit status: 101)
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label May 25, 2025
@bjorn3 bjorn3 added C-bug Category: This is a bug. T-libs Relevant to the library team, which will review and decide on the PR/issue. O-aix OS: Big Blue's Advanced Interactive eXecutive.. labels May 25, 2025
@Noratrieb
Copy link
Member

@Noratrieb Noratrieb removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label May 25, 2025
@jyn514
Copy link
Member

jyn514 commented May 25, 2025

cc #109099

@gilamn5tr
Copy link

Is this is a result of #140143 ? I think @daltenty is still working through the 1.88 issues. We will take a look.

@thaliaarchi
Copy link
Contributor

thaliaarchi commented Jun 5, 2025

Sorry for the breakage. AIX was not one of the ~20 platforms I regularly check while working to get decent cfg coverage. I've pushed a fix as #142091. This is the only instance of this class of errors from that PR.

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this issue Jun 5, 2025
…ubilee

Fix AIX build

Fix rust-lang#141543.

`getenv` was moved out of this file to `sys::env::getenv` in rust-lang#140143. Replace its usage with `std::env::var_os`, the publicly exposed version. This matches the other usages of the same function in this file.
@bors bors closed this as completed in 9c89555 Jun 6, 2025
rust-timer added a commit that referenced this issue Jun 6, 2025
Rollup merge of #142091 - thaliaarchi:aix-getenv, r=workingjubilee

Fix AIX build

Fix #141543.

`getenv` was moved out of this file to `sys::env::getenv` in #140143. Replace its usage with `std::env::var_os`, the publicly exposed version. This matches the other usages of the same function in this file.
github-actions bot pushed a commit to rust-lang/miri that referenced this issue Jun 7, 2025
Fix AIX build

Fix rust-lang/rust#141543.

`getenv` was moved out of this file to `sys::env::getenv` in rust-lang/rust#140143. Replace its usage with `std::env::var_os`, the publicly exposed version. This matches the other usages of the same function in this file.
@Thomasdezeeuw
Copy link
Contributor Author

Thanks for fixing this @thaliaarchi

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
C-bug Category: This is a bug. O-aix OS: Big Blue's Advanced Interactive eXecutive.. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants