Re: patch para pbuilder

Antonio Terceiro <[email protected]>
Newsgroups gmane.linux.debian.devel.portuguese
Message-ID <[email protected]>
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 <[email protected]>
signature.asc (application/pgp-signature, 819 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCAAGBQJUKxA/AAoJEPwNsbvNRgve4xAQAKMWqcgyHcjwLyqvWxgVYRpT
DInz90vX3hZY8XLBbDZr9BQOfk4CHbfDTtDh/WBxS2vjKxcwaro9OH1k81lziaxb
ZmCBYlm2f86EVsgoUQKqf6JLj64jI0DJ205UMXIaNBVvjJNHYYpYhdJuVrO4KUJ2
OFY/Y0CvHu1wQ+zkLVuDR+IcfIFPJFlU30nztE/Oq2kSjS3WrGjXENforViCxm/M
XLPE7v5hnCD2X4nJQoB6fDSOozK5g6xYWVv3DysHO/FcgR9S5w8+Bk7JTYExJGse
RuNkSP+xrFZc1LrZ1pbgIZNbXCMsKMazklFmAaZ5IBt4r17f4l23RULycuRos8nk
2lQ7eZsiztqcBstsW6csCsxmJHoJIV6Hh2of4tM9X7RaHDZREE8adrzBp47z2cTB
Cn5XjIROkYQEwPhqCx51DMxbKNnftZews8lCiUv59a3KE8ACGHdzgY68ld4JY41l
Kbwx2SPUZDXEAxJyv2To9tG7X05VM+gQbCCa5f7w48qY0Jx93hEbE+OE9yiHRmNr
FwjH4EfcIZO/QTn2TVSS+e8jt7mChiMJNaM2+8XvO0uzMPnovRsSOHNYk5/7+urs
5hfprgXKbD6J5o1MMQKXfI/cPH7S2EIcPNki1yovmlKJVL3h9jLXszOs/PNFBNHi
STPHmfqC3g0v0rObx1/A
=uoZw
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.