[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: patch para pbuilder



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

-- 
Antonio Terceiro <terceiro@debian.org>

Attachment: signature.asc
Description: Digital signature


Reply to: