From 396152f0065d3d6d60afc77540fdd78f2bacf8c1 Mon Sep 17 00:00:00 2001 From: Nick Biryulin Date: Wed, 11 Oct 2023 11:05:21 -0700 Subject: [PATCH] Add unix shebang for executables generated by analysis tests See #18940 for more details. I've also wanted to update the tests, but it seems that this code doesn't have any test coverage (or I couldn't find it). To be fair, it is probably simple enough to not need it. Closes #19780. PiperOrigin-RevId: 572627238 Change-Id: I3685e782833712199a2696ac67477b9f2bb2f7ae --- .../build/lib/analysis/test/AnalysisTestActionBuilder.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/main/java/com/google/devtools/build/lib/analysis/test/AnalysisTestActionBuilder.java b/src/main/java/com/google/devtools/build/lib/analysis/test/AnalysisTestActionBuilder.java index d119a9d3757c0e..7e718cfceef8cd 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/test/AnalysisTestActionBuilder.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/test/AnalysisTestActionBuilder.java @@ -47,6 +47,8 @@ public static void writeAnalysisTestAction( StringBuilder sb = new StringBuilder(); if (ruleContext.isExecutedOnWindows()) { sb.append("@echo off\n"); + } else { + sb.append("#!/bin/sh\n"); } for (String line : Splitter.on("\n").split(escapedMessage)) { sb.append("echo ").append(line).append("\n");