Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new package stirling-pdf #1191

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

knyipab
Copy link
Contributor

@knyipab knyipab commented Sep 11, 2024

  • follow the practice of AUR to keep runtime files at /var/lib/stirling-pdf
  • It is probably better to have a package ocrmypdf (provided that debian also package it) instead of installing through pip, many in TERMUX_PKG_DEPENDS are actually for ocrmypdf. I only tested it on my contaminated smartphone python environment.

@knyipab knyipab force-pushed the dev/stirling-pdf branch 5 times, most recently from bb64cd2 to 2dd0624 Compare September 12, 2024 01:33
@knyipab knyipab marked this pull request as ready for review September 12, 2024 10:44
@licy183
Copy link
Collaborator

licy183 commented Sep 13, 2024

Yes, I think TUR should package ocrmypdf rather than installing it in postinst.

@knyipab
Copy link
Contributor Author

knyipab commented Sep 13, 2024

I'm actually not so sure the right way to ocrmypdf provided that it has quite a number of python python packages (that may need postinst to run pip install I suppose). Otherwise, will need to package them to TUR.

Actually what is the best practice of packaging python package? Would the termux pkg version mess up with pip?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants