From 17d424fc817a2c7531eaeecee50eb9eec585f672 Mon Sep 17 00:00:00 2001
From: catch <catch@35733.no-reply.drupal.org>
Date: Wed, 2 Aug 2023 13:53:31 +0100
Subject: [PATCH] Issue #3377131 by longwave, smustgrave: File mode check in
commit-code-check.sh is too strict
(cherry picked from commit 5052fcddde7744bd1f78f21bbcfe0a502bd44cd0)
---
core/scripts/dev/commit-code-check.sh | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/core/scripts/dev/commit-code-check.sh b/core/scripts/dev/commit-code-check.sh
index 001744596a7d..9a8da6bda791 100755
--- a/core/scripts/dev/commit-code-check.sh
+++ b/core/scripts/dev/commit-code-check.sh
@@ -352,13 +352,8 @@
# Ensure the file still exists (i.e. is not being deleted).
if [ -a $FILE ]; then
if [ ${FILE: -3} != ".sh" ]; then
- # Ensure the file has the correct mode.
- STAT="$(stat -f "%A" $FILE 2>/dev/null)"
- if [ $? -ne 0 ]; then
- STAT="$(stat -c "%a" $FILE 2>/dev/null)"
- fi
- if [ "$STAT" -ne "644" ]; then
- printf "${red}check failed:${reset} file $FILE should be 644 not $STAT\n"
+ if [ -x $FILE ]; then
+ printf "${red}check failed:${reset} file $FILE should not be executable\n"
STATUS=1
fi
fi
--
GitLab