Skip to content

chore: restrict Android cfg too 64-bit to match linux_execfn#12416

Open
YumeYuka wants to merge 1 commit into
uutils:mainfrom
OOM-WG:main
Open

chore: restrict Android cfg too 64-bit to match linux_execfn#12416
YumeYuka wants to merge 1 commit into
uutils:mainfrom
OOM-WG:main

Conversation

@YumeYuka
Copy link
Copy Markdown
Contributor

linux_execfn cfg requirements to only support Linux and 64-bit Android targets

#[cfg(any(
    all(target_os = "android", target_pointer_width = "64"),
    target_os = "linux",
))]
#[inline]
pub(crate) fn linux_execfn() -> &'static CStr {
    if let Some(libc_getauxval) = getauxval.get() {
        unsafe { CStr::from_ptr(libc_getauxval(c::AT_EXECFN).cast()) }
    } else {
        cstr!("")
    }
}

Co-Authored-By: 白彩恋 <169267914+shirorren@users.noreply.github.com>
@oech3
Copy link
Copy Markdown
Contributor

oech3 commented May 21, 2026

@sunfishcode No 32bit support?

@oech3
Copy link
Copy Markdown
Contributor

oech3 commented May 21, 2026

I think 32bit CI is passing at here with current cfg. No 32bit android CI at this repo (broken).

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/date/resolution (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/cut/cut-huge-range is now passing!

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.

2 participants