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
<m.desouza20@gmail.com> escreveu:
> Corrigido o changelog, agora está Set myself as Maintainer.
> Novamente obrigado Samuel.
>
> Abraço
>
>> 2016-03-18 0:34 GMT-03:00 SamuelOPH <samueloph@gmail.com>:
>> 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
>> <m.desouza20@gmail.com>:
>> 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 -
>> <matteo.trotta@lib.unimib.it>
>> * 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/)
>>
>
Reply to: