From db910bb25dff82d120a4219f38e1b22979baefb0 Mon Sep 17 00:00:00 2001 From: Michael Bianco Date: Sat, 16 Nov 2024 10:40:49 -0700 Subject: [PATCH 1/2] docs: note that set shell is not used for [script] this was surprising functionality for me --- README.md | 1 + 1 file changed, 1 insertion(+) diff --git a/README.md b/README.md index b318a5dbcb..52f86a6bec 100644 --- a/README.md +++ b/README.md @@ -2757,6 +2757,7 @@ recipes, such as the use of `cygpath` on Windows, the need to use Recipes with an empty `[script]` attribute are executed with the value of `set script-interpreter := […]`1.33.0, defaulting to `sh -eu`. +Note that the `set shell` value is *not* used for `[script]` recipes. The body of the recipe is evaluated, written to disk in the temporary directory, and run by passing its path as an argument to `COMMAND`. From 3eed395fa191f0071d1b483c2481ecfe80a651e0 Mon Sep 17 00:00:00 2001 From: Casey Rodarmor Date: Sat, 16 Nov 2024 15:56:13 -0800 Subject: [PATCH 2/2] Tweak --- README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 52f86a6bec..9ae33a6284 100644 --- a/README.md +++ b/README.md @@ -2755,9 +2755,9 @@ scripts interpreted by `COMMAND`. This avoids some of the issues with shebang recipes, such as the use of `cygpath` on Windows, the need to use `/usr/bin/env`, and inconsistences in shebang line splitting across Unix OSs. -Recipes with an empty `[script]` attribute are executed with the value of -`set script-interpreter := […]`1.33.0, defaulting to `sh -eu`. -Note that the `set shell` value is *not* used for `[script]` recipes. +Recipes with an empty `[script]` attribute are executed with the value of `set +script-interpreter := […]`1.33.0, defaulting to `sh -eu`, and *not* +the value of `set shell`. The body of the recipe is evaluated, written to disk in the temporary directory, and run by passing its path as an argument to `COMMAND`.