[issue42782] shutil.move creates a new directory even on failure

Senthil Kumaran added the comment: New changeset e808c9d5c78e5a7a9d804eced013a02c0c7df1a5 by Jelle Zijlstra in branch '3.9': [3.9] bpo-42782: fix broken shutil test (GH-31971) https://github.com/python/cpython/commit/e808c9d5c78e5a7a9d804eced013a02c0c7df1a5 --

[issue42782] shutil.move creates a new directory even on failure

Change by Jelle Zijlstra : -- nosy: +JelleZijlstra nosy_count: 3.0 -> 4.0 pull_requests: +30061 pull_request: https://github.com/python/cpython/pull/31971 ___ Python tracker

[issue42782] shutil.move creates a new directory even on failure

Change by Senthil Kumaran : -- stage: resolved -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue42782] shutil.move creates a new directory even on failure

Change by Senthil Kumaran : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Senthil Kumaran added the comment: New changeset bf566847f5a97e6ce391f8fb94185ee756cb94a2 by Senthil Kumaran in branch '3.9': [3.9] bpo-42782: Fail fast for permission errors in shutil.move() (GH-24001) (#24725)

[issue42782] shutil.move creates a new directory even on failure

miss-islington added the comment: New changeset 59e857650cf49a5e28cb82acc2641b1b53efeeeb by Miss Islington (bot) in branch '3.8': bpo-42782: Fail fast for permission errors in shutil.move() (GH-24001) https://github.com/python/cpython/commit/59e857650cf49a5e28cb82acc2641b1b53efeeeb

[issue42782] shutil.move creates a new directory even on failure

Change by Senthil Kumaran : -- pull_requests: +23498 pull_request: https://github.com/python/cpython/pull/24725 ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Change by miss-islington : -- pull_requests: +23497 pull_request: https://github.com/python/cpython/pull/24724 ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Change by miss-islington : -- pull_requests: +23489 pull_request: https://github.com/python/cpython/pull/24711 ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Change by miss-islington : -- nosy: +miss-islington nosy_count: 2.0 -> 3.0 pull_requests: +23487 pull_request: https://github.com/python/cpython/pull/24709 ___ Python tracker

[issue42782] shutil.move creates a new directory even on failure

Change by miss-islington : -- pull_requests: +23488 pull_request: https://github.com/python/cpython/pull/24710 ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Senthil Kumaran added the comment: New changeset 132131b404e06ee1a19b040a1f96cd1118abed0c by Winson Luk in branch 'master': bpo-42782: Fail fast for permission errors in shutil.move() (GH-24001) https://github.com/python/cpython/commit/132131b404e06ee1a19b040a1f96cd1118abed0c --

[issue42782] shutil.move creates a new directory even on failure

Change by Senthil Kumaran : -- assignee: -> orsenthil nosy: +orsenthil ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue42782] shutil.move creates a new directory even on failure

Change by Senthil Kumaran : -- versions: -Python 3.6, Python 3.7 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue42782] shutil.move creates a new directory even on failure

Winson Luk added the comment: To replicate: $ mkdir foo $ sudo chown root foo $ sudo touch foo/child $ python3 >>> import shutil >>> shutil.move('foo', 'bar') PermissionError $ ls foo child $ ls bar child If shutil.move() encountered a permission error and failed, bar should not have been

[issue42782] shutil.move creates a new directory even on failure

New submission from Winson Luk : shutil.move calls shutil.copytree(), then os.rmtree() (in that order). If the user does not have permission to delete the source directory, copytree succeeds but rmtree fails. The user sees an error (Permission Denied), but the destination directory is still

[issue42782] shutil.move creates a new directory even on failure

Change by Winson Luk : -- keywords: +patch pull_requests: +22843 stage: -> patch review pull_request: https://github.com/python/cpython/pull/24001 ___ Python tracker ___

[issue42782] shutil.move creates a new directory even on failure

Change by Winson Luk : -- components: Library (Lib) nosy: winsonluk priority: normal severity: normal status: open title: shutil.move creates a new directory even on failure type: behavior versions: Python 3.10, Python 3.6, Python 3.7, Python 3.8, Python 3.9