threads / patch / 38506

patch, 2 partscheck-builtins: Strip any executable suffix to make it work on Windows

Subject: [PATCH 2/2] check-builtins: Strip any executable suffix to make it work on Windows

## tl;dr

2 messages between Feb 5, 2015 and Feb 5, 2015. Diffs are folded; open one to read it.

replies: 1people: 2as markdown or json

Sebastian Schuberth· Feb 5, 2015, 13:28 UTC · lore

Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
---
 check-builtins.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to check-builtins.sh +1 −1

diff --git a/check-builtins.sh b/check-builtins.sh
index 07cff69..a0aaf3a 100755
--- a/check-builtins.sh
+++ b/check-builtins.sh
@@ -3,7 +3,7 @@
 {
     cat <<\EOF
 sayIt:
-    $(foreach b,$(BUILT_INS),echo XXX $b YYY;)
+    $(foreach b,$(BUILT_INS),echo XXX $(b:$X=) YYY;)
 EOF
     cat Makefile
 } |

-- 
2.1.2-mingw-1

Junio C Hamano· Feb 5, 2015, 20:01 UTC · re: Sebastian Schuberth · lore

Re: [PATCH 2/2] check-builtins: Strip any executable suffix to make it work on Windows

Sebastian Schuberth <sschuberth@gmail.com> writes:

Show 16 quoted lines

> Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
> ---
>  check-builtins.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/check-builtins.sh b/check-builtins.sh
> index 07cff69..a0aaf3a 100755
> --- a/check-builtins.sh
> +++ b/check-builtins.sh
> @@ -3,7 +3,7 @@
>  {
>      cat <<\EOF
>  sayIt:
> -    $(foreach b,$(BUILT_INS),echo XXX $b YYY;)
> +    $(foreach b,$(BUILT_INS),echo XXX $(b:$X=) YYY;)
>  EOF

Makes sense; thanks.