Fix

.github/workflows/test.yml

@@ -41,10 +41,10 @@ jobs:
       run: |
         go version
         go test ./...
-  test-conventions:
+  code-checks:
     runs-on: ubuntu-latest
     steps:
     - name: Checkout code
       uses: actions/checkout@v2
-    - name: Test that on LF line endings are used
+    - name: Test that only LF line endings are used
       run: bash crlf_test.sh

crlf_test.sh

@@ -1,27 +1,19 @@
 #!/bin/bash
-echo "searching for CRLF endings in: ."
-
-BOLD_RED='\033[1;31m'
-BOLD_GREEN='\033[1;32m'
-NC='\033[0m'
-
-ERROR_COUNT=0
-
 if \
-  grep \
-    --recursive \
-    --files-with-matches \
-    --binary \
-    --binary-files=without-match \
-    --max-count=1 \
-    --exclude-dir="\.git" \
-    $'\r' \
-    . \
-  ; then
-  # TODO exit status should be number of files with wrong endings found
-  echo -e "${BOLD_RED}Found at least a file with CRLF endings.${NC}"
-  exit 1
+	grep \
+		--recursive \
+		--files-with-matches \
+		--binary \
+		--binary-files=without-match \
+		--max-count=1 \
+		--exclude-dir="\.git" \
+		$'\r' \
+		. \
+	; then
+	# TODO exit status should be number of files with wrong endings found
+	echo -e "Found at least a file with CRLF endings."
+	exit 1
 fi
 
-echo -e "${BOLD_GREEN}No files with CRLF endings found.${NC}"
+echo -e "No files with CRLF endings found."
 exit 0

testdata/scripts/tiny.txt

@@ -1,41 +1,41 @@
-env TINY_PATTERN='^\/\/line :1$'
-env DEFAULT_PATTERN='^\/\/line \w\.go:[1-9][0-9]*$'
-env DEFAULT_STACK_PATTERN='^\t\w\.go:[1-9][0-9]*(\s\+0x[0-9a-f]+)?'
-env TINY_STACK_PATTERN='^\t\?\?:[0-9][0-9]*(\s\+0x[0-9a-f]+)?$'
-
-# Tiny mode
-garble -tiny -debugdir=.obf-src build
-
-grep $TINY_PATTERN .obf-src/main/main.go
-! grep $DEFAULT_PATTERN .obf-src/main/main.go
-
-! exec ./main$exe
-! stderr 'main\.go'
-! stderr $DEFAULT_STACK_PATTERN
-stderr $TINY_STACK_PATTERN
-
-[short] stop # no need to verify this with -short
-
-# Default mode
-
-garble -debugdir=.obf-src build
-
-# Check for file name leak protection
-grep $TINY_PATTERN .obf-src/main/main.go
-
-# Check for default line obfuscation
-grep $DEFAULT_PATTERN .obf-src/main/main.go
-
-! exec ./main$exe
-! stderr 'main\.go'
-! stderr $TINY_STACK_PATTERN
-stderr $DEFAULT_STACK_PATTERN
-
--- go.mod --
-module main
--- main.go --
-package main
-
-func main() {
-	panic("Test")
+env TINY_PATTERN='^\/\/line :1$'
+env DEFAULT_PATTERN='^\/\/line \w\.go:[1-9][0-9]*$'
+env DEFAULT_STACK_PATTERN='^\t\w\.go:[1-9][0-9]*(\s\+0x[0-9a-f]+)?'
+env TINY_STACK_PATTERN='^\t\?\?:[0-9][0-9]*(\s\+0x[0-9a-f]+)?$'
+
+# Tiny mode
+garble -tiny -debugdir=.obf-src build
+
+grep $TINY_PATTERN .obf-src/main/main.go
+! grep $DEFAULT_PATTERN .obf-src/main/main.go
+
+! exec ./main$exe
+! stderr 'main\.go'
+! stderr $DEFAULT_STACK_PATTERN
+stderr $TINY_STACK_PATTERN
+
+[short] stop # no need to verify this with -short
+
+# Default mode
+
+garble -debugdir=.obf-src build
+
+# Check for file name leak protection
+grep $TINY_PATTERN .obf-src/main/main.go
+
+# Check for default line obfuscation
+grep $DEFAULT_PATTERN .obf-src/main/main.go
+
+! exec ./main$exe
+! stderr 'main\.go'
+! stderr $TINY_STACK_PATTERN
+stderr $DEFAULT_STACK_PATTERN
+
+-- go.mod --
+module main
+-- main.go --
+package main
+
+func main() {
+	panic("Test")
 }