Skip to content

Commit

Permalink
[Xamarin.Android.Tools.AndroidSdk] Permit NDK r25 (#190)
Browse files Browse the repository at this point in the history
Context: dotnet/android#7233

Update `AndroidSdkBase.MaximumCompatibleNDKMajorVersion` to 25, so
that NDK r25 is considered as a valid version.
  • Loading branch information
jonathanpeppers authored Aug 4, 2022
1 parent 3c55e9a commit cc715d9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ abstract class AndroidSdkBase
{
// When this changes, update the test: Xamarin.Android.Tools.Tests.AndroidSdkInfoTests.Ndk_MultipleNdkVersionsInSdk
const int MinimumCompatibleNDKMajorVersion = 16;
const int MaximumCompatibleNDKMajorVersion = 24;
const int MaximumCompatibleNDKMajorVersion = 25;

static readonly char[] SourcePropertiesKeyValueSplit = new char[] { '=' };

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ public void Ndk_MultipleNdkVersionsInSdk ()
{
// Must match like-named constants in AndroidSdkBase
const int MinimumCompatibleNDKMajorVersion = 16;
const int MaximumCompatibleNDKMajorVersion = 24;
const int MaximumCompatibleNDKMajorVersion = 25;

CreateSdks(out string root, out string jdk, out string ndk, out string sdk);

Expand All @@ -94,9 +94,10 @@ public void Ndk_MultipleNdkVersionsInSdk ()
"22.1.7171670",
"23.1.7779620",
"24.0.8215888",
"25.0.3735928559", // 0xdeadbeef
"25.0.8775105",
"26.0.3735928559", // 0xdeadbeef
};
string expectedVersion = "24.0.8215888";
string expectedVersion = "25.0.8775105";
string expectedNdkPath = Path.Combine (sdk, "ndk", expectedVersion);

try {
Expand Down

0 comments on commit cc715d9

Please # to comment.