guyzmo/git-repo

config wizard does not work if ~/.config/git/config contains include statements

Open

#153 aperta il 7 mag 2017

Vedi su GitHub
 (5 commenti) (0 reazioni) (0 assegnatari)Python (86 fork)batch import
backlogbughelp wanted

Metriche repository

Star
 (837 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Steps to reproduce:

  • Make sure ~/.git does not exist
  • Create ~/.config/git/config` with a least two includes, like so:
[include]
path = /path/to/1
path = /path/to/2

Run git-repo config, do the oauth dance.

Nothing is stored.

PS: I think this is a bug in the code that reads and writes git config. If I were you, I'll just run git config --get foo.bar and git config --global foo.bar 42 instead of using a re-implementation of git config parsing in Python.

Guida contributor

Direzione di ricerca
Indaga il modulo di lettura/scrittura della configurazione nel repository (probabilmente un modulo Python). Il problema suggerisce di sostituire il parsing personalizzato con chiamate a `git config get` e `git config global`. Cerca dove le direttive include non sono gestite correttamente. Considera di implementare l'approccio suggerito o di correggere il parsing per gestire più include.
Tech stack
python
Dominio
clitooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Pythongit config basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60