rcaloras / bashhub-client

:cloud: Bash history in the cloud. Indexed and searchable.
https://bashhub.com
Apache License 2.0
1.25k stars 80 forks source link

Setup doesn't work for host with Python 3.12.3 #138

Open rafauke opened 3 months ago

rafauke commented 3 months ago

Hello, thanks for the package :)

Unfortunately, there are some issues when installing:

$ curl -OL https://bashhub.com/setup && $SHELL setup
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  8717  100  8717    0     0  16059      0 --:--:-- --:--:-- --:--:-- 16083

Sorry can't seem to find a version of python 3.6-3.11 or 2.7.9+ installed

Version of python reported by os:

$ python --version
Python 3.12.3

Let me know if I can help with anything.

rcaloras commented 3 months ago

@rafauke thanks for opening. You can test fixing locally via one character change in the setup script: https://github.com/rcaloras/bashhub-client/blob/c408de68365557d792557e4b8f510ed29c75f910/install-bashhub.sh#L41

Try changing the 12 to 13 and see if everything installs fine. If alls good would welcome a PR!

bogorad commented 1 month ago

alas. fedora-40. changed 12 to 13:

 $SHELL setup
Using Python path /usr/bin/python3
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 10.7M  100 10.7M    0     0  5810k      0  0:00:01  0:00:01 --:--:-- 6747k
Pulling down bashhub-client from 2.4.1 branch
Pulling down a few dependencies...(this may take a moment)
Traceback (most recent call last):
  File "/home/chuck/.bashhub/bashhub-client-2.4.1/../env/bin/pip", line 5, in <module>
    from pip._internal.cli.main import main
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/cli/main.py", line 9, in <module>
    from pip._internal.cli.autocompletion import autocomplete
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/cli/autocompletion.py", line 10, in <module>
    from pip._internal.cli.main_parser import create_main_parser
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/cli/main_parser.py", line 8, in <module>
    from pip._internal.cli import cmdoptions
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/cli/cmdoptions.py", line 23, in <module>
    from pip._internal.cli.parser import ConfigOptionParser
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/cli/parser.py", line 12, in <module>
    from pip._internal.configuration import Configuration, ConfigurationError
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/configuration.py", line 20, in <module>
    from pip._internal.exceptions import (
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_internal/exceptions.py", line 7, in <module>
    from pip._vendor.pkg_resources import Distribution
  File "/home/chuck/.bashhub/env/lib/python3.12/site-packages/pip/_vendor/pkg_resources/__init__.py", line 2164, in <module>
    register_finder(pkgutil.ImpImporter, find_on_path)
                    ^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pkgutil' has no attribute 'ImpImporter'. Did you mean: 'zipimporter'?
rsmoorthy commented 1 month ago

+1. I am also getting the above python error on ubuntu 24.04

rsmoorthy commented 1 month ago

I got this working, based on SO post that says you need to upgrade pip. (The current setup downloads a virtualenv pre-packaged, that may not work best for python 3.12. Either the virtualenv pre-packaged download should be made to work for python 3.12 or it may be best to create the virtual env from scratch).

After solving that problem, I ran into another one. The requests package 2.23 version is mentioned in setup.py, while that version does not work well with python 3.12, and I tried with the latest and it works fine. I have fixed this too in the following fix.

I have raised a PR #139