[issue41082] Error handling and documentation of Path.home()

2021-09-08 Thread Łukasz Langa

Łukasz Langa  added the comment:

Behavior is fixed in Python 3.10+. For Python 3.9 the docs are updated to list 
possible exceptions. Since this was reported with 3.9 in mind, I'm marking this 
one as wontfix.

Thanks for picking this up, Andrei! ✨  ✨

--
resolution:  -> wont fix
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-09-08 Thread Łukasz Langa

Łukasz Langa  added the comment:


New changeset dc2e11ed5a5a8083db1d8b5e2396c9238999568c by andrei kulakov in 
branch '3.9':
[3.9] bpo-41082: Add note on errors that may be raised by home() and 
expanduser() (GH-28186)
https://github.com/python/cpython/commit/dc2e11ed5a5a8083db1d8b5e2396c9238999568c


--

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-09-08 Thread Łukasz Langa

Łukasz Langa  added the comment:

Using `os.path.expanduser` is a functional change so it wasn't backported at 
the time to 3.9 and sure shouldn't be backported now. Going with a doc update 
is the right approach.

--
nosy: +lukasz.langa

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-09-06 Thread Andrei Kulakov


Andrei Kulakov  added the comment:

I looked a bit more into this, it's been fixed in 3.10, in this PR: 
https://github.com/python/cpython/pull/18841

It wasn't backported to 3.9. I'm not quite sure if it's best to backport to 3.9 
or fix via docs for 3.9. Just in case I've put up a small PR for such a docfix.

--

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-09-06 Thread Andrei Kulakov


Change by Andrei Kulakov :


--
versions: +Python 3.9

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-09-06 Thread Andrei Kulakov


Change by Andrei Kulakov :


--
keywords: +patch
pull_requests: +26614
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/28186

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2021-08-14 Thread Andrei Kulakov


Andrei Kulakov  added the comment:

In current dev version (3.11), _PosixFlavor doesn't have gethomedir(), and 
instead os.path.expanduser() is used via _NormalAccessor(). So at least for the 
current dev version, the docs are accurate.

--
nosy: +andrei.avk

___
Python tracker 

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



[issue41082] Error handling and documentation of Path.home()

2020-06-22 Thread Tim Hoffmann


New submission from Tim Hoffmann :

Path.home() may fail un
(https://github.com/matplotlib/matplotlib/issues/17707#issuecomment-647180252).

1. I think the raised key error is too low-level, and it should be something 
else; what exactly t.b.d.
2. The documentation 
(https://docs.python.org/3/library/pathlib.html#pathlib.Path.home) should 
specify what happens in case of failure.
3. The documentation links to 
https://docs.python.org/3/library/os.path.html#os.path.expanduser, but that's 
not correct. _PosixFlavor.gethomedir() implements it's own user lookup, which 
is slightly different from os.path.expanduser()

--
components: Library (Lib)
messages: 372116
nosy: timhoffm
priority: normal
severity: normal
status: open
title: Error handling and documentation of Path.home()
type: behavior

___
Python tracker 

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