gitui-org / gitui-org/gitui

installing gitui releases from source results in a "nightly" version string by default

Aperta
#2,254 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
Rust
Stelle
22.5k
Fork
773
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Running `cargo install gitui` as documented here:
https://github.com/extrawurst/gitui?tab=readme-ov-file#cargo-install

Results in `gitui --version` returning:
`gitui nightly 2024-06-02 ()` (replace `2024-06-02` with the current date)

I'm also having trouble making this work sensibly for the Fedora Linux package for gitui.

I can easily set `GITUI_RELEASE=1`, but then the `get_git_hash()` function from build.rs will fail hard (with an `std::io::Error: No such file or directory`) if `git` is not installed in the build environment. And *if* `git` is available, it will either return `""` (in the case where the git command fails with no output on stdout) or random garbage if there is an unrelated git repository in any of the parent directories.

Would you accept a PR that adapted build.rs to work better in more cases, for example, when `git` is not available?

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia riproducendo il flusso documentato `cargo install gitui` e verificando il `gitui --version` risultante. Poi esamina `build.rs`, in particolare `get_git_hash()` e il percorso `GITUI_RELEASE=1`, per capire il comportamento con e senza `git` disponibile. Il lavoro è completato quando l'installazione della release produce una stringa di versione sensata senza fallire né usare un repository padre non correlato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
git, rust
Ambito
build-system, cli
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
30/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.