aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndy Doan <andy.doan@linaro.org>2016-01-15 14:00:53 -0600
committerAndy Doan <andy.doan@linaro.org>2016-01-15 14:00:53 -0600
commit0cb734455251f407f150914d66917bffc02969d0 (patch)
treeae88aa9249765c8be0935f463fa4ca2b00e81ee4
parent873d404a293fbafd587bb109b0ee55851a320e85 (diff)
downloadlinaro-git-tools-0cb734455251f407f150914d66917bffc02969d0.tar.gz
bug #1993: fix extraneous debug output
In the event no user was found in crowd, we should not be displaying the group membership warning. The old version of the script would have still printed out the message on the KeyError case in get_crowd_user, but I think the current "ERROR User does "... is sufficient given how unlikely that is to occur. Change-Id: I3764034a7dc51d21c22e421202956a04d97e6f13
-rwxr-xr-xgitolite-tools/gitolite-groups2
1 files changed, 1 insertions, 1 deletions
diff --git a/gitolite-tools/gitolite-groups b/gitolite-tools/gitolite-groups
index c5434bb..a6c8008 100755
--- a/gitolite-tools/gitolite-groups
+++ b/gitolite-tools/gitolite-groups
@@ -88,7 +88,7 @@ def get_groups(user):
sys.stderr.write(e)
crowd_error = True
- if crowd_error or not user:
+ if crowd_error:
sys.stderr.write("gitolite-groups: Warning: Group memberships "
"unavailble, access to some repositories may "
"be blocked.\n")