diff options
| author | Kari Oikarinen <kari.oikarinen@qt.io> | 2019-06-10 15:09:25 +0300 |
|---|---|---|
| committer | Kari Oikarinen <kari.oikarinen@qt.io> | 2019-06-11 10:25:42 +0000 |
| commit | 61191bf37d3e03f190943725c2c99d9c5fcb3fdf (patch) | |
| tree | c7e1bb93ed43d7d958b187b36b5106689f343563 /git-hooks/gerrit-bot | |
| parent | 53caba0307b908f7008e8be8588d5b734f946074 (diff) | |
gerrit-bot: Give the project when adding reviewers to a change
Work around an issue where set-reviewers call causes an internal server error
due to a NullPointerException. Giving the project explicitly avoids that error.
Task-number: QTQAINFRA-3035
Change-Id: I466a85eb4bf7a7a14a5054e3436b16159e75f4c1
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Reviewed-by: Orgad Shaneh <orgads@gmail.com>
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Diffstat (limited to 'git-hooks/gerrit-bot')
| -rwxr-xr-x | git-hooks/gerrit-bot | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/git-hooks/gerrit-bot b/git-hooks/gerrit-bot index 6697d91..c46fbfa 100755 --- a/git-hooks/gerrit-bot +++ b/git-hooks/gerrit-bot @@ -226,7 +226,7 @@ sub process_commit($$$$$) $$verdict{notify} = "NONE"; } if (@invite) { - if (system(@gerrit_actor, "set-reviewers", (map { ('-a', $_) } @invite), '--', $rev)) { + if (system(@gerrit_actor, "set-reviewers", (map { ('-a', $_) } @invite), '-p', $project, '--', $rev)) { print "===== ".strftime("%c", localtime(time()))." ===== invitation FAILED\n"; printerr("Inviting reviewers to ".$rev." (".$project."/".$ref.") failed"); } else { |