Re: patch para pbuilder

Em Seg, 2014-09-29 às 18:16 -0300, Antonio Terceiro escreveu:
> On Mon, Sep 29, 2014 at 12:09:02PM -0300, Herbert Parentes Fortes Neto wrote:
> > Olá,
> > 
> > estou seguindo as orientações do how-can-i-help.
> > 
> > peguei um bug 'tagged as gift'   do pacote pbuilder-0.215
> > +nmu3(testing)[0].
> > 
> > Rodei o shellcheck no arquivo pbuilder-apt-config. Tirei os alertas mais
> > fáceis e fiz um patch com o '$ diff -u pbuilder-apt-config.orig
> > pbuilder-apt-config > pbuilder-apt-config.patch'. Arquivo anexo para
> > quem quiser verificar.
> > 
> > Tá tudo certo?
> 
> o código original usa identação de 4 espaços e o seu patch está
> inserindo tabs.
> Normalmente ler o patch você mesmo antes de enviar te ajuda a detectar
> essas coisas facilmente.

não consigo fazer o patch ter a mesma identação. tentei algumas opções
do diff mas não deu certo. A que pareceu melhor foi a '-t', mas dois
if-then-else ficam desalinhados. Não mexi na identação, não entendo o
erro. Edito o patch na mão?

> 
> > Eu teria que agora reportar o patch? Como faria isso?
> 
> você pode responder a mensagem original do bug e anexar o patch na sua
> resposta.

ok

> 
> Agora, note que o bug fala  de todo o código do pbuilder, não só do
> pbuilder-apt-config que 1 único script no meio de vários (claro que o
> patch pra 1 deles já é um bom começo).

sim, rodarei o shellcheck em todos eles. penso em reportar um de cada
vez.

> 
> > Ainda tem dois alertas. Que para mim demorariam de serem verificados.
> > 
> > In pbuilder-apt-config line 105:
> >               \#*|""|*)
> >               ^-- SC1001: This \# will be a regular '#' in this context.
> 
> substituir \# por '#' faz o shellcheck ficar quieto, mas na minha
> opinião essa warning não é muito útil.
> 
> > In pbuilder-apt-config line 457:
> > command="$1"
> > ^-- SC2034: command appears unused. Verify it or export it.
> 
> a variável command realmente não é usada dali para a frente e você
> poderia simplesmente deletar aquela linha.

ok

> 

obrigado