From 5dd4466ab32d55f1369ad6a30670b3559ff41e11 Mon Sep 17 00:00:00 2001 From: Chris Date: Wed, 25 Oct 2017 11:33:19 -0500 Subject: [PATCH] add missing license headers, test to makefile (#7711) --- Makefile | 5 ++++- cmd/platform/init.go | 3 +++ model/bundle_info.go | 3 +++ model/manifest.go | 3 +++ plugin/api.go | 3 +++ plugin/hooks.go | 3 +++ plugin/pluginenv/environment.go | 3 +++ plugin/pluginenv/options.go | 3 +++ plugin/pluginenv/search_path.go | 3 +++ plugin/plugintest/api.go | 3 +++ plugin/plugintest/hooks.go | 3 +++ plugin/rpcplugin/api.go | 3 +++ plugin/rpcplugin/hooks.go | 3 +++ plugin/rpcplugin/http.go | 3 +++ plugin/rpcplugin/io.go | 3 +++ plugin/rpcplugin/ipc.go | 3 +++ plugin/rpcplugin/main.go | 3 +++ plugin/rpcplugin/muxer.go | 3 +++ plugin/rpcplugin/process.go | 3 +++ plugin/rpcplugin/process_unix.go | 3 +++ plugin/rpcplugin/supervisor.go | 3 +++ plugin/supervisor.go | 3 +++ scripts/license-check.sh | 28 ++++++++++++++++++++++++++++ store/constants.go | 3 +++ store/sqlstore/file_info_store.go | 1 + utils/i18n.go | 3 +++ utils/inbucket.go | 3 +++ utils/logger/log4go_json_writer.go | 3 +++ utils/logger/logger.go | 3 +++ utils/time.go | 3 +++ 30 files changed, 114 insertions(+), 1 deletion(-) create mode 100755 scripts/license-check.sh diff --git a/Makefile b/Makefile index ac00067ac4..a32743ca6e 100644 --- a/Makefile +++ b/Makefile @@ -271,7 +271,10 @@ gofmt: done @echo "gofmt success"; \ -check-style: govet gofmt +check-licenses: + ./scripts/license-check.sh $(TE_PACKAGES) $(EE_PACKAGES) + +check-style: govet gofmt check-licenses test-te-race: @echo Testing TE race conditions diff --git a/cmd/platform/init.go b/cmd/platform/init.go index 145c754d1d..a5db40a572 100644 --- a/cmd/platform/init.go +++ b/cmd/platform/init.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package main import ( diff --git a/model/bundle_info.go b/model/bundle_info.go index 67b5dd0ed7..6965159ccb 100644 --- a/model/bundle_info.go +++ b/model/bundle_info.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package model type BundleInfo struct { diff --git a/model/manifest.go b/model/manifest.go index b466660af2..4352a2bd21 100644 --- a/model/manifest.go +++ b/model/manifest.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package model import ( diff --git a/plugin/api.go b/plugin/api.go index 4df6cb30a5..e9a2892fc7 100644 --- a/plugin/api.go +++ b/plugin/api.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package plugin import ( diff --git a/plugin/hooks.go b/plugin/hooks.go index 7f0d8ae3c0..c5daec45bc 100644 --- a/plugin/hooks.go +++ b/plugin/hooks.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package plugin import ( diff --git a/plugin/pluginenv/environment.go b/plugin/pluginenv/environment.go index 37bd1a482f..62eb20e9d5 100644 --- a/plugin/pluginenv/environment.go +++ b/plugin/pluginenv/environment.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + // Package pluginenv provides high level functionality for discovering and launching plugins. package pluginenv diff --git a/plugin/pluginenv/options.go b/plugin/pluginenv/options.go index 827520fac4..35ecf1d7f8 100644 --- a/plugin/pluginenv/options.go +++ b/plugin/pluginenv/options.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package pluginenv import ( diff --git a/plugin/pluginenv/search_path.go b/plugin/pluginenv/search_path.go index f8617347d6..6984243320 100644 --- a/plugin/pluginenv/search_path.go +++ b/plugin/pluginenv/search_path.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package pluginenv import ( diff --git a/plugin/plugintest/api.go b/plugin/plugintest/api.go index 6c8646c23d..c0e77648b1 100644 --- a/plugin/plugintest/api.go +++ b/plugin/plugintest/api.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package plugintest import ( diff --git a/plugin/plugintest/hooks.go b/plugin/plugintest/hooks.go index 721a709ea8..56d048d6a4 100644 --- a/plugin/plugintest/hooks.go +++ b/plugin/plugintest/hooks.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package plugintest import ( diff --git a/plugin/rpcplugin/api.go b/plugin/rpcplugin/api.go index 4195a4f86a..98333b1d9b 100644 --- a/plugin/rpcplugin/api.go +++ b/plugin/rpcplugin/api.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/hooks.go b/plugin/rpcplugin/hooks.go index 18e4a6672f..22f26e22ef 100644 --- a/plugin/rpcplugin/hooks.go +++ b/plugin/rpcplugin/hooks.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/http.go b/plugin/rpcplugin/http.go index cfcb7419d6..72b1aa445c 100644 --- a/plugin/rpcplugin/http.go +++ b/plugin/rpcplugin/http.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/io.go b/plugin/rpcplugin/io.go index f4fdd3a1b4..44b89956c2 100644 --- a/plugin/rpcplugin/io.go +++ b/plugin/rpcplugin/io.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/ipc.go b/plugin/rpcplugin/ipc.go index 3e6c89c4f2..e8dd43c04d 100644 --- a/plugin/rpcplugin/ipc.go +++ b/plugin/rpcplugin/ipc.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/main.go b/plugin/rpcplugin/main.go index 241d70bc95..96a61c068b 100644 --- a/plugin/rpcplugin/main.go +++ b/plugin/rpcplugin/main.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/muxer.go b/plugin/rpcplugin/muxer.go index 393a122c4d..8a8ce89330 100644 --- a/plugin/rpcplugin/muxer.go +++ b/plugin/rpcplugin/muxer.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/process.go b/plugin/rpcplugin/process.go index 4b3362d687..a795be133e 100644 --- a/plugin/rpcplugin/process.go +++ b/plugin/rpcplugin/process.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/rpcplugin/process_unix.go b/plugin/rpcplugin/process_unix.go index f196e34f85..142043cc63 100644 --- a/plugin/rpcplugin/process_unix.go +++ b/plugin/rpcplugin/process_unix.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + // +build !windows package rpcplugin diff --git a/plugin/rpcplugin/supervisor.go b/plugin/rpcplugin/supervisor.go index 7e37e28518..7b036d726e 100644 --- a/plugin/rpcplugin/supervisor.go +++ b/plugin/rpcplugin/supervisor.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package rpcplugin import ( diff --git a/plugin/supervisor.go b/plugin/supervisor.go index 5ddf5f1695..f3e576e992 100644 --- a/plugin/supervisor.go +++ b/plugin/supervisor.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package plugin // Supervisor provides the interface for an object that controls the execution of a plugin. diff --git a/scripts/license-check.sh b/scripts/license-check.sh new file mode 100755 index 0000000000..be3b62508f --- /dev/null +++ b/scripts/license-check.sh @@ -0,0 +1,28 @@ +#!/usr/bin/env bash +set -e +IFS=$'\n' +count=0 +for fileType in GoFiles; do + for file in `go list -f $'{{range .GoFiles}}{{$.Dir}}/{{.}}\n{{end}}' "$@"`; do + case $file in + */utils/lru.go) + # Third-party, doesn't require a header. + ;; + *) + if ! grep 'Mattermost, Inc. All Rights Reserved.' $file -q; then + >&2 echo "FAIL: $file is missing a license header." + ((count++)) + fi + esac + done +done +if [ $count -eq 0 ]; then + exit 0 +fi + +if [ $count -gt 1 ]; then + >&2 echo "$count files are missing license headers." +else + >&2 echo "$count file is missing a license header." +fi +exit 1 diff --git a/store/constants.go b/store/constants.go index f541c97472..6d3c4812e4 100644 --- a/store/constants.go +++ b/store/constants.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package store const ( diff --git a/store/sqlstore/file_info_store.go b/store/sqlstore/file_info_store.go index 1657709621..1d0767d1e2 100644 --- a/store/sqlstore/file_info_store.go +++ b/store/sqlstore/file_info_store.go @@ -1,3 +1,4 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. // See License.txt for license information. package sqlstore diff --git a/utils/i18n.go b/utils/i18n.go index acb4f55bae..71e1aaee16 100644 --- a/utils/i18n.go +++ b/utils/i18n.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package utils import ( diff --git a/utils/inbucket.go b/utils/inbucket.go index f80d16efdf..53f6c759d2 100644 --- a/utils/inbucket.go +++ b/utils/inbucket.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package utils import ( diff --git a/utils/logger/log4go_json_writer.go b/utils/logger/log4go_json_writer.go index cc054dc6c1..ede541b2b3 100644 --- a/utils/logger/log4go_json_writer.go +++ b/utils/logger/log4go_json_writer.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + // glue functions that allow logger.go to leverage log4Go to write JSON-formatted log records to a file package logger diff --git a/utils/logger/logger.go b/utils/logger/logger.go index e81b24e9c7..410a5a5fce 100644 --- a/utils/logger/logger.go +++ b/utils/logger/logger.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + // this is a new logger interface for mattermost package logger diff --git a/utils/time.go b/utils/time.go index baddd73297..5227887f7e 100644 --- a/utils/time.go +++ b/utils/time.go @@ -1,3 +1,6 @@ +// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. +// See License.txt for license information. + package utils import (