On Sat, Aug 04, 2018 at 11:11:47AM +0200, Nicolas Iooss wrote: > "except OSError, ImportError:" does not perform what it says: it is the > Python 2 syntax of catching OSError exceptions as "ImportError" (like > "except OSError, e:"), and this is indeed caught by Python3: > > File "./python/semanage/seobject.py", line 143 > except OSError, ImportError: > ^ > SyntaxError: invalid syntax > > The correct syntax consists in using parentheses. > > Signed-off-by: Nicolas Iooss <nicolas.io...@m4x.org>
Acked-by: Jason Zaman <ja...@perfinion.com> > --- > For the record, I found this while playing with flake8 Python's linter. > I am currently working on upstreaming a script which would run this on > our Travis-CI environment in order to catch Python syntax errors before > they are introduced. Awesome! more linters are always good. I really do need to find the time to kill seobject more thoroughly tho, its so slow compared to direct setools. :( -- Jason > > python/semanage/seobject.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/python/semanage/seobject.py b/python/semanage/seobject.py > index 78ffb091ad2f..c14671855049 100644 > --- a/python/semanage/seobject.py > +++ b/python/semanage/seobject.py > @@ -140,7 +140,7 @@ try: > > self.log_list = [] > self.log_change_list = [] > -except OSError, ImportError: > +except (OSError, ImportError): > class logger: > > def __init__(self): > -- > 2.18.0 > > _______________________________________________ > Selinux mailing list > Selinux@tycho.nsa.gov > To unsubscribe, send email to selinux-le...@tycho.nsa.gov. > To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov. _______________________________________________ Selinux mailing list Selinux@tycho.nsa.gov To unsubscribe, send email to selinux-le...@tycho.nsa.gov. To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov.