[coreboot-gerrit] Change in ...coreboot[master]: util/release/genrelnotes: Don't assume we're on origin/master

Patrick Georgi (Code Review) gerrit at coreboot.org
Thu Dec 20 17:27:09 CET 2018


Hello Martin Roth,

I'd like you to do a code review. Please visit

    https://review.coreboot.org/c/coreboot/+/30341

to review the following change.


Change subject: util/release/genrelnotes: Don't assume we're on origin/master
......................................................................

util/release/genrelnotes: Don't assume we're on origin/master

A release may be done from an older commit. It's also not a problem as
commits are stored in the reflog (unlike local changes that the script
guards properly).

Change-Id: I26f1c16c1cdfc9e77e28528b3327ce30c5b82b19
Signed-off-by: Patrick Georgi <pgeorgi at google.com>
---
M util/release/genrelnotes
1 file changed, 1 insertion(+), 2 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/30341/1

diff --git a/util/release/genrelnotes b/util/release/genrelnotes
index ee4bd25..6d5aa52 100755
--- a/util/release/genrelnotes
+++ b/util/release/genrelnotes
@@ -39,8 +39,7 @@
 fi
 
 # Try to verify that the repo is clean before losing state.
-if ! git diff-index --quiet --cached HEAD 2>/dev/null || \
-	[ "$(git diff origin/master --shortstat 2>/dev/null | tail -n1)" != "" ]; then
+if ! git diff-index --quiet --cached HEAD 2>/dev/null; then
 	echo "ERROR: repo is not clean.  Exiting."
 	exit 1
 fi

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/30341
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I26f1c16c1cdfc9e77e28528b3327ce30c5b82b19
Gerrit-Change-Number: 30341
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181220/3d2a3dfb/attachment-0001.html>


More information about the coreboot-gerrit mailing list