search

dsanson / termpdf.py

A graphical pdf and epub reader that works inside the kitty terminal
MIT License
496 stars 30 forks source link

setup.py lacks `roman` dependency #3

Closed teto closed 4 years ago

teto commented 4 years ago

lacks the roman dep I think ?

@ setup.py:12 @ setup(name='termpdf.py',
      author_email='dsanson@gmail.com',
      url='https://github.com/dsanson/termpdf.py',
      scripts=['termpdf.py'],
      requires=['PyMuPDF', 'pyperclip']
      requires=['PyMuPDF', 'pyperclip', 'roman']
     )
romain-fontugne commented 4 years ago

For me this was missing in the requirements.txt: roman pdfrw pagelabels

teto commented 4 years ago

would it be possible to move (or duplicate) the requirements to setup.py please ? it' the best way to distribute/install packages https://packaging.python.org/discussions/install-requires-vs-requirements/

dsanson commented 4 years ago

Sorry. Should have up to date requirements.txt and setup.py now.