githooks: add check for file size into pre-commit hook

1 files changed, 26 insertions, 4 deletions

diff --git a/.githooks/pre-commit b/.githooks/pre-commit
index b286691a3d..3901bc09de 100755
--- a/.githooks/pre-commit
+++ b/.githooks/pre-commit
@@ -5,6 +5,10 @@
 #
 set -eu
 
+# Maximal allowed size (in bytes) of a file.
+FILE_SIZE_LIMIT=262144  # 256 kiB
+
+
 if ! command -v sha512sum >/dev/null; then
 	# macOS / BSDs (?) don't have sha512sum, but shasum.
 	alias sha512sum='shasum -a 512'
@@ -14,13 +18,13 @@ error() {
 	printf '\033[0;31mpre-commit:\033[0m %s\n' "$1" >&2  # red
 }
 
-# Prints paths of created or modified APKBUILDs being committed.
-changed_apkbuilds() {
+# Prints paths of created or modified files being committed.
+changed_files() {
 	git diff-index \
 		--name-only \
 		--cached \
 		--diff-filter=ACMR HEAD \
-		-- '**/APKBUILD'
+		-- "$@"
 }
 
 # Prints file names and checksums (in format <SHA-512>:<filename>) of local
@@ -79,7 +83,25 @@ check_local_sources() {
 	return $status
 }
 
+# Checks if the file ($1) being committed is not bigger than FILE_SIZE_LIMIT.
+check_file_size() {
+	local path="$1"
+	local size
+
+	size=$(git cat-file -s ":$path")
+	if [ $size -gt $FILE_SIZE_LIMIT ]; then
+		local size_kb=$(( size / 1024 ))
 
-for apkbuild in $(changed_apkbuilds); do
+		error "file \"$path\" is quite big ($(( size / 1024 )) kiB), better to upload it to dev.alpinelinux.org"
+		return 1
+	fi
+}
+
+
+for apkbuild in $(changed_files '**/APKBUILD'); do
 	check_local_sources "$apkbuild"
 done
+
+for path in $(changed_files); do
+	check_file_size "$path"
+done