From 3ec337484b80647ee23217533c1d62e914afa4f1 Mon Sep 17 00:00:00 2001 From: kcbanner Date: Thu, 13 Jul 2023 12:58:42 -0400 Subject: [PATCH] linker tests: add missing dependOn calls for CheckObject steps --- test/link/macho/dead_strip/build.zig | 2 ++ test/link/macho/dylib/build.zig | 1 + test/link/macho/entry/build.zig | 1 + test/link/macho/entry_in_dylib/build.zig | 1 + test/link/macho/needed_library/build.zig | 1 + test/link/macho/search_strategy/build.zig | 1 + test/link/macho/stack_size/build.zig | 1 + test/link/macho/strict_validation/build.zig | 1 + test/link/macho/unwind_info/build.zig | 1 + test/link/macho/weak_library/build.zig | 1 + 10 files changed, 11 insertions(+) diff --git a/test/link/macho/dead_strip/build.zig b/test/link/macho/dead_strip/build.zig index 5ca3e5f89f194a10debc957925e4803c25415fdb..4489fdf3ad8946f863c6225c7e216aa31bfe3082 100644 --- a/test/link/macho/dead_strip/build.zig +++ b/test/link/macho/dead_strip/build.zig @@ -16,6 +16,7 @@ pub fn build(b: *std.Build) void { const check = exe.checkObject(); check.checkInSymtab(); check.checkNext("{*} (__TEXT,__text) external _iAmUnused"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; @@ -31,6 +32,7 @@ pub fn build(b: *std.Build) void { const check = exe.checkObject(); check.checkInSymtab(); check.checkNotPresent("{*} (__TEXT,__text) external _iAmUnused"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/dylib/build.zig b/test/link/macho/dylib/build.zig index 5fbf73dd1b89fa3f5e95defb819f306a1e272c17..499a0089e1b59e0d8c8754899bddd1a8a32be004 100644 --- a/test/link/macho/dylib/build.zig +++ b/test/link/macho/dylib/build.zig @@ -53,6 +53,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize check_exe.checkStart("cmd RPATH"); check_exe.checkNextFileSource("path", dylib.getOutputDirectorySource()); + test_step.dependOn(&check_exe.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/entry/build.zig b/test/link/macho/entry/build.zig index 454956ad4134e21ae581a57d2758e677a5d52497..1e0c146e114eb80a4da1d93642c58950e9c3fa44 100644 --- a/test/link/macho/entry/build.zig +++ b/test/link/macho/entry/build.zig @@ -34,6 +34,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize check_exe.checkNext("{n_value} (__TEXT,__text) external _non_main"); check_exe.checkComputeCompare("vmaddr entryoff +", .{ .op = .eq, .value = .{ .variable = "n_value" } }); + test_step.dependOn(&check_exe.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/entry_in_dylib/build.zig b/test/link/macho/entry_in_dylib/build.zig index acb26efcebe60ec325b39cb6dfe1282d34c3ee54..cf1d028706fd9daba52bb323718037b845eb5744 100644 --- a/test/link/macho/entry_in_dylib/build.zig +++ b/test/link/macho/entry_in_dylib/build.zig @@ -47,6 +47,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize .op = .eq, .value = .{ .variable = "stubs_vmaddr" }, // The entrypoint should be a synthetic stub }); + test_step.dependOn(&check_exe.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/needed_library/build.zig b/test/link/macho/needed_library/build.zig index d39166c18b5b95898aff5724b3dd9203d3819ef3..4b7dbfd1e054efd8e0b6dd6c980129df7e7254d1 100644 --- a/test/link/macho/needed_library/build.zig +++ b/test/link/macho/needed_library/build.zig @@ -41,6 +41,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize const check = exe.checkObject(); check.checkStart("cmd LOAD_DYLIB"); check.checkNext("name @rpath/liba.dylib"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/search_strategy/build.zig b/test/link/macho/search_strategy/build.zig index 336ca593b14f628bad1597ebab640048505fd8b6..b30540b8181348e521391f4623c8243c68bb1577 100644 --- a/test/link/macho/search_strategy/build.zig +++ b/test/link/macho/search_strategy/build.zig @@ -23,6 +23,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize const check = exe.checkObject(); check.checkStart("cmd LOAD_DYLIB"); check.checkNext("name @rpath/libsearch_dylibs_first.dylib"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/stack_size/build.zig b/test/link/macho/stack_size/build.zig index 219d65cdb84be4af36df9b26340eccf4c6f4e506..81ca05254531fd6e7d3b9a195997a07e25ce66ed 100644 --- a/test/link/macho/stack_size/build.zig +++ b/test/link/macho/stack_size/build.zig @@ -27,6 +27,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize const check_exe = exe.checkObject(); check_exe.checkStart("cmd MAIN"); check_exe.checkNext("stacksize 100000000"); + test_step.dependOn(&check_exe.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/strict_validation/build.zig b/test/link/macho/strict_validation/build.zig index d75b4c58201971f8ecf3d44da4154a6c6de78cc5..fd62a4712e3cb96307ce4fee98d992f86d3f1cf2 100644 --- a/test/link/macho/strict_validation/build.zig +++ b/test/link/macho/strict_validation/build.zig @@ -121,6 +121,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize }, else => unreachable, } + test_step.dependOn(&check_exe.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/unwind_info/build.zig b/test/link/macho/unwind_info/build.zig index 6f19acef3897a5f594e1733b9658e4d040520908..bc0373e69371d62d4f4939dba63b9680f9dadc74 100644 --- a/test/link/macho/unwind_info/build.zig +++ b/test/link/macho/unwind_info/build.zig @@ -46,6 +46,7 @@ fn testUnwindInfo( check.checkInSymtab(); check.checkNext("{*} (__TEXT,__text) external ___gxx_personality_v0"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; diff --git a/test/link/macho/weak_library/build.zig b/test/link/macho/weak_library/build.zig index 81a694be9b4cd33da841128ef180abab0f64452c..79b7c748b88b7a426e9c6bf95fbee30be1c136a1 100644 --- a/test/link/macho/weak_library/build.zig +++ b/test/link/macho/weak_library/build.zig @@ -45,6 +45,7 @@ fn add(b: *std.Build, test_step: *std.Build.Step, optimize: std.builtin.Optimize check.checkInSymtab(); check.checkNext("(undefined) weak external _asStr (from liba)"); + test_step.dependOn(&check.step); const run = b.addRunArtifact(exe); run.skip_foreign_checks = true; -- 2.54.0