Return non-zero exit code if the command cannot be completed successfully.
Fixes #5084
This commit is contained in:
parent
e340879d3c
commit
47a2ab9cfc
@ -16,6 +16,8 @@
|
|||||||
|
|
||||||
from __future__ import print_function
|
from __future__ import print_function
|
||||||
|
|
||||||
|
import sys
|
||||||
|
|
||||||
import six
|
import six
|
||||||
|
|
||||||
from mediagoblin.gmg_commands import util as commands_util
|
from mediagoblin.gmg_commands import util as commands_util
|
||||||
@ -50,6 +52,7 @@ def adduser(args):
|
|||||||
|
|
||||||
if users_with_username:
|
if users_with_username:
|
||||||
print(u'Sorry, a user with that name already exists.')
|
print(u'Sorry, a user with that name already exists.')
|
||||||
|
sys.exit(1)
|
||||||
|
|
||||||
else:
|
else:
|
||||||
# Create the user
|
# Create the user
|
||||||
@ -95,6 +98,7 @@ def makeadmin(args):
|
|||||||
print(u'The user is now Admin')
|
print(u'The user is now Admin')
|
||||||
else:
|
else:
|
||||||
print(u'The user doesn\'t exist')
|
print(u'The user doesn\'t exist')
|
||||||
|
sys.exit(1)
|
||||||
|
|
||||||
|
|
||||||
def changepw_parser_setup(subparser):
|
def changepw_parser_setup(subparser):
|
||||||
@ -119,6 +123,7 @@ def changepw(args):
|
|||||||
print(u'Password successfully changed')
|
print(u'Password successfully changed')
|
||||||
else:
|
else:
|
||||||
print(u'The user doesn\'t exist')
|
print(u'The user doesn\'t exist')
|
||||||
|
sys.exit(1)
|
||||||
|
|
||||||
|
|
||||||
def deleteuser_parser_setup(subparser):
|
def deleteuser_parser_setup(subparser):
|
||||||
@ -139,3 +144,4 @@ def deleteuser(args):
|
|||||||
print('The user %s has been deleted' % args.username)
|
print('The user %s has been deleted' % args.username)
|
||||||
else:
|
else:
|
||||||
print('The user %s doesn\'t exist' % args.username)
|
print('The user %s doesn\'t exist' % args.username)
|
||||||
|
sys.exit(1)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user