Re: RFS: mp3wrap/0.5-4 - ITA: mp3wrap - Utility for MP3 wrapping (rolling multiple MP3s into one)

Oi Marcio, tudo bem?

Vamos lá.

1. d/changelog:

- Seria melhor usar debian/source em vez de d/source. Eu costumo
escrever d/xyz mas é só no texto de revisão. No debian/changelog é
melhor estar completo.

- Você alterou o d/compat mas não citou isso no d/changelog. Eu sugiro
colocar como terceira linha: Updated DH level to 9. Isso também
tornará desnecessária a frase sobre Build-Depends no parágrafo que
trata do debian/control. EM TEMPO: vejo que você colocou isso no
final. Ficaria mais claro na terceira linha, na minha opinião.

- Apesar de não estar errado, a frase "Set myself as Maintainer" é
dispensável, uma vez que você já disse "New maintainer".

- Por favor, use Vcs-Git em vez de Vcs-git para citar o campo. (por
acaso, essa linha ficou fora de ordem alfabética no changelog)

- A descrição longa não pode exceder 80 caracteres, e não 60. Assim,
se preferir deixá-la um pouquinho maior, não haverá problema.

- A descrição curta sim, pode ter até 60 caracteres para não exceder
80 caracteres no total (incluindo agora o nome do campo). Mas há uma
pequena tolerância. Assim, com o nome do campo (Description: )
poderemos chegar até 80 caracteres. Apesar de não estar errado, me
parece um pouco estranho a descrição curta estar tão condensada. Mas
deixarei você decidir como fazer.

- Você precisa explicar um pouco melhor o que fez no d/copyright. Que
tal dizer que passou para o formato 1.0, atualizou os dados dos
autores e dos mantenedores? Voce também, acertadamente, atualizou o
texto da licença.

- Vi que você criou o patch fix_manpage mas isso não está citado.
Sugiro usar o sufixo .patch para que as linhas sejam iluminadas dentro
de editores de texto. Mas isso é opcional.

- Por algum motivo, vejo uma pequena alteração no arquivo mp3wrap.c.
Você desaplicou algum patch? Veja:

--- mp3wrap-0.5/mp3wrap.c    2016-03-21 21:29:36.000000000 -0300
+++ mp3wrap-0.5/mp3wrap.c    2003-01-17 07:15:52.000000000 -0200
@@ -70,8 +70,8 @@
     FILE *file_output, *file_input;
     unsigned long splitpoints[MAXNUMFILE+1], begin=0, end, len=0, crc, fcrc;
     unsigned char filename[512], *wrapindex, c, ext[512];
-    char *ptr;
-    int option, i = 0, j = 0, files, oldfiles = 0, indexsize=0, id3offset=0;
+    char option, *ptr;
+    int i = 0, j = 0, files, oldfiles = 0, indexsize=0, id3offset=0;
     short optoffset = 0, addoption = 0, listoption = 0, verboption =
0, ismp3wrap = 0;

     printf (NAME" Version "VERSION" "DATE". See README and COPYING
for more!\n");

Olhando o conteúdo do arquivo mp3wrap_0.5-3.diff.gz, vejo que o você
desaplicou o patch que foi inserido na revisão 0.5-1.1 e que fechou o
bug 384995. Como o sistema de patches foi modernizado, você precisa
colocar isso dentro de debian/patches/, dando os créditos ao autor
original.


2. d/copyright:

- Esse programa estava no Debian como GPL-2+. No entanto, se você
observar com cuidado a licença, ele é LGPL-2+. Dei uma olhada nos
pacotes anteriores[1] e ele entrou certo no Debian. No entanto, na
terceira revisão (0.5-2), o mantendor se perdeu e alterou a licença.
Então, até que se prove ao contrário, a licença está errada há 6
revisões (7 com a sua), ou seja, desde 2008. A confusão ocorreu em um
trabalho de QA, quando ele ficou órfão. O empacotamento original
também é LGPL-2+. Você tem que dizer no d/changelog que está
revertendo a licença.

[1] http://snapshot.debian.org/binary/mp3wrap/

- Quanto aos mantenedores, por favor coloque os nomes de 2008 em ordem
alfabética.


3. d/dirs: esse arquivo é dispensável, pois o autotools sabe criar o
diretório usr/bin


4. d/docs: eu penso que o arquivo README nada acresce para o usuário
final. Você poderia removê-lo? Também não vejo necessidade do AUTHORS.
Tem apenas o nome completo do autor e um email alternativo. Se achar
melhor, adicione um campo 'Upstream-Contact' no cabeçalho do
d/copyright e coloque os dados novos (aqui pode citar os dois emails e
o nome completo). Alternativamente, você pode usar um campo Comment no
fim do bloco 'Files: *'. Com uma dessas opções ficaria mais
organizado.

5. Patch: o campo Last-Update está com a data invertida. Segundo a
DEP3[2]: "It should use the ISO date format YYYY-MM-DD".

[2] http://dep.debian.net/deps/dep3/


6. d/rules: tem uma linha em branco a mais no fim desse arquivo. :-)


7. d/watch: tem um pequeno problema aqui. O seu d/watch está
procurando por essas extensões:

.tar.gz
.tar.bz2
.tar.xz
.tgz.gz
.tgz.bz2
.tgz.xz

Eu acho que você queria isso:

version=4
http://sf.net/mp3wrap/mp3wrap-(.*)-src\.(?:tgz|tar\.(?:gz|bz2|xz))


8. Eu vejo alguns lintians aqui (jaula desatualizada?):

I: mp3wrap: spelling-error-in-manpage usr/share/man/man1/mp3wrap.1.gz
informations information
I: mp3wrap: spelling-error-in-manpage usr/share/man/man1/mp3wrap.1.gz
splitted split
I: mp3wrap: spelling-error-in-manpage usr/share/man/man1/mp3wrap.1.gz
informations information
I: mp3wrap: spelling-error-in-manpage usr/share/man/man1/mp3wrap.1.gz
informations information
I: mp3wrap: possible-documentation-but-no-doc-base-registration

É isso! Parabéns por estar adotando mais um pacote!

Grande abraço,

Eriberto


Em 20 de março de 2016 20:23, Marcio de Souza Oliveira
<[email protected]> escreveu:
> Corrigido o changelog, agora está Set myself as Maintainer.
> Novamente obrigado Samuel.
>
> Abraço
>
>> 2016-03-18 0:34 GMT-03:00 SamuelOPH <[email protected]>:
>>         Olá Marcio, boa noite
>>
>>
>>         Se me permite (por favor me corrijam se eu estiver errado), eu
>>         acho que a parte "- Set my self like Maintainer." ficaria
>>         melhor como "- Set myself as the Maintainer" ou "- Set myself
>>         as Maintainer". A palavra "myself" não contém espaços.
>>
>>
>>         Abraço
>>
>>
>>
>>
>>         Samuel Henrique O. P. [samueloph]
>>         Técnico em Informática - UTFPR [2012].
>>         Estudante de Engenharia de Computação - UTFPR.
>>
>>         2016-03-18 0:19 GMT-03:00 Marcio de Souza Oliveira
>>         <[email protected]>:
>>                 Prezados,
>>
>>                 Estou procurando um sponsor para o pacote mp3wrap, eu
>>                 sou DM, estou
>>                 adotando o pacote.
>>
>>                  * Nome do pacote     : mp3wrap
>>                  * Versão             : 0.5-4
>>                  * Upstream Author    : M. Trotta -
>>                 <[email protected]>
>>                  * URL                :
>>                 http://mp3wrap.sourceforge.net/
>>                  * License            : GPL-2+
>>                    Section            : sound
>>
>>                 Produz os seguintes binários:
>>
>>                 mp3wrap   - Utility for MP3 wrapping (rolling multiple
>>                 MP3s into one)
>>
>>                 Para acessar o pacote:
>>                 http://mentors.debian.net/package/mp3wrap
>>                 http://mentors.debian.net/debian/pool/main/m/mp3wrap/mp3wrap_0.5-4.dsc
>>
>>                 Changelog:
>>
>>                 mp3wrap (0.5-4) unstable; urgency=medium
>>
>>                   * New maintainer. Thanks Emanuele, Nico, Michael
>>                     and Paul. (Closes: #457131).
>>                   * Created the directory d/source and format file.
>>                   * debian/control:
>>                       - Added ${misc:Depends} in Depends field.
>>                       - Adjusted the long and short description to
>>                         not exceed 60 characters per line.
>>                       - Bumped Standards-Version to 3.9.7.
>>                       - Added Vcs-git and Vcs-Browser.
>>                       - Updated Build-Depends to use debhelper >= 9.
>>                       - Set my self like Maintainer.
>>                   * debian/copyright:
>>                       - Updated file.
>>                   * debian/rules:
>>                       - Enabled hardening options.
>>                       - Updated to use autoreconf.
>>                       - Updated to small format.
>>                   * debian/watch:
>>                       - Updated to version 4.
>>                       - Updated file.
>>                   * Updated DH level to 9.
>>
>>
>>
>>
>>
>>
>> --
>> Marcio de Souza Oliveira
>> http://www.respirandoti.com.br
>> -----------------------------------------------------------------------------------------------
>> Cansou de vírus, pragas virtuais, travamentos e falta de desempenho?
>> O melhor anti vírus é usar Linux :)
>> Debian (http://www.debian.org/)
>>
>