➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2018-10-30 12:02 by nubirstein, last changed 2022-04-11 14:59 by admin. This issue is now closed.
| Messages (6) | |||
|---|---|---|---|
| msg328904 - (view) | Author: (nubirstein) | Date: 2018-10-30 12:02 | |
Although seekable() method is implemented in the IOBase (which means it is callable in BytesIO and StringIO), SpooledTemporaryFile class from lib/tempfile.py still does not implement it. |
|||
| msg328905 - (view) | Author: Karthikeyan Singaravelan (xtreak) *  |
Date: 2018-10-30 12:10 | |
Thanks for the report. Search gives me a related issue with PR to implement IOBase for SpooledTemporaryFile : issue26175 |
|||
| msg328910 - (view) | Author: (nubirstein) | Date: 2018-10-30 12:40 | |
Should I have been added my request there? Anyway I do suffer from lack of 'seekable()' implementation there. I'll go there and make a request |
|||
| msg328912 - (view) | Author: Karthikeyan Singaravelan (xtreak) * |
Date: 2018-10-30 12:46 | |
The related issue already implements seekable and other methods for IOBase. As I can see from the open PR there are PR comments with respect to tests and docs that need to be acted upon as per reviewer's comments. With respect to this issue I propose closing this as a duplicate with issue26175 as superseder to continue your discussion there where it might be useful. Thanks! |
|||
| msg328913 - (view) | Author: (nubirstein) | Date: 2018-10-30 12:49 | |
According to Karthikeyan request I'am moving this discussion to the right place, i.e. https://bugs.python.org/issue26175 I hope to get some help there. Thx Karthikeyan for such a short response time. |
|||
| msg328914 - (view) | Author: Karthikeyan Singaravelan (xtreak) * |
Date: 2018-10-30 12:54 | |
@nubirstein You're welcome :) As noted in https://bugs.python.org/issue26175#msg328911 the PR has the implementation of seekable at https://github.com/python/cpython/pull/3249/files#diff-34b2145d7fe189e893ec7934afe9829cR761 . You can help with the PR reviews if any and hopefully it gets merged. Adding 26175 as superseder. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:07 | admin | set | github: 79293 |
| 2018-10-30 12:54:02 | xtreak | set | superseder: Fully implement IOBase abstract on SpooledTemporaryFile messages: + msg328914 |
| 2018-10-30 12:49:48 | nubirstein | set | status: open -> closed resolution: duplicate messages: + msg328913 stage: resolved |
| 2018-10-30 12:46:27 | xtreak | set | messages: + msg328912 |
| 2018-10-30 12:40:27 | nubirstein | set | messages: + msg328910 |
| 2018-10-30 12:10:37 | xtreak | set | nosy:
+ xtreak messages: + msg328905 |
| 2018-10-30 12:02:15 | nubirstein | create | |