[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

One thing to note is the copy_function parameter: if a path-like object is 
given as input should the copy_function receive the same object instead of a 
path string?


>>> copytree(path_like_src, path_like_dst, copy_function=foo)


Because of issue33695 in case of copytree(..., copy_function=...) we currently 
force to always pass path strings if copy_function is specified:
https://github.com/python/cpython/blob/ed1deb0719f0ac1b08a374e30ad26a701d4d51a2/Lib/shutil.py#L450

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Thrlwiti]

Change by Thrlwiti :


--
nosy: +THRlWiTi

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Mickaël Schoentgen]

Change by Mickaël Schoentgen :


--
versions: +Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Mickaël Schoentgen]

Mickaël Schoentgen  added the comment:

There is another place where the use of Path objects is not yet working: when 
move() is given a directory as argument. A patch is in review in issue32689.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Mickaël Schoentgen]

Change by Mickaël Schoentgen :


--
nosy: +Tiger-222

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Marco Rougeth]

Change by Marco Rougeth :


--
keywords: +patch
pull_requests: +5774
stage: test needed -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Jelle Zijlstra]

Changes by Jelle Zijlstra :


--
pull_requests: +1501

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Brett Cannon]

Changes by Brett Cannon :


--
versions: +Python 3.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Brett Cannon]

New submission from Brett Cannon:

If you look at the docs for shutil 
(https://docs.python.org/3/library/shutil.html), you will notice all over it 
says things like "path names given as strings". The question is whether that 
statement is true in the face of path-like objects? If it is true then the 
corresponding function in shutil should be updated to accept path-like objects. 
If it's false then the docs for the function should be updated.

This is a meta-issue to track working on the overall module (IOW separate PRs 
to fix things piecemeal is fine). Looks like the following functions need 
checking/updating:

- copyfile
- copymode
- copystat
- copy
- copy2
- copytree
- rmtree
- move
- disk_usage
- chown
- make_archive


[ unpack_archive is covered by https://github.com/python/cpython/pull/1367]

--
components: Library (Lib)
messages: 292788
nosy: brett.cannon
priority: normal
severity: normal
stage: test needed
status: open
title: Validate shutil supports path-like objects, update docs accordingly
type: behavior
versions: Python 3.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com