Re: RFS: bogofilter -- fast Bayesian spam filter [QA-RC]
Oi Herbert, tudo bem?
Inicialmente, desculpe mas ainda não pude fazer upload. O pacote é
muito complicado e ainda tem problemas. (e eu sei que você adora esse
tipo de empacotamento mais complexo, parabéns pelo trabalho!)
Vamos lá.
Em 7 de maio de 2016 12:02, Herbert Fortes <hpfn@ig.com.br> escreveu:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA512
>
> Olá,
>
>>
>> 1. d/changelog:
>>
>> - Sobre o que ocorreu com o debian/*.NEWS, eu também só entendi ao
>> analisar o debdiff. Uma sugestão de texto (modifique à vontade):
>>
>> - Fixed a spelling error (and and).
>> - Removed all leading '*' from text.
>> - Reorganized the text.
>>
>> EM TEMPO: após analisar o arquivo, vi que você alterou um texto que
>> foi escrito e assinado por outra pessoa. Não me parece uma boa ideia,
>> pois vai dar a impressão que tal pessoa fez o trabalho da forma como
>> você reescreveu. Penso que você deveria, apenas, corrigir o 'and and'.
>> Ainda, com o mcedit, consigo ver que ficaram alguns espaços em branco
>> e tabs perdidos no texto. O que você acha?
>
> Não alterei o texto. Apenas a formatação. São dois .NEWS.
Essa foi uma das coisas que eu quis dizer. Alterar a formatação e
manter o nome de outra pessoa como autor não é uma boa ideia. Eu já
tive problemas na vida, não com pacotes, mas com textos, até em
provas, quando fizeram alterações, mantendo o meu nome, sem a minha
autorização, e várias coisas certas ficaram erradas ou de um jeito que
eu não concordava. Em um livro, por exemplo, em shell scripts, um
editor trocou todas as crases por apóstrofos porque ficava "mais
bonito". Já viu no que deu, não é? Então, continuo não achando uma boa
ideia alterar o texto de alguém, mantendo o nome dele, mesmo que seja
só o estilo. Será que ele gostaria disso? Uma saída para alterar
seria, depois da assinatura, colocar algo como um "reformatted by..."
ou algo similar. Ou mesmo manter o texto original.
> cat -vet debian/bogofilter-common.NEWS
> bogofilter (1.2.0-2) unstable; urgency=low$
> $
> You may now use update-alternatives to choose another than the default db$
> backend. For instance, to switch the backend to sqlite, install$
> bogofilter-sqlite and run as root:$
> update-alternatives --set bogofilter /usr/bin/bogofilter-sqlite$
> Bear in mind that, to switch to a different backend you have to dump your$
> word lists using bogoutil of the old backend, and load your word$
> lists again using bogoutil of the new backend.$
> $
> -- Serafeim Zanikolas <serzan@hellug.gr> Wed, 06 May 2009 22:55:15 +0200$
> $
>
> Acho que você se refere ao 'debian/bogofilter-bdb.NEWS'. Troquei a
> formatação, e estranhamente, o registros de versões passadas não
> estão presentes. São 8. Só tem a '1.2.4+dfsg1-3'. Errei ai.
Também me referi a isso, pois, como você sabe, sempre confiro o antes
e o depois com um debdiff.
> Desta vez usei 'sed -i 's#^ [\*,\-]# #g' debian/bogofilter-bdb.NEWS'
> e não toquei no arquivo. Ainda tem uns '*', por isso troquei sua
> sugestão no debian/changelog. Não tem lintian, então deixei.
>
>>
>> - Para o d/copyright seria bom dizer que passou a usar o formato 1.0.
>
> Feito.
Certo. Mas está escrito debian/copyrighti, em vez de debian/copyright.
Ainda, o bloco está fora da ordem alfabética. Deveria vir depois de
d/control.
>> 2. d/copyright
>> - No bloco 'Files: contrib/*', você não pode usar 'GPL-2 or GPL-2+ or
>> GPL-3+'. O arquivo contrib/bogofilter-milter.pl, por exemplo, é
>> claramente GPL-2+. Assim sendo, não se pode submetê-lo a um triplo
>> licenciamento. Com isso, é necessária uma revisão.
>
> Ok.
Acho que você sabe isso mas você poderia ter unido todos os contrib/
que são GPL-2 em um único bloco, assim como todos os GPL-2+ em outro.
Não está errado como você fez , mas penso que ficaria mais simples com
isso[1].
[1] https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/#copyright-field
>> - Encontrei isso:
>>
>> contrib/spamitarium.pl:# Copyright © 2004 Order amid Chaos, Inc.
>> contrib/stripsearch.pl:# Copyright © 2005 Order amid Chaos, Inc.
>> contrib/vm-bogofilter.el:;; Based on vm-spamassassin.el v1.1,
>> Copyright (C) 2002 by Markus Mohnen
>> contrib/bfproxy.pl:# Copyright © 2003 Order amid Chaos, Inc.
>
> Ok.
O contrib/bfproxy.pl tem copyright em 2003 e o bloco no qual ele está
inserido começa em 2004. Se puder, coloque os arquivos do bloco em
ordem cronológica/alfabética. Aliás, por favor, faça uma revisão geral
de ordem alfabética. Tem blocos fora de ordem, dificultando qualquer
revisão e arquivos fora de ordem cronológica/alfabética dentro dos
blocos. Quer um exemplo? A entrada para contrib/vm-bogofilter.el está
duplicada e com informações diferentes dentro do d/copyright. Se
estivesse na ordem alfabética, você teria visto isso.
Para Order amid Chaos, Inc., você pode usar o email
neo+stripsearch@orderamidchaos.com.
> O contrib/vm-bogofilter.el está lá, mas tem um lintian:
>
> I: bogofilter source: unused-file-paragraph-in-dep5-copyright paragraph at line 37
Hum... Eu tinha visto antes de você falar. É por causa do que eu disse acima.
>> - Quanto ao arquivo que Trevor aponta para a URL, o copyright é de
>> 2004, quando ainda não havia GPL-3.
>
> Coloquei então GPL-2+. Mas mantive o comentário.
Esse é o principal motivo pelo qual não fiz upload. Você está se
contradizendo com esse comentário. Ficou algo como "coloquei GPL-2 mas
é GPL-3'. Seria bom colocar um comentário que justificasse o GPL-2,
como o que escrevi no email anterior.
>> - Encontrei isso:
>>
>> gsl/specfunc/erfc.c: * Copyright (C) 1996, 1997, 1998, 1999, 2000,
>> 2001, 2002, 2003 Gerard Jungman
>
> O Gerard está lá.
Desculpe, dever ter sido erro meu.
>
>> src/bogoQDBMupgrade.c: Copyright (C) 2004 Stefan Bellon
>
> Ok.
>
>> src/bogoconfig.c: "Copyright (C) 2002-2003 Eric S. Raymond,
>> Adrian Otto, Gyepi Sam\n\n"
>
> No meio do arquivo tem mais gente. Adicionei.
Coloque na ordem cronológica/alfabética, por favor.
>> 3. O pacote não constrói em uma jaula limpa (teste com cowbuilder).
>
> Construí três vezes com o cowbuilder. Passou aqui.
Estranho. Eu tinha tentado duas vezes aqui. Mas agora constrói. Deve
ter sido erro aqui. Desculpe.
É isso!
Abração!
Eriberto
Reply to: