[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