Enable Flake8 E722

This check looks for uses of bare 'except:'; these are bad because they
catch e.g. SystemExit, which should not normally be caught.

.flake8

@@ -10,7 +10,6 @@ ignore =
     E305,
     E501,
     E713,
-    E722,
     # W503 and W504 require line breaks after or before binary operators; you
     # can only have one enabled
     W504

tools/gfx_tools/compose.py

@@ -284,7 +284,7 @@ def walk_dirs(self, refs):
                     with open(filepath, "r") as fp:
                         try:
                             tile_entry = json.load(fp)
-                        except:
+                        except Exception:
                             print("error loading {}".format(filepath))
                             raise
 

tools/update_blueprint_needs.py

@@ -99,7 +99,7 @@ def main(argv):
                     try:
                         with open(json_path, 'r', encoding='utf-8') as fs:
                             content = json.load(fs)
-                    except:
+                    except Exception:
                         sys.stderr.write('Error parsing %r\n' % json_path)
                         raise
                 if type(content) is list:

utilities/make_iso.py

@@ -188,19 +188,19 @@ def tile_convert(otile, main_id, new_tile_number):
 
 try:
     os.mkdir(new_tileset_name)
-except:
+except OSError:
     pass
 
 try:
     os.mkdir(os.path.join(new_tileset_name, 'tiles'))
-except:
+except OSError:
     pass
 
 os.system('rm -rf ' + new_tileset_name + '/tiles/*')
 
 try:
     os.mkdir(os.path.join(new_tiles_dir, 'to_merge'))
-except:
+except OSError:
     pass
 
 print('reading ' + old_tileset_name + '/tile_config.json')