Skip to content

Commit 2f7c84b

Browse files
committed
minor changes in build scripts
1 parent 237836d commit 2f7c84b

File tree

5 files changed

+10
-12
lines changed

5 files changed

+10
-12
lines changed

build/build/Tasks/Package/PackageArchive.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ public override void Run(BuildContext context)
1212
{
1313
context.EnsureDirectoryExists(Paths.Native);
1414

15-
var platform = context.Environment.Platform.Family;
15+
var platform = context.Platform;
1616
var runtimes = context.NativeRuntimes[platform];
1717

1818
foreach (var runtime in runtimes)

build/build/Tasks/Package/PackagePrepare.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public override void Run(BuildContext context)
2525
private static void PackPrepareNative(BuildContext context)
2626
{
2727
// publish single file for all native runtimes (self contained)
28-
var platform = context.Environment.Platform.Family;
28+
var platform = context.Platform;
2929
var runtimes = context.NativeRuntimes[platform];
3030

3131
foreach (var runtime in runtimes)
@@ -34,7 +34,7 @@ private static void PackPrepareNative(BuildContext context)
3434

3535
// testing windows and macos artifacts, the linux is tested with docker
3636
if (platform == PlatformFamily.Linux) continue;
37-
if (runtime.EndsWith("arm64")) continue;
37+
if (runtime.EndsWith("arm64")) continue; // TODO: enable when we have arm64 macos agent
3838

3939
context.Information("Validating native lib:");
4040
var nativeExe = outputPath.CombineWithFilePath(context.IsOnWindows ? "gitversion.exe" : "gitversion");
@@ -44,7 +44,7 @@ private static void PackPrepareNative(BuildContext context)
4444

4545
private static DirectoryPath PackPrepareNative(BuildContext context, string runtime)
4646
{
47-
var platform = context.Environment.Platform.Family;
47+
var platform = context.Platform;
4848
var outputPath = Paths.Native.Combine(platform.ToString().ToLower()).Combine(runtime);
4949

5050
var settings = new DotNetPublishSettings

build/build/Tasks/Test/UnitTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public override void Finally(BuildContext context)
5353
var data = new AzurePipelinesPublishTestResultsData
5454
{
5555
TestResultsFiles = testResultsFiles.ToArray(),
56-
Platform = context.Environment.Platform.Family.ToString(),
56+
Platform = context.Platform.ToString(),
5757
TestRunner = AzurePipelinesTestRunnerType.JUnit
5858
};
5959
context.BuildSystem().AzurePipelines.Commands.PublishTestResults(data);

build/common/Utilities/BuildContextBase.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,9 @@ namespace Common.Utilities;
22

33
public class BuildContextBase : FrostingContext
44
{
5-
protected BuildContextBase(ICakeContext context) : base(context)
6-
{
7-
}
5+
protected BuildContextBase(ICakeContext context) : base(context) => Platform = Environment.Platform.Family;
6+
public PlatformFamily Platform { get; set; }
87
public BuildVersion? Version { get; set; }
9-
108
public bool IsOriginalRepo { get; set; }
119
public string BranchName { get; set; } = string.Empty;
1210
public string RepositoryName { get; set; } = string.Empty;

build/common/Utilities/DockerContextExtensions.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ private static void DockerTestRun(this BuildContextBase context, string image, A
153153
{
154154
var settings = GetDockerRunSettings(context, arch);
155155
context.Information($"Testing image: {image}");
156-
var output = context.DockerRunImage(settings, image, command, args);
156+
var output = context.DockerRun(settings, image, command, args);
157157
context.Information("Output : " + output);
158158

159159
Assert.NotNull(context.Version?.GitVersion);
@@ -235,7 +235,7 @@ private static DockerContainerRunSettings GetDockerRunSettings(this BuildContext
235235

236236
return settings;
237237
}
238-
private static string DockerRunImage(this ICakeContext context, DockerContainerRunSettings settings, string image, string command, params string[] args)
238+
/*private static string DockerRunImage(this ICakeContext context, DockerContainerRunSettings settings, string image, string command, params string[] args)
239239
{
240240
if (string.IsNullOrEmpty(image))
241241
{
@@ -257,5 +257,5 @@ private static string DockerRunImage(this ICakeContext context, DockerContainerR
257257
258258
var result = runner.RunWithResult("run", settings, r => r.ToArray(), arguments.ToArray());
259259
return string.Join("\n", result);
260-
}
260+
}*/
261261
}

0 commit comments

Comments
 (0)