janw / letterboxd-rss

Convert your Letterboxd Watchlist to an RSS feed.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Change the pip package call name to something without dash

LilleAndersen opened this issue · comments

Having dash between the words confuses windows and docker when trying to call the script

Can you elaborate and show some examples of where that confusion occurs?

Error 1:

letterboxd-rss
Traceback (most recent call last):
File "", line 1, in
NameError: name 'letterboxd' is not defined

I get this error from windows running the command in python3.12

Seems to me that the dash is causing the issue.

Also got the error in chronos in a docker container. https://github.com/simse/chronos

Error 2:
Traceback (most recent call last):
File "/chronos/scripts/letterboxd/letterboxd.py", line 1, in
letterboxd-rss
NameError: name 'letterboxd' is not defined

I have gotten it to work in linux python virtual environment.

I don't know chronos but it looks like you are trying to run letterboxd-rss as a command in a python script. That will not work because that's not valid python syntax.

You may have to put

from letterboxd_rss.__main__ import main

main(
  # your arguments here
)

in the script but I cannot support this specific use-case.

The dash in the name is not an issue here, thus I'm closing this.