Feb 18, 2024 · Jan 29, 2024
diff --git a/Doc/library/os.path.rst b/Doc/library/os.path.rst

 ..function::commonpath(paths)

   Return the longest common sub-path of each pathname in thesequence
   Return the longest common sub-path of each pathname in theiterable
   *paths*.  Raise:exc:`ValueError` if *paths* contain both absolute
   and relative pathnames, the *paths* are on the different drives or
   if *paths* is empty.  Unlike:func:`commonprefix`, this returns a
   ..versionadded::3.5

   ..versionchanged::3.6
      Acceptsa sequence of:term:`path-like objects <path-like object>`.
      Acceptsan iterable of:term:`path-like objects <path-like object>`.


 ..function::commonprefix(list)
diff --git a/Lib/posixpath.py b/Lib/posixpath.py
 defcommonpath(paths):
 """Given a sequence of path names, returns the longest common sub-path."""

 paths=tuple(map(os.fspath,paths))

 ifnotpaths:
 raiseValueError('commonpath() arg is an empty sequence')

 paths=tuple(map(os.fspath,paths))
 ifisinstance(paths[0],bytes):
 sep=b'/'
 curdir=b'.'
diff --git a/Lib/test/test_posixpath.py b/Lib/test/test_posixpath.py
 self.assertRaises(exc,posixpath.commonpath,
                              [os.fsencode(p)forpinpaths])

 self.assertRaises(TypeError,posixpath.commonpath,None)
 self.assertRaises(ValueError,posixpath.commonpath, [])
 self.assertRaises(ValueError,posixpath.commonpath,iter([]))
 check_error(ValueError, ['/usr','usr'])
 check_error(ValueError, ['usr','/usr'])

diff --git a/Misc/NEWS.d/next/Library/2024-01-29-13-46-41.gh-issue-114709.SQ998l.rst b/Misc/NEWS.d/next/Library/2024-01-29-13-46-41.gh-issue-114709.SQ998l.rst
 :func:`posixpath.commonpath()` now raises a:exc:`ValueError` exception when
 passed an empty iterable. Previously,:exc:`IndexError` was raised.

 :func:`posixpath.commonpath()` now raises a:exc:`TypeError` exception when
 passed ``None``. Previously,:exc:`ValueError` was raised.
Original file line number	Diff line number	Diff line change
Expand Up		@@ -79,7 +79,7 @@ the :mod:`glob` module.)

		..function::commonpath(paths)

		Return the longest common sub-path of each pathname in thesequence
		Return the longest common sub-path of each pathname in theiterable
		paths. Raise:exc:`ValueError` if paths contain both absolute
		and relative pathnames, the paths are on the different drives or
		if paths is empty. Unlike:func:`commonprefix`, this returns a
Expand All		@@ -90,7 +90,7 @@ the :mod:`glob` module.)
		..versionadded::3.5

		..versionchanged::3.6
		Acceptsa sequence of:term:`path-like objects <path-like object>`.
		Acceptsan iterable of:term:`path-like objects <path-like object>`.
serhiy-storchaka marked this conversation as resolved. Show resolvedHide resolved


		..function::commonprefix(list)
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -546,10 +546,11 @@ def relpath(path, start=None):
		defcommonpath(paths):
		"""Given a sequence of path names, returns the longest common sub-path."""

		paths=tuple(map(os.fspath,paths))

		ifnotpaths:
		raiseValueError('commonpath() arg is an empty sequence')

		paths=tuple(map(os.fspath,paths))
		ifisinstance(paths[0],bytes):
		sep=b'/'
		curdir=b'.'
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -703,7 +703,9 @@ def check_error(exc, paths):
		self.assertRaises(exc,posixpath.commonpath,
		[os.fsencode(p)forpinpaths])

		self.assertRaises(TypeError,posixpath.commonpath,None)
		self.assertRaises(ValueError,posixpath.commonpath, [])
		self.assertRaises(ValueError,posixpath.commonpath,iter([]))
		check_error(ValueError, ['/usr','usr'])
		check_error(ValueError, ['usr','/usr'])

Expand Down
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,5 @@
		:func:`posixpath.commonpath()` now raises a:exc:`ValueError` exception when
		passed an empty iterable. Previously,:exc:`IndexError` was raised.

		:func:`posixpath.commonpath()` now raises a:exc:`TypeError` exception when
		passed ``None``. Previously,:exc:`ValueError` was raised.