Fix erroneous use of "is"

[larsks]

The is operator checks for identity, not equality. To check for an empty
list, one should use either:

if roles == []:

Or just take advantage of the fact that an empty list is False when used
in a boolean expression, and write:

if not roles:

This commit takes the second option.