Just as a side note, this is not the first example, but one of those commits
that made a total rewrite necessary. By quality, the last ones by Ludwig and
Vincent aren't any better either. Ideally, fixing sth. shouldn't be only about
adding lines but about re-factoring lines. Otherwise the next commiter might
be tempted to add his personal 'osc scratchmyback' command w/o communication
too.

@suse-tux: For the osc rewrite, I strongly urge you to enforce peer-reviews
for commits. Or even better, don't let anyone commit directly but let them
file merge requests, which can be reviewed and commented appropriatly.
Otherwise the new osc will end up exactly like the current one, as an
unmaintainable pile of crap code.

We also should think about adapting a similar policy for OBS code, but I fear
this would break quite some odd habbits...

- if not opts.role in ('maintainer', 'bugowner'):
+ role = 'maintainer'
+ if subcmd in ( 'reqbugownership', 'requestbugownership', 'reqbs'
): + role = 'bugowner'
+ if opts.role:
+ role = opts.role
+ if not role in ('maintainer', 'bugowner'):
raise oscerr.WrongOptions('invalid \'--role\': either specify
\'maintainer\' or \'bugowner\'') if not opts.message:
opts.message = edit_message()