Fix license header CI checks (#1492)

Author: dhrubabasu

LICENSE.header

@@ -1,9 +1,9 @@
-//go:build !windows && !plan9 && !js
-// +build !windows,!plan9,!js
-
 // Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
 
+//go:build !windows && !plan9 && !js
+// +build !windows,!plan9,!js
+
 package socket
 
 import (

ipcs/socket/socket_unix.go

@@ -1,9 +1,9 @@
-//go:build windows
-// +build windows
-
 // Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
 
+//go:build windows
+// +build windows
+
 package socket
 
 import (

ipcs/socket/socket_windows.go

@@ -0,0 +1,4 @@
+header: |
+  // Copyright (C) 2019-{{YEAR}}, Ava Labs, Inc. All rights reserved.
+  // See the file LICENSE for licensing terms.
+  

license.yml

@@ -28,25 +28,17 @@ function test_golangci_lint {
   golangci-lint run --config .golangci.yml
 }
 
-# find_go_files [package]
-# all go files except generated ones
-function find_go_files {
-  local target="${1}"
-  go fmt -n "${target}"  | grep -Eo "([^ ]*)$" | grep -vE "(\\.pb\\.go|\\.pb\\.gw.go)"
-}
-
 # automatically checks license headers
-# to modify the file headers (if missing), remove "--check" flag
-# TESTS='license_header' ADDLICENSE_FLAGS="-v" ./scripts/lint.sh
-_addlicense_flags=${ADDLICENSE_FLAGS:-"--check -v"}
+# to modify the file headers (if missing), remove "--verify" flag
+# TESTS='license_header' ADDLICENSE_FLAGS="--debug" ./scripts/lint.sh
+_addlicense_flags=${ADDLICENSE_FLAGS:-"--verify --debug"}
 function test_license_header {
-  go install -v github.com/google/addlicense@latest
-  local target="${1}"
+  go install -v github.com/palantir/go-license@v1.25.0
   local files=()
-  while IFS= read -r line; do files+=("$line"); done < <(find_go_files "${target}")
+  while IFS= read -r line; do files+=("$line"); done < <(find . -type f -name '*.go' ! -name '*.pb.go' ! -name 'mock_*.go')
 
-  addlicense \
-  -f ./LICENSE.header \
+  go-license \
+  --config=./license.yml \
   ${_addlicense_flags} \
   "${files[@]}"
 }

scripts/lint.sh

@@ -14,14 +14,25 @@ then
   go install -v github.com/golang/mock/mockgen@v1.6.0
 fi
 
+if ! command -v go-license &> /dev/null
+then
+  echo "go-license not found, installing..."
+  # https://github.com/palantir/go-license
+  go install -v github.com/palantir/go-license@v1.25.0
+fi
+
 # tuples of (source interface import path, comma-separated interface names, output file path)
 input="scripts/mocks.mockgen.txt"
 while IFS= read -r line
 do
   IFS='=' read src_import_path interface_name output_path <<< "${line}"
   package_name=$(basename $(dirname $output_path))
   echo "Generating ${output_path}..."
-  mockgen -copyright_file=./LICENSE.header -package=${package_name} -destination=${output_path} ${src_import_path} ${interface_name}
+  mockgen -package=${package_name} -destination=${output_path} ${src_import_path} ${interface_name}
+  
+  go-license \
+  --config=./license.yml \
+  "${output_path}"
 done < "$input"
 
 echo "SUCCESS"

scripts/mock.gen.sh

@@ -1,5 +1,6 @@
 // Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
+
 package block
 
 import (

snow/engine/snowman/block/height_indexed_vm.go

@@ -1,4 +1,4 @@
-// Copyright (C) 2019-2023, Ava Labs, Inte. All rights reserved.
+// Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
 
 package linkedhashmap

utils/linkedhashmap/iterator.go

@@ -1,4 +1,4 @@
-// Copyright (C) 2019-2023, Ava Labs, Inte. All rights reserved.
+// Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
 
 package linkedhashmap

utils/linkedhashmap/linkedhashmap.go

@@ -1,9 +1,9 @@
-//go:build !windows
-// +build !windows
-
 // Copyright (C) 2019-2023, Ava Labs, Inc. All rights reserved.
 // See the file LICENSE for licensing terms.
 
+//go:build !windows
+// +build !windows
+
 package storage
 
 import "syscall"