shutil.make_archive(): restore select CPython <= 3.10.5 behavior

Restore following CPython <= 3.10.5 behavior of `shutil.make_archive()`:
do not create an empty archive if `root_dir` is not a directory, and, in
that case, raise `FileNotFoundError` or `NotADirectoryError` regardless
of `format` choice. Beyond the brought-back behavior, the function may
now also raise these exceptions in `dry_run` mode.

Author: 6t8k

Lib/shutil.py

@@ -1107,6 +1107,10 @@ def make_archive(base_name, format, root_dir=None, base_dir=None, verbose=0,
     supports_root_dir = getattr(func, 'supports_root_dir', False)
     save_cwd = None
     if root_dir is not None:
+        stmd = os.stat(root_dir).st_mode
+        if not stat.S_ISDIR(stmd):
+            raise NotADirectoryError(root_dir)
+
         if supports_root_dir:
             # Support path-like base_name here for backwards-compatibility.
             base_name = os.fspath(base_name)

Lib/test/test_shutil.py

@@ -170,6 +170,11 @@ def _maxdataOK():
     else:
         return True
 
+def _unlink_existing_file(path):
+    try:
+        os.unlink(path)
+    except FileNotFoundError:
+        pass
 
 class BaseTest:
 
@@ -182,6 +187,15 @@ def mkdtemp(self, prefix=None):
         self.addCleanup(os_helper.rmtree, d)
         return d
 
+    def mkstemp(self, prefix=None):
+        """Create a temporary file that will be cleaned up.
+
+        Returns the path of the file.
+        """
+        f = tempfile.mkstemp(prefix=prefix, dir=os.getcwd())
+        self.addCleanup(_unlink_existing_file, f[1])
+        return f
+
 
 class TestRmTree(BaseTest, unittest.TestCase):
 
@@ -1665,6 +1679,33 @@ def test_register_archive_format(self):
         formats = [name for name, params in get_archive_formats()]
         self.assertNotIn('xxx', formats)
 
+    def test_make_tarfile_rootdir_nodir(self):
+        # GH-99203
+        self.addCleanup(_unlink_existing_file, f'{TESTFN}.tar')
+
+        root_dir = self.mkstemp()[1]
+        self.assertRaises(NotADirectoryError, make_archive, TESTFN, 'tar', root_dir)
+        self.assertFalse(os.path.exists(f'{TESTFN}.tar'))
+
+        root_dir = self.mkstemp()[1]
+        os.unlink(root_dir)
+        self.assertRaises(FileNotFoundError, make_archive, TESTFN, 'tar', root_dir)
+        self.assertFalse(os.path.exists(f'{TESTFN}.tar'))
+
+    @support.requires_zlib()
+    def test_make_zipfile_rootdir_nodir(self):
+        # GH-99203
+        self.addCleanup(_unlink_existing_file, f'{TESTFN}.zip')
+
+        root_dir = self.mkstemp()[1]
+        self.assertRaises(NotADirectoryError, make_archive, TESTFN, 'zip', root_dir)
+        self.assertFalse(os.path.exists(f'{TESTFN}.zip'))
+
+        root_dir = self.mkstemp()[1]
+        os.unlink(root_dir)
+        self.assertRaises(FileNotFoundError, make_archive, TESTFN, 'zip', root_dir)
+        self.assertFalse(os.path.exists(f'{TESTFN}.zip'))
+
     ### shutil.unpack_archive
 
     def check_unpack_archive(self, format):

Misc/NEWS.d/next/Library/2022-11-26-22-05-22.gh-issue-99203.j0DUae.rst

@@ -0,0 +1,5 @@
+Restore following CPython <= 3.10.5 behavior of :func:`shutil.make_archive`:
+do not create an empty archive if ``root_dir`` is not a directory, and, in that
+case, raise :class:`FileNotFoundError` or :class:`NotADirectoryError`
+regardless of ``format`` choice. Beyond the brought-back behavior, the function
+may now also raise these exceptions in ``dry_run`` mode.