Title: fileinput inplace does not work with pathlib.Path
Type: behavior Stage: resolved
Components: Library (Lib) Versions: Python 3.7, Python 3.6
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: eric.smith Nosy List: eric.smith, zmwangx
Priority: normal Keywords:

Created on 2017-08-26 01:50 by zmwangx, last changed 2017-09-04 18:15 by eric.smith. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 3208 merged zmwangx, 2017-08-26 02:23
Messages (3)
msg300860 - (view) Author: Zhiming Wang (zmwangx) * Date: 2017-08-26 01:50

    import fileinput
    import pathlib
    with fileinput.input(files=(pathlib.Path('in.txt'),), inplace=True) as fp:
        for line in fp:
            print(line, end='')

which results in

    Traceback (most recent call last):
      File "./", line 6, in <module>
        for line in fp:
      File "/Users/zmwang/.pyenv/versions/3.6.1/lib/python3.6/", line 250, in __next__
        line = self._readline()
      File "/Users/zmwang/.pyenv/versions/3.6.1/lib/python3.6/", line 331, in _readline
        self._filename + (self._backup or ".bak"))
    TypeError: unsupported operand type(s) for +: 'PosixPath' and 'str'

A trivial fix is converting the specified filename to str when assigning to self._filename:

    -        self._filename = self._files[0]
    +        self._filename = str(self._files[0])
msg301231 - (view) Author: Eric V. Smith (eric.smith) * (Python committer) Date: 2017-09-04 17:37
New changeset 06de1aeff94e524bed21d188065c4cd1590fb046 by ericvsmith (Zhiming Wang) in branch 'master':
bpo-31281: Fix pathlib.Path incompatibility in fileinput (gh-3208)
msg301237 - (view) Author: Eric V. Smith (eric.smith) * (Python committer) Date: 2017-09-04 18:15
I did not backport this to 3.6, because it depends on other changes that themselves have not been backported.
Date User Action Args
2017-09-04 18:15:28eric.smithsetstatus: open -> closed
versions: - Python 3.4, Python 3.5
messages: + msg301237

assignee: eric.smith
resolution: fixed
stage: resolved
2017-09-04 17:37:26eric.smithsetnosy: + eric.smith
messages: + msg301231
2017-08-26 02:23:00zmwangxsetpull_requests: + pull_request3245
2017-08-26 01:50:52zmwangxcreate