#14862: Add missing names to os.__all__

diff --git a/Lib/os.py b/Lib/os.py
index af4990fb8bfae18de1117a4489fb16de2f380f85..a1a35cd8e366c060083a3a42d393359adc2693cb 100644 (file)
--- a/Lib/os.py
+++ b/Lib/os.py
@@ -30,8 +30,9 @@ _names = sys.builtin_module_names
 
 # Note:  more names are added to __all__ later.
 __all__ = ["altsep", "curdir", "pardir", "sep", "pathsep", "linesep",
-           "defpath", "name", "path", "devnull",
-           "SEEK_SET", "SEEK_CUR", "SEEK_END"]
+           "defpath", "name", "path", "devnull", "SEEK_SET", "SEEK_CUR",
+           "SEEK_END", "fsencode", "fsdecode", "get_exec_path", "fdopen",
+           "popen", "extsep"]
 
 def _exists(name):
     return name in globals()
@@ -50,6 +51,7 @@ if 'posix' in _names:
     from posix import *
     try:
         from posix import _exit
+        __all__.append('_exit')
     except ImportError:
         pass
     import posixpath as path
@@ -64,6 +66,7 @@ elif 'nt' in _names:
     from nt import *
     try:
         from nt import _exit
+        __all__.append('_exit')
     except ImportError:
         pass
     import ntpath as path
@@ -78,6 +81,7 @@ elif 'os2' in _names:
     from os2 import *
     try:
         from os2 import _exit
+        __all__.append('_exit')
     except ImportError:
         pass
     if sys.version.find('EMX GCC') == -1:
@@ -96,6 +100,7 @@ elif 'ce' in _names:
     from ce import *
     try:
         from ce import _exit
+        __all__.append('_exit')
     except ImportError:
         pass
     # We can use the standard Windows path.
@@ -700,6 +705,8 @@ if _exists("fork") and not _exists("spawnv") and _exists("execv"):
     P_WAIT = 0
     P_NOWAIT = P_NOWAITO = 1
 
+    __all__.extend(["P_WAIT", "P_NOWAIT", "P_NOWAITO"])
+
     # XXX Should we support P_DETACH?  I suppose it could fork()**2
     # and close the std I/O streams.  Also, P_OVERLAY is the same
     # as execv*()?

diff --git a/Misc/NEWS b/Misc/NEWS
index c8301bd65c12ab55fb2aa109f1a45795b0d009e8..f0e1a55834718952a6744bf2c37d49ebfd26a56d 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -42,6 +42,8 @@ Core and Builtins
 Library
 -------
 
+- Issue #14862: Add missing names to os.__all__
+
 - Issue #14875: Use float('inf') instead of float('1e66666') in the json module.
 
 - Issue #13585: Added contextlib.ExitStack