TypeError: invalid file: PosixPath('wall.log') #75

Closed
by denzilc opened 5 years ago · 4 comments
denzilc commented 5 years ago (Migrated from github.com)
Owner

I receive the following error

TypeError: invalid file: PosixPath('wall.log')

I have used the following command (git clone + run) to tun the script.

python -m deletefb.deletefb -E 'email@com' -P 'pwd' -U 'https://www.facebook.com/username'

What could this possibly refer to?

I receive the following error > TypeError: invalid file: PosixPath('wall.log') I have used the following command (git clone + run) to tun the script. > python -m deletefb.deletefb -E 'email@com' -P 'pwd' -U 'https://www.facebook.com/username' What could this possibly refer to?
weskerfoot commented 5 years ago (Migrated from github.com)
Owner

@denzilc I think it's your python version. Are you using <3.6?

It's tested with 3.6 or greater (see https://github.com/weskerfoot/DeleteFB#how-to-use-it ). If your system python is older than 3.6, then I would recommend either installing 3.6 (if you can) or use something like Pyenv to allow for newer versions of Python 3 alongside the system one.

https://github.com/pyenv/pyenv

I wasn't aware that this interface is not backwards compatible, so maybe I can take a look and see if it's possible to make backwards compatible, but TBH I would rather just support 3.6 or greater at this point, since it's already tied to Python 3 and doesn't support 2.

https://docs.python.org/3/whatsnew/3.6.html#pep-519-adding-a-file-system-path-protocol

@denzilc I think it's your python version. Are you using <3.6? It's tested with 3.6 or greater (see https://github.com/weskerfoot/DeleteFB#how-to-use-it ). If your system python is older than 3.6, then I would recommend either installing 3.6 (if you can) or use something like Pyenv to allow for newer versions of Python 3 alongside the system one. https://github.com/pyenv/pyenv I wasn't aware that this interface is not backwards compatible, so maybe I can take a look and see if it's possible to make backwards compatible, but TBH I would rather just support 3.6 or greater at this point, since it's already tied to Python 3 and doesn't support 2. https://docs.python.org/3/whatsnew/3.6.html#pep-519-adding-a-file-system-path-protocol
weskerfoot commented 5 years ago (Migrated from github.com)
Owner

Hmm actually it looks like it should be easy to fix. Just calling the str() constructor on it should work. I'll try to get a fix in soon.

Hmm actually it looks like it should be easy to fix. Just calling the `str()` constructor on it should work. I'll try to get a fix in soon.
weskerfoot commented 5 years ago (Migrated from github.com)
Owner

@denzilc Should be fixed in master now. Please give it another try :)

@denzilc Should be fixed in master now. Please give it another try :)
denzilc commented 5 years ago (Migrated from github.com)
Poster
Owner

Thanks, it works : )

Thanks, it works : )
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.