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

Remove documentation recommendation to install pipx through brew or pip #1886

Open
andrewmcguinness opened this issue May 1, 2024 · 1 comment · May be fixed by #1889
Open

Remove documentation recommendation to install pipx through brew or pip #1886

andrewmcguinness opened this issue May 1, 2024 · 1 comment · May be fixed by #1889
Labels
documentation Improvements or additions to documentation

Comments

@andrewmcguinness
Copy link

Affected Page(s)

https://jrnl.sh/en/stable/installation/

What Could Be Better?

I'd like to challenge the advice "pipx should be installed through brew or pip, not apt". As far as I can see, the only sensible way to install jrnl on Debian is to install pipx through apt and jrnl with pipx. Installing pipx with pip on Debian is not possible (it just refuses). Using brew on Debian seems like a weird and dangerous thing to do.

pipx's job is to manage obtaining the dependencies of a package in a safe, isolated way. I would be surprised if any dependency problems emerged (and it worked perfectly for me on bookworm). At a guess the note in the docs dates from a time when having an old version of pipx itself could cause problems, but that is very stable now.

I'm not a heavy python developer and the tooling always confuses me, so I might be completely wrong, but for me the only choice for installing jrnl that I was happy with was the one that is specifically advised against.

Other Information

No response

@andrewmcguinness andrewmcguinness added 🆕 New! documentation Improvements or additions to documentation labels May 1, 2024
@micahellison micahellison changed the title Installing on Debian Remove documentation recommendation to install pipx through brew or pip May 7, 2024
@micahellison micahellison removed the 🆕 New! label May 7, 2024
@micahellison
Copy link
Member

Hi @andrewmcguinness, thanks for filing this and providing this analysis. You're absolutely right that this recommendation is from years ago when the apt pipx package wasn't exactly complete (#1087), but the situation has changed since then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
2 participants