Skip to content

Conversation

@finagolfin
Copy link
Member

The Android community CI has been broken ever since this was merged in #66334 last month, because that CI builds against Android API 21:

/home/ubuntu/jenkins/workspace/oss-swift-RA-linux-ubuntu-16.04-android-arm64/swift/stdlib/public/runtime/CrashHandlerLinux.cpp:665:12: error: use of undeclared identifier 'process_vm_readv'
    return process_vm_readv(memserver_pid, &local, 1, &remote, 1, 0);
           ^
1 error generated.

Disable invoking this function before API 23, where it was added, which I confirmed gets the stdlib building with API 21 again.

@al45tair, please review.

@al45tair
Copy link
Contributor

@swift-ci Please smoke test

@finagolfin
Copy link
Member Author

@bnbarham, ready for merge.

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.

3 participants