Re: patch para pbuilder

Em Ter, 2014-09-30 às 17:19 -0300, Antonio Terceiro escreveu:
> On Tue, Sep 30, 2014 at 12:46:47PM -0300, Herbert Parentes Fortes Neto wrote:
> > 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?
> 
> o único jeito são de arrumar isso é editar o arquivo alterado, corrigir
> a indentação (i.e. tirar os tabs) e gerar o patch de novo.
> 
> o ideal seria baixar o repositório git do pacote e manter um branch com
> todas as suas modificações; assim gerar o patch é trivial
> 
>   $ git diff master..mybranch > /tmp/changes.diff
> 

ok


obrigado