Skip to content

gh-84538: add strict argument to pathlib.PurePath.relative_to #19813

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
miss-islington merged 17 commits into from
Oct 28, 2022
Merged

gh-84538: add strict argument to pathlib.PurePath.relative_to #19813

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ec64368
bpo-40358: add strict param to pathlib.PurePath.relative_to
domragusa Apr 30, 2020
6d721a0
resolve conflicts with GH-19611
domragusa May 28, 2020
09422e7
Merge branch 'python:main' into relative_to_strict_param
domragusa May 15, 2021
d9c07c2
Update Doc/library/pathlib.rst
domragusa May 15, 2021
029c3d5
fix wording in documentation as requested
domragusa May 17, 2021
01e789b
fix typo
domragusa May 17, 2021
39b24f3
change strict argument to walk_up in pathlib.PurePath.relative_to
domragusa May 29, 2021
ad9c0dd
Merge branch 'main' into relative_to_strict_param
domragusa Oct 14, 2022
8401974
make the documentation clearer
domragusa Oct 14, 2022
e255f99
Apply suggestions from code review
domragusa Oct 22, 2022
b781356
Update Lib/pathlib.py
domragusa Oct 22, 2022
7974ee9
fix doc warning
domragusa Oct 22, 2022
8a4edb5
improve variable names
domragusa Oct 22, 2022
e6cf222
remove trailing spaces
domragusa Oct 22, 2022
b8b218c
Merge branch 'main' into relative_to_strict_param
domragusa Oct 22, 2022
a4c77c2
Update Doc/library/pathlib.rst
brettcannon Oct 28, 2022
97a5e22
Merge branch 'main' into relative_to_strict_param
brettcannon Oct 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update Doc/library/pathlib.rst 
Co-authored-by: Terry Jan Reedy <tjreedy@udel.edu>
  • Loading branch information
@domragusa @terryjreedy
domragusa and terryjreedy authored May 15, 2021
commit d9c07c2f0f73dcf5c7dc863f9d4eeaeb3cfe64c4
4 changes: 2 additions & 2 deletions Doc/library/pathlib.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -565,8 +565,8 @@ Pure paths provide the following methods and properties:

If the path doesn't start with *other* and *strict* is ``True``, :exc:`ValueError` is raised. If *strict* is ``False`` and one path is relative and the other is absolute or if they reference different drives :exc:`ValueError` is raised.
Copy link
Copy Markdown
Contributor

@barneygale barneygale May 17, 2021

Choose a reason for hiding this comment

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

I think the key difference should be spelled out in terms of whether adding .. entries is allowed. How about something like this?

In strict mode (the default), the path must start with *other*. In non-strict 
mode, ``..`` entries may be added to form the relative path. In all other 
cases, such as the paths referencing different drives, :exe:`ValueError` is 
raised.

.. warning::
    Non-strict mode assumes that no symlinks are present in the path; you 
    should call :meth:`~Path.resolve` first to ensure this.


NOTE: This function is part of :class:`PurePath` and works with strings. It does not check or access the underlying file structure.

.. versionadded:: 3.11
The *strict* argument (pre-3.11 behavior is strict).
.. versionadded:: 3.10
The *strict* argument (pre-3.10 behavior is strict).

Expand Down