Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

Commit 1d6e777

Browse files
committed
DesignTimeBuild no longer needed in verify targets
1 parent 27ab318 commit 1d6e777

File tree

4 files changed

+15
-13
lines changed

4 files changed

+15
-13
lines changed

source/buildtasks/support-annotations-tests/Tests.cs

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
11
using System;
22
using System.Collections.Generic;
33
using System.IO;
4-
using System.Runtime.Serialization.Json;
54
using System.Threading.Tasks;
6-
using System.Web.Script.Serialization;
75
using Xunit;
86

97
namespace Xamarin.Android.Support.BuildTasks.Tests
@@ -47,13 +45,14 @@ public void Test_Should_Detect_Multiple_Versions_Project_Assets()
4745
}
4846

4947
[Theory]
50-
[InlineData(23, "23.x")]
51-
[InlineData(24, "24.x")]
52-
[InlineData(25, "25.4.0.2")]
53-
[InlineData(26, "26.1.0.1")]
48+
[InlineData(23, "23.*")]
49+
[InlineData(24, "24.*")]
50+
[InlineData(25, "25.*")]
51+
[InlineData(26, "26.*")]
52+
[InlineData(29, "28.*")]
5453
public void Test_Recommended_NuGet_Version(int apiLevel, string expectedVersion)
5554
{
56-
var v = NugetPackages.GetRecommendedSupportPackageVersion(apiLevel, false);
55+
var v = NugetPackages.GetRecommendedSupportPackageVersion(apiLevel);
5756

5857
Assert.Equal(expectedVersion, v);
5958
}

source/buildtasks/support-annotations/NugetPackages.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,13 @@ public static string GetSupportVersion(string packageIdPrefix, Dictionary<string
127127
return supportVersion;
128128
}
129129

130-
public static string GetRecommendedSupportPackageVersion(int apiLevel, bool skipNugetQuery)
131-
=> apiLevel.ToString() + ".*";
130+
public static string GetRecommendedSupportPackageVersion(int apiLevel)
131+
{
132+
if (NugetPackages.AndroidApiLevelsAndVersions.ContainsKey(apiLevel))
133+
return apiLevel.ToString() + ".*";
134+
135+
return "28.*"; // There will never be a newer major android support version
136+
}
137+
132138
}
133139
}

source/buildtasks/support-annotations/VerifyVersionsTask.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@ public class VerifyVersionsTask : Task
3939
[Required]
4040
public ITaskItem AndroidSdkBuildToolsVersion { get; set; }
4141

42-
public bool DesignTimeBuild { get;set;}
43-
4442
readonly Dictionary<string, string> packageVersions = new Dictionary<string, string>();
4543

4644
public override bool Execute()
@@ -85,7 +83,7 @@ public override bool Execute()
8583
sb.AppendLine("The following Xamarin.Android.Support.* packages and versions were detected:");
8684
sb.AppendLine();
8785

88-
var recommendedSupportVersion = NugetPackages.GetRecommendedSupportPackageVersion(apiLevel, DesignTimeBuild);
86+
var recommendedSupportVersion = NugetPackages.GetRecommendedSupportPackageVersion(apiLevel);
8987

9088
foreach (var pkg in packageVersions.OrderBy(p => p.Value + "-" + p.Key))
9189
sb.AppendLine($" {pkg.Key} ({pkg.Value})");

source/com.android.support/support-annotations/merge.targets

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
AndroidSdkBuildToolsVersion="$(AndroidSdkBuildToolsVersion)"
2828
ProjectExtensionsPath="$(MSBuildProjectExtensionsPath)"
2929
ProjectPath="$(MSBuildProjectDirectory)"
30-
DesignTimeBuild="$(DesignTimeBuild)"
3130
>
3231
</VerifyVersionsTask>
3332

0 commit comments

Comments
 (0)