Skip to content

[android] Fix CI for recent trunk pulls#87075

Merged
finagolfin merged 2 commits intoswiftlang:mainfrom
finagolfin:droid
Feb 9, 2026
Merged

[android] Fix CI for recent trunk pulls#87075
finagolfin merged 2 commits intoswiftlang:mainfrom
finagolfin:droid

Conversation

@finagolfin
Copy link
Member

@j-hui, you just added this in #87026, likely for your own local testing, but it fails on the Android CI. I see that most tests don't use tee for this reason.

Let me know if this is okay, but don't merge, as the Android CI was already broken with another issue, so I think I'll need to add another commit for that too.

@finagolfin
Copy link
Member Author

@swift-ci test android

Copy link
Contributor

@j-hui j-hui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes I did add that just for testing, sorry about that!

…orms where backtracing was disabled

This was failing to compile on platforms like Android armv7 where it is disabled.
@finagolfin
Copy link
Member Author

finagolfin commented Feb 9, 2026

@al45tair, added a fix commit for your recent #84906 that is breaking Android armv7 on the CI, because backtracing was never enabled there.

@swift-ci test android

@finagolfin finagolfin changed the title [test] Remove unneeded tee command from C++ Interop test [android] Fix CI for recent trunk pulls Feb 9, 2026
@finagolfin
Copy link
Member Author

@swift-ci test linux

Copy link
Contributor

@al45tair al45tair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Backtrace.cpp change looks OK.

@finagolfin
Copy link
Member Author

@swift-ci smoke test macOS

@finagolfin
Copy link
Member Author

@swift-ci smoke test Windows

@finagolfin finagolfin enabled auto-merge February 9, 2026 14:51
@finagolfin finagolfin merged commit cddeec6 into swiftlang:main Feb 9, 2026
5 checks passed
@finagolfin finagolfin deleted the droid branch February 10, 2026 01:35
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