Skip to content

[Android] fix NDK extract location #75968

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

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

andrurogerz
Copy link
Contributor

@andrurogerz andrurogerz commented Aug 19, 2024

The Android NDK is being extracted with an extra intermediate directory:

$BinaryCache\android-ndk-r26b\android-ndk-r26b

The Android build fails because it expects the NDK contents to live immediately under the directory:

$BinaryCache\android-ndk-r26b

The redundant android-ndk-r26b sub-dir comes from the NDK zip archive itself, which we have no control over. To work-around, allow callers of Extract-ZipFile to control whether or not it should create the extraction sub-dir. Override the default behavior for NDK extraction only.

@andrurogerz
Copy link
Contributor Author

@swift-ci please test

@andrurogerz andrurogerz marked this pull request as ready for review August 19, 2024 23:27
The Android NDK is being extracted with an extra intermediate directory:

  $BinaryCache\android-ndk-r26b\android-ndk-r26b

The Android build fails because it expects the NDK contents to live
immediately under the directory:

  $BinaryCache\android-ndk-r26b

The extra android-ndk-r26b sub-dir comes from the NDK zip archive
itself, which we have no control over. To work-around, allow callers
of Extract-ZipFile to control whether or not it should create the
extraction sub-dir. Override the default behavior for NDK extraction
only.
@andrurogerz
Copy link
Contributor Author

@swift-ci please test

@compnerd
Copy link
Member

@swift-ci please smoke test

@compnerd compnerd enabled auto-merge August 19, 2024 23:48
@compnerd compnerd merged commit b18c8bb into swiftlang:main Aug 20, 2024
3 checks passed
@andrurogerz andrurogerz deleted the android-ndk-extract branch August 20, 2024 14:35
@andrurogerz
Copy link
Contributor Author

The issue fixed by this PR was introduced in ddedfe0

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