guyzmo/git-repo

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

Open

#153 aberto em 7 de mai. de 2017

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Python (86 forks)batch import
backlogbughelp wanted

Métricas do repositório

Stars
 (837 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Investigue o módulo de leitura/escrita de configuração no repositório (provavelmente um módulo Python). O problema sugere substituir a análise personalizada por chamadas a `git config get` e `git config global`. Procure onde as diretivas include não são tratadas corretamente. Considere implementar a abordagem sugerida ou corrigir a análise para lidar com múltiplos includes.
Pilha de tecnologia
python
Domain
clitooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythongit config basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60