lnxpy / hey

💬 AI-Powered Pair Programming Friend!
https://pypi.org/project/hey-mindsdb/
MIT License
231 stars 12 forks source link

change installation to install with apt #18

Open darkkLUCIFER opened 3 months ago

darkkLUCIFER commented 3 months ago

Suggest

make the installation more easy with apt

@lnxpy

Description

i tried to install it with pip in my whole system but it dont allow to install without using an environment as simple and needs to use some commands

What I Did

i used the below commands to install sudo pip install -U hey-mindsdb --break-system-packages

and after using this i raise some errors about package that was installed by the python named typing-extensions and forced me to use the below command sudo pip install -U hey-mindsdb --break-system-packages --ignore-installed typing-extensions

amirsoroush commented 2 months ago

and after using this i raise some errors about package that...

What errors exactly? provide the traceback. Did you update the pip before installation? Which version of Python are you using?

Also, to install system-wide Python apps you may take a look at pipx instead.

lnxpy commented 2 months ago

@amirsoroush Good point, but not all the Linux users are familiar with Python and its package management. They are quite experienced at using their OS-specific package managers. Also–they don't need to install pipx to have Hey installed on their machines.

amirsoroush commented 2 months ago

@lnxpy Apparently by using those switches he is familiar with the package manager.

I didn't say they have to, pipx was just a suggestion. Installing something into the global interpreter is almost always a bad idea. People do that since they don't tend to deal with creating/activating/deactivating virtual environments. They just want to invoke their installed app on terminal. Here is where pipx comes into play. It handles the isolation of installed apps without needing the user to deal with venv directly.

lnxpy commented 2 months ago

What do you mean by "switches" here, @amirsoroush?!

Apparently by using those switches he is familiar with the package manager..

Well, the user might not be familiar with Python tools like pipx or even pip. Package managers like apt and pacman are installed by default on the distros. No matter the user's stack or expertise, he knows how to work with them for sure. Additionally, by using them, he goes straight to the point and installs the actual package other than messing around with installing the pipx and then going for the package installation. That's my point!

amirsoroush commented 2 months ago

Oops now I read the issue's title for the second time :) my bad, I didn't realize that he's "suggesting" a new way of installation.

I thought he's asking about their problem without providing any useful information. That's why I said to you how would on earth someone uses --break-system-packages, --ignore-installed typing-extensions switches without knowing how pip works.

lnxpy commented 2 months ago

@amirsoroush 😅🤙

sepehrrasooli commented 2 months ago

Is this issue still active @lnxpy ?

lnxpy commented 2 months ago

Yes, it is. We have some refactoring steps to do before this one. @sepehrrasooli