Created on 2013-11-25 19:48 by pitrou, last changed 2014-05-13 08:55 by pitrou. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| pathlib_samefile.patch | vajrasky, 2013-12-01 05:00 | review | ||
| pathlib_samefile_v2.patch | vajrasky, 2013-12-01 14:31 | review | ||
| pathlib_samefile_v3.patch | vajrasky, 2013-12-05 03:07 | review | ||
| Messages (9) | |||
|---|---|---|---|
| msg204386 - (view) | Author: Antoine Pitrou (pitrou) * | Date: 2013-11-25 19:48 | |
It would probably be useful to provide samefile() on Path objects - basically doing the same thing as os.path.samefile(). |
|||
| msg204639 - (view) | Author: STINNER Victor (vstinner) * | Date: 2013-11-28 01:54 | |
I like the idea :-) |
|||
| msg204880 - (view) | Author: Vajrasky Kok (vajrasky) * | Date: 2013-12-01 05:00 | |
Here is the patch. |
|||
| msg204887 - (view) | Author: Arfrever Frehtes Taifersar Arahesis (Arfrever) * | Date: 2013-12-01 07:51 | |
About doc string in patch: 1. s/same/the same/ 2. "same" is usually used with "as", not "with": https://books.google.com/ngrams/graph?content=same+as%2Csame+with%2Csame+to&year_start=1800&year_end=2000&corpus=15&smoothing=3 |
|||
| msg204923 - (view) | Author: Vajrasky Kok (vajrasky) * | Date: 2013-12-01 14:31 | |
Updated grammar according to Arfrever's review. Thanks! Anyway, should samefile accepts only string? Or should it accept Path object as well? |
|||
| msg205263 - (view) | Author: Antoine Pitrou (pitrou) * | Date: 2013-12-04 23:30 | |
Thanks for the patch! Some comments: 1. It should path objects as well as str objects. 2. I don't think you have to call resolve() here. 3. you should probably test what happens when one of the files doesn't exist 4. you need to update the documentation too |
|||
| msg205270 - (view) | Author: Vajrasky Kok (vajrasky) * | Date: 2013-12-05 03:03 | |
Thanks for the review! Attached the patch addressing the request by Antoine. |
|||
| msg218413 - (view) | Author: Antoine Pitrou (pitrou) * | Date: 2014-05-13 08:40 | |
Sorry, it seems I have let this issue slip. I will update the patch and commit it soon! |
|||
| msg218415 - (view) | Author: Roundup Robot (python-dev) | Date: 2014-05-13 08:50 | |
New changeset 197ac5d79456 by Antoine Pitrou in branch 'default': Issue #19775: Add a samefile() method to pathlib Path objects. http://hg.python.org/cpython/rev/197ac5d79456 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2014-05-13 08:55:56 | pitrou | set | status: open -> closed resolution: fixed stage: resolved |
| 2014-05-13 08:50:39 | python-dev | set | nosy:
+ python-dev messages: + msg218415 |
| 2014-05-13 08:40:34 | pitrou | set | messages: + msg218413 |
| 2013-12-05 03:07:34 | vajrasky | set | files: + pathlib_samefile_v3.patch |
| 2013-12-05 03:07:28 | vajrasky | set | files: - pathlib_samefile_v3.patch |
| 2013-12-05 03:05:43 | vajrasky | set | files: + pathlib_samefile_v3.patch |
| 2013-12-05 03:05:23 | vajrasky | set | files: - pathlib_samefile_v3.patch |
| 2013-12-05 03:03:17 | vajrasky | set | files:
+ pathlib_samefile_v3.patch messages: + msg205270 |
| 2013-12-04 23:30:27 | pitrou | set | messages: + msg205263 |
| 2013-12-01 14:31:55 | vajrasky | set | files:
+ pathlib_samefile_v2.patch messages: + msg204923 |
| 2013-12-01 07:51:11 | Arfrever | set | messages: + msg204887 |
| 2013-12-01 05:00:55 | vajrasky | set | files:
+ pathlib_samefile.patch nosy:
+ vajrasky keywords: + patch |
| 2013-11-28 01:54:05 | vstinner | set | nosy:
+ gvanrossum, vstinner messages: + msg204639 |
| 2013-11-26 02:29:10 | Arfrever | set | nosy:
+ Arfrever |
| 2013-11-25 19:48:21 | pitrou | create | |