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 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 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+stripsea...@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
Re: RFS: bogofilter -- fast Bayesian spam filter [QA-RC]
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Olá, > > 1. d/changelog: > > - Para mim ficou bem confusa a explicação sobre o que houve com os > arquivos debian/bogofilter-bdb.docs e debian/bogofilter-sqlite.docs. > Eu entendi olhando um debdiff e também lendo o seu email. Coloquei separado. > > - Há novos links para usr/share/man/man1/bogofilter.1.gz que não estão > declarados. Feito. > > - 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. 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 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. 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. > > > 2. d/copyright > > - No bloco 'Files: *', o Greg está fora da ordem cronológica. Feito. > > - 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. > > - 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/vm-bogofilter.el está lá, mas tem um lintian: I: bogofilter source: unused-file-paragraph-in-dep5-copyright paragraph at line 37 > > - 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. > > - No bloco 'gsl/*', falta o ano de copyright para Jason. Veja: > > gsl/gsl_cdf.h: * Copyright (C) 2002 Jason H. Stover. > gsl/cdf/gamma.c: * Copyright (C) 2003 Jason Stover. Ok. > > - Encontrei isso: > > gsl/specfunc/erfc.c: * Copyright (C) 1996, 1997, 1998, 1999, 2000, > 2001, 2002, 2003 Gerard Jungman O Gerard está lá. > 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. > > - Aqui temos GPL-2 apenas: > > src/tests/unsort.pl:# Copyright (C) 2004 by Matthias Andree > src/tests/unsort.pl-# This file is under the GNU General Public License, v2 > > src/tests/t.leakfind:# (C) Copyright 2005 Matthias Andree. GNU GPL v2 Ok. > > - Também encontrei isso: > > src/bogoutil.c: "Copyright (C) 2002-2003 Gyepi Sam.\n\n" Estão todos em 'Files; *'. Também tem mais gente no meio do arquivo. > > > 3. O pacote não constrói em uma jaula limpa (teste com cowbuilder). Construí três vezes com o cowbuilder. Passou aqui. dpkg-genchanges: warning: package bogofilter-tokyocabinet-dbgsym listed in files list but not in control info dpkg-genchanges: warning: package bogofilter-sqlite-dbgsym listed in files list but not in control info dpkg-genchanges: warning: package bogofilter-bdb-dbgsym listed in files list but not in control info dpkg-genchanges: info: not including original source code in upload dpkg-source --after-build bogofilter-1.2.4+dfsg1 dpkg-buildpackage: info: binary and diff upload (original source NOT included) I: copying local configuration I: Copying back the cached apt archive contents I: unmounting dev/pts filesystem I: unmounting run/shm filesystem I: unmounting proc filesystem I: Current
