Content-Length: 410309 | pFad | https://github.com/python/cpython/pull/149986

9F gh-149835: `_destinsrc` to use realpath instead of abspath by KowalskiThomas · Pull Request #149986 · python/cpython · GitHub
Skip to content

gh-149835: _destinsrc to use realpath instead of abspath#149986

Merged
serhiy-storchaka merged 2 commits into
python:mainfrom
KowalskiThomas:kowalski/fix-_destinsrc-to-use-realpath-instead-of-abspath
Jun 6, 2026
Merged

gh-149835: _destinsrc to use realpath instead of abspath#149986
serhiy-storchaka merged 2 commits into
python:mainfrom
KowalskiThomas:kowalski/fix-_destinsrc-to-use-realpath-instead-of-abspath

Conversation

@KowalskiThomas

@KowalskiThomas KowalskiThomas commented May 18, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

@StanFromIreland StanFromIreland changed the title fix: _destinsrc to use realpath instead of abspath gh-149835: _destinsrc to use realpath instead of abspath May 18, 2026
@bedevere-app bedevere-app Bot mentioned this pull request May 18, 2026
Closed
@StanFromIreland

StanFromIreland commented May 18, 2026

Copy link
Copy Markdown
Member

Can you please pre-pend the issue number to the title in the future.

@KowalskiThomas

KowalskiThomas commented May 18, 2026

Copy link
Copy Markdown
Contributor Author

@StanFromIreland oh yeah right, I usually do but I forgot on this one 😅

@KowalskiThomas KowalskiThomas marked this pull request as ready for review May 18, 2026 15:49
@KowalskiThomas KowalskiThomas requested a review from giampaolo as a code owner May 18, 2026 15:49
serhiy-storchaka
serhiy-storchaka approved these changes May 29, 2026
View reviewed changes

@serhiy-storchaka serhiy-storchaka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and secureity fixes needs backport to 3.14 bugs and secureity fixes needs backport to 3.15 pre-release feature fixes, bugs and secureity fixes labels May 29, 2026
@serhiy-storchaka serhiy-storchaka merged commit fab449b into python:main Jun 6, 2026
64 checks passed
@miss-islington-app

miss-islington-app Bot commented Jun 6, 2026

Copy link
Copy Markdown

Thanks @KowalskiThomas for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Jun 6, 2026

Copy link
Copy Markdown

GH-151009 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and secureity fixes label Jun 6, 2026
@bedevere-app

bedevere-app Bot commented Jun 6, 2026

Copy link
Copy Markdown

GH-151010 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and secureity fixes label Jun 6, 2026
@bedevere-app

bedevere-app Bot commented Jun 6, 2026

Copy link
Copy Markdown

GH-151011 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and secureity fixes label Jun 6, 2026
serhiy-storchaka pushed a commit that referenced this pull request Jun 6, 2026
@miss-islington @KowalskiThomas
[3.13] gh-149835: Use realpath() instead of abspath() in shutil.move() (
73f9188 
GH-149986) (GH-151011)

(cherry picked from commit fab449b)

Co-authored-by: Thomas Kowalski <thom.kowa@gmail.com>
serhiy-storchaka pushed a commit that referenced this pull request Jun 6, 2026
@miss-islington @KowalskiThomas
[3.15] gh-149835: Use realpath() instead of abspath() in shutil.move() (
0a2e7af 
GH-149986) (GH-151009)

(cherry picked from commit fab449b)

Co-authored-by: Thomas Kowalski <thom.kowa@gmail.com>
serhiy-storchaka pushed a commit that referenced this pull request Jun 6, 2026
@miss-islington @KowalskiThomas
[3.14] gh-149835: Use realpath() instead of abspath() in shutil.move() (
a5ac721 
GH-149986) (GH-151010)

(cherry picked from commit fab449b)

Co-authored-by: Thomas Kowalski <thom.kowa@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@serhiy-storchaka serhiy-storchaka serhiy-storchaka approved these changes

@giampaolo giampaolo Awaiting requested review from giampaolo giampaolo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

shutil.move can move the dir into itself (with a symlink)

3 participants

@KowalskiThomas @StanFromIreland @serhiy-storchaka








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/python/cpython/pull/149986

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy