fossasia / pslab-python

Python Library for PSLab Desktop: https://pslab.io
GNU General Public License v3.0
1.63k stars 226 forks source link

Revert "Add documentation to Fix permission for make file #88 (#89)" #120

Closed orangecms closed 4 years ago

orangecms commented 4 years ago

This reverts commit 24c57a0b4190af07324ac4780e7d3cbc68cf9a38.

Arbitrarily running sudo messes up file permissions. In this case, a user would no longer be able to access the generated documentation files. Prefer installing tools through the system package manager instead.

bessman commented 4 years ago

I don't necessarily agree that the package manager is always preferable to pypi, but I agree that the documentation should not suggest running every command with sudo.

In my opinion, the best way to ensure up-to-date packages while avoiding permission problems and package conflicts is to use virtual environments, in which case sudo should not be used.

orangecms commented 4 years ago

Yea the downside of tools in the home directory is that people need to know that they need to adjust their PATH. With, say, Python, Ruby, and a few other tools, it gets very messy and hard to maintain when upgrading the runtime. That's what system package managers solve. :)

orangecms commented 4 years ago

People can act according to their preferences of course. :+1: