use directly OSError everywhere

Author: fchapoton

src/sage/databases/cremona.py

@@ -121,7 +121,7 @@ def build(name, data_tgz, largest_conductor=0, mini=False, decompress=True):
         raise RuntimeError('Please (re)move %s before building ' % db_path
                 + 'database')
     if not os.path.exists(data_tgz):
-        raise IOError("The data file is not at %s" % data_tgz)
+        raise OSError("The data file is not at %s" % data_tgz)
     t = walltime()
 
     if decompress:

src/sage/databases/cubic_hecke_db.py

@@ -918,7 +918,7 @@ def read(self, section):
         try:
             data_lib[section] = load(fname)
             verbose('... finished!')
-        except IOError:
+        except OSError:
             self.reset_library(section)
             verbose('... not found!')
         except (ImportError, ModuleNotFoundError):

src/sage/databases/db_modular_polynomials.py

@@ -60,7 +60,7 @@ def _dbz_to_string(name):
     try:
         with open(filename, 'rb') as f:
             data = bz2.decompress(f.read())
-    except IOError:
+    except OSError:
         raise ValueError('file not found in the Kohel database')
     return bytes_to_str(data)
 

src/sage/databases/findstat.py

@@ -881,7 +881,7 @@ def statistic(pi):
                 code = inspect.getsource(function.f)
             else:
                 code = inspect.getsource(function)
-        except (IOError, TypeError):
+        except (OSError, TypeError):
             verbose("inspect.getsource could not get code from function provided",
                     caller_name='FindStat')
     return code

src/sage/databases/jones.py

@@ -152,7 +152,7 @@ def _init(self, path):
         self.root = {}
         self.root[tuple()] = [x - 1]
         if not os.path.exists(path):
-            raise IOError("Path %s does not exist." % path)
+            raise OSError("Path %s does not exist." % path)
         for X in os.listdir(path):
             if X[-4:] == "solo":
                 Z = path + "/" + X

src/sage/databases/oeis.py

@@ -187,8 +187,8 @@ def _fetch(url):
         result = f.read()
         f.close()
         return bytes_to_str(result)
-    except IOError as msg:
-        raise IOError("%s\nerror fetching %s" % (msg, url))
+    except OSError as msg:
+        raise OSError("%s\nerror fetching %s" % (msg, url))
 
 
 def _urls(html_string):

src/sage/databases/sloane.py

@@ -197,20 +197,20 @@ def install(self, oeis_url="https://oeis.org/stripped.gz",
         """
         # See if the encyclopedia already exists
         if not overwrite and os.path.exists(self.__file__):
-            raise IOError("Sloane encyclopedia is already installed")
+            raise OSError("Sloane encyclopedia is already installed")
 
         tm = verbose("Downloading stripped version of Sloane encyclopedia")
         ssl._create_default_https_context = ssl.create_default_context
         try:
             fname, _ = urlretrieve(oeis_url)
-        except IOError as msg:
-            raise IOError("%s\nError fetching the following website:\n    %s\nTry checking your internet connection." % (msg, oeis_url))
+        except OSError as msg:
+            raise OSError("%s\nError fetching the following website:\n    %s\nTry checking your internet connection." % (msg, oeis_url))
 
         if names_url is not None:
             try:
                 nname, _ = urlretrieve(names_url)
-            except IOError as msg:
-                raise IOError("%s\nError fetching the following website:\n    %s\nTry checking your internet connection." % (msg, names_url))
+            except OSError as msg:
+                raise OSError("%s\nError fetching the following website:\n    %s\nTry checking your internet connection." % (msg, names_url))
         else:
             nname = None
         verbose("Finished downloading", tm)
@@ -237,7 +237,7 @@ def install_from_gz(self, stripped_file, names_file, overwrite=False):
           old encyclopedia.
         """
         if not overwrite and os.path.exists(self.__file__):
-            raise IOError("Sloane encyclopedia is already installed")
+            raise OSError("Sloane encyclopedia is already installed")
 
         copy_gz_file(stripped_file, self.__file__)
 
@@ -262,8 +262,8 @@ def load(self):
             return
         try:
             file_seq = bz2.BZ2File(self.__file__, 'r')
-        except IOError:
-            raise IOError("The Sloane Encyclopedia database must be installed."
+        except OSError:
+            raise OSError("The Sloane Encyclopedia database must be installed."
                           " Use e.g. 'SloaneEncyclopedia.install()' to download and install it.")
 
         self.__data__ = {}
@@ -296,7 +296,7 @@ def load(self):
             # Some sequence in the names file is not in the database
             raise KeyError("Sloane OEIS sequence and name files do not match."
                            " Try reinstalling, e.g. SloaneEncyclopedia.install(overwrite=True).")
-        except IOError:
+        except OSError:
             # The names database is not installed
             self.__loaded_names__ = False
 
@@ -323,7 +323,7 @@ def sequence_name(self, N):
         """
         self.load()
         if not self.__loaded_names__:
-            raise IOError("The Sloane OEIS names file is not installed."
+            raise OSError("The Sloane OEIS names file is not installed."
                           " Try reinstalling, e.g. SloaneEncyclopedia.install(overwrite=True).")
 
         if N not in self.__data__:  # sequence N does not exist
@@ -364,14 +364,14 @@ def copy_gz_file(gz_source, bz_destination):
         gz_input = gzip.open(gz_source, 'r')
         db_text = gz_input.read()
         gz_input.close()
-    except IOError as msg:
-        raise IOError("Error reading gzipped input file:\n%s" % msg)
+    except OSError as msg:
+        raise OSError("Error reading gzipped input file:\n%s" % msg)
 
     # Write the bzipped output
     try:
         os.makedirs(os.path.dirname(bz_destination), exist_ok=True)
         bz2_output = bz2.BZ2File(bz_destination, 'w')
         bz2_output.write(db_text)
         bz2_output.close()
-    except IOError as msg:
-        raise IOError("Error writing bzipped output file:\n%s" % msg)
+    except OSError as msg:
+        raise OSError("Error writing bzipped output file:\n%s" % msg)

src/sage/databases/stein_watkins.py

@@ -214,8 +214,8 @@ def __iter__(self):
         """
         try:
             file = bz2.open(self._file, 'rt', encoding="utf-8")
-        except IOError:
-            raise IOError("The Stein-Watkins data file %s must be installed." % self._file)
+        except OSError:
+            raise OSError("The Stein-Watkins data file %s must be installed." % self._file)
         C = None
         for L in file:
             if len(L) == 0:

src/sage/databases/symbolic_data.py

@@ -149,11 +149,11 @@ def _dom2ideal(node):
         try:
             name = self.__intpath + name + ".xml"
             open(name)
-        except IOError:
+        except OSError:
             try:
                 name = self.__genpath + name + ".xml"
                 open(name)
-            except IOError:
+            except OSError:
                 raise AttributeError("No ideal matching '%s' found in database." % orig_name)
 
         dom = parse(name)

src/sage/doctest/control.py

@@ -532,7 +532,7 @@ def __init__(self, options, args):
                 # string from DocTestDefaults
                 try:
                     self.logfile = open(options.logfile, 'a')
-                except IOError:
+                except OSError:
                     print("Unable to open logfile {!r}\nProceeding without logging.".format(options.logfile))
                     self.logfile = None
         else: