Alex Thiessen has uploaded this change for review. ( https://review.coreboot.org/23098
Change subject: util/lint: Check license headers of git hooks
......................................................................
util/lint: Check license headers of git hooks
Now that all the files under util/gitconfig have their license headers,
enable lint-000-license-headers to check the directory too.
Change-Id: I242256f72ac70553535509f83166c6d1ddb16fdc
Signed-off-by: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
---
M util/lint/lint-000-license-headers
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/98/23098/1
diff --git a/util/lint/lint-000-license-headers b/util/lint/lint-000-license-headers
index d8d04be..103c1f0 100755
--- a/util/lint/lint-000-license-headers
+++ b/util/lint/lint-000-license-headers
@@ -23,7 +23,6 @@
^util/kconfig/|\
^util/romcc/tests|\
^util/romcc/results|\
-^util/gitconfig|\
Kconfig|\
\<COPYING\>|\
\<LICENSE\>|\
--
To view, visit https://review.coreboot.org/23098
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I242256f72ac70553535509f83166c6d1ddb16fdc
Gerrit-Change-Number: 23098
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
Alex Thiessen has uploaded this change for review. ( https://review.coreboot.org/23097
Change subject: util/gitconfig: Add missing license headers
......................................................................
util/gitconfig: Add missing license headers
License header for the `gitconfig.sh` was copied from the Makefile it
was extracted from in commit 9ab8ae6a (util/gitconfig: Make gitconfig
a bash script).
License header for the pre-commit hook names Patrik Georgy as the
copyright holder as he is the original author.
Change-Id: Ie051e5e6ae7571050ece383e6be8236ed7d1ddd9
Signed-off-by: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
---
M util/gitconfig/gitconfig.sh
M util/gitconfig/pre-commit
2 files changed, 32 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/23097/1
diff --git a/util/gitconfig/gitconfig.sh b/util/gitconfig/gitconfig.sh
index 5f91a69..2f96fd4 100755
--- a/util/gitconfig/gitconfig.sh
+++ b/util/gitconfig/gitconfig.sh
@@ -1,4 +1,20 @@
#!/bin/bash
+
+##
+## This file is part of the coreboot project.
+##
+## Copyright (C) 2011 secunet Security Networks AG
+##
+## This program is free software; you can redistribute it and/or modify
+## it under the terms of the GNU General Public License as published by
+## the Free Software Foundation; version 2 of the License.
+##
+## This program is distributed in the hope that it will be useful,
+## but WITHOUT ANY WARRANTY; without even the implied warranty of
+## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+## GNU General Public License for more details.
+##
+
if [ ! -d .git ]; then
echo "Error: Not in a git repository"
exit 1
diff --git a/util/gitconfig/pre-commit b/util/gitconfig/pre-commit
index 1c04bc3..ac0db4a 100755
--- a/util/gitconfig/pre-commit
+++ b/util/gitconfig/pre-commit
@@ -1,4 +1,20 @@
#!/bin/sh
+
+##
+## This file is part of the coreboot project.
+##
+## Copyright (C) 2015 Patrick Georgi <patrick(a)georgi-clan.de>
+##
+## This program is free software; you can redistribute it and/or modify
+## it under the terms of the GNU General Public License as published by
+## the Free Software Foundation; version 2 of the License.
+##
+## This program is distributed in the hope that it will be useful,
+## but WITHOUT ANY WARRANTY; without even the implied warranty of
+## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+## GNU General Public License for more details.
+##
+
%MAKE% lint-stable
PATCHDIFF=$(git diff --cached)
--
To view, visit https://review.coreboot.org/23097
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie051e5e6ae7571050ece383e6be8236ed7d1ddd9
Gerrit-Change-Number: 23097
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23096 )
Change subject: util/gitconfig: Use `make` in git hooks
......................................................................
Patch Set 2:
Build Successful
https://qa.coreboot.org/job/coreboot-gerrit/65292/ : SUCCESS
--
To view, visit https://review.coreboot.org/23096
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia78e06567b904b342dc9b7778569201fe02e6897
Gerrit-Change-Number: 23096
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
Gerrit-Reviewer: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 04 Jan 2018 20:34:03 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19918 )
Change subject: nb/intel/x4x: Adapt post JEDEC for DDR3
......................................................................
Patch Set 9:
Build Successful
https://qa.coreboot.org/job/coreboot-checkpatch/20021/ : SUCCESS
https://qa.coreboot.org/job/coreboot-gerrit/65288/ : SUCCESS
--
To view, visit https://review.coreboot.org/19918
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I708f98dc2f36af73bb5933d186b4984649e149a1
Gerrit-Change-Number: 19918
Gerrit-PatchSet: 9
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 04 Jan 2018 20:32:28 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/23096 )
Change subject: util/gitconfig: Use `make` in git hooks
......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/23096/2/util/gitconfig/gitconfig.sh
File util/gitconfig/gitconfig.sh:
https://review.coreboot.org/#/c/23096/2/util/gitconfig/gitconfig.sh@a10
PS2, Line 10:
or add MAKE?=make somewhere and use s,%MAKE%,${MAKE},g here? that way it uses make by default and whatever the build system was called with in case of "make gitconfig".
--
To view, visit https://review.coreboot.org/23096
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia78e06567b904b342dc9b7778569201fe02e6897
Gerrit-Change-Number: 23096
Gerrit-PatchSet: 2
Gerrit-Owner: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
Gerrit-Reviewer: Alex Thiessen <alex.thiessen.de+coreboot(a)gmail.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 04 Jan 2018 20:24:18 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No