Description
Python has begun to use pathlib.Path instead of os.path since Python 3.4 .
Since we no longer need to go back to Python 2.7 in PySide6, I propose to switch PySide6 also to use the pathlib.
After changing very much code, it became obvious that we always need to revert the effect of Path when a PySide function deals with it. That was enough motivation to work on supporting the __fspath__ attribute that will make it easy to implement Path acceptance everywhere.
Attachments
Issue Links
- relates to
-
PYSIDE-1110
Embrace Python types and modules in PySide
-
- Closed
-
- resulted in
-
PYSIDE-1736 QQmlApplicationEngine::load() no longer works with string argument
-
- Closed
-
-
-
- Closed
-
| For Gerrit Dashboard: PYSIDE-1499 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 376443,6 | Inactive: Add explicit constructors for PyPathlike modifications (not really) | dev | pyside/pyside-setup | Status: NEW | 0 | 0 |
| 333352,8 | Switch from os.path to pathlib.Path, all source changes but tests | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 334707,35 | build support for pathlib.Path, baseline | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 334847,2 | PySide6/__init__.py.in: Fix type error | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 335606,3 | sbkconverter: fix a potential refcounting crash | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 335607,3 | cppgenerator: simplify code further using AutoDecref | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 335750,3 | sbkconverter: fix a potential refcounting crash | tqtc/lts-5.15 | pyside/tqtc-pyside-setup | Status: MERGED | +2 | 0 |
| 342002,6 | Port tests to pathlib | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 342087,1 | Port tests to pathlib | 6.0 | pyside/pyside-setup | Status: ABANDONED | -1 | 0 |
| 353024,2 | cppgenerator: remove a left-over "this" to get rid of a warning | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 353103,2 | cppgenerator: remove a left-over "this" to get rid of a warning | 6.1 | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 355202,2 | PySide6: Fix some shiboken warnings related to Pathlib support | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 358423,3 | pathlib: support all cases which are found in the examples | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 358444,1 | pathlib: support all cases which are found in the examples | 6.1 | pyside/pyside-setup | Status: ABANDONED | +2 | 0 |
| 359435,3 | pathlib: support QQmlApplicationEngine load | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 359655,4 | pathlib: fix empty constructors in conversion | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 374073,5 | Workaround to make shibokenwizard work | tqtc/dev | pyside/tqtc-pyside-setup | Status: MERGED | +2 | 0 |
| 374144,2 | Workaround to make shibokenwizard work | tqtc/6.2 | pyside/tqtc-pyside-setup | Status: MERGED | +2 | 0 |
| 376607,6 | Fix QIcon.addPixmap() to accept a PyPathLike argument | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 378755,2 | Fix QIcon.addPixmap() to accept a PyPathLike argument | 6.2 | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 386676,3 | PySide6: Fix QQmlApplicationEngine.load(str) to not construct a QUrl | dev | pyside/pyside-setup | Status: MERGED | +2 | 0 |
| 386719,2 | PySide6: Fix QQmlApplicationEngine.load(str) to not construct a QUrl | 6.2 | pyside/pyside-setup | Status: MERGED | +2 | 0 |