Re: patch para pbuilder

Antonio Terceiro <[email protected]> Mon, 29 Sep 2014 18:16:54 -0300

Newsgroups	gmane.linux.debian.devel.portuguese
Message-ID	<[email protected]>

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.

> 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.

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).

> 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.

-- 
Antonio Terceiro <[email protected]>

signature.asc (application/pgp-signature, 819 B)