Modify mssql-cli bash wrapper to look for and prefer python3
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 1.4k
- Fork
- 191
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
On Ubuntu 20.04, the following install via pip does not work:
```bash
pip3 install --user mssqlcli
```
There are instructions to use a deb package for 18.04, but that's not an option. Related issues: #505, #482
In the bash wrapper to run this, I noticed python, and not python3, is assumed as the executable at https://github.com/dbcli/mssql-cli/blob/53a7a8a35193b1d2c8c8d85a2cdd7a4c3e3e7409/mssql-cli#L16
Some distributions (at least Debian/Ubuntu) facilitate the transition from python2 to python3 by having python point to python2 and python3 be a separate binary.
Instead of:
```bash
python -m mssqlcli.main "$@"
```
The following way of using command to check and match might help:
```bash
python_exe=$(command -v python3 || command -v python)
$python_exe -m mssqlcli.main "$@"
```
Then absolute path should be nicely resolved, e.g. /usr/bin/python3 for systems that have a binary found in the PATH.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dal wrapper bash collegato nell’issue, in particolare dalla riga 16, e verifica come seleziona l’eseguibile Python su Ubuntu 20.04. Verifica che il wrapper preferisca python3 quando disponibile e ricorra a python in alternativa, quindi conferma che invochi mssqlcli.main con gli argomenti originali.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python, shell
- Ambito
- cli
- Tipo di issue
- Bug
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 35/100