On Fri, Oct 12, 2012 at 2:08 PM, Josué Ortega <josueort...@debian.org.gt>wrote:
> > > On Fri, Oct 12, 2012 at 1:38 PM, Bart Martens <ba...@debian.org> wrote: > >> Hi Josue, >> >> I had a look at mp3diags at mentors uploaded there on 2012-10-12 06:43. >> I'm >> afraid that bug 689811 is not yet fully solved. >> >> For example, upstream wrote that "the icons are mainly taken from KDE 4' >> Oxygen >> icon set", and I don't see the copyright holder(s) mentioned in >> debian/copyright. >> >> Also, the text from upstream you quoted is helpful for clarification, as >> part >> of the fix for bug 689811. I suggest to add the exact text in >> debian/copyright >> including the From and Date headers. >> >> And since you're updating the package anyway : the white lines in >> debian/rules >> are still there. You may want to remove them before requesting an >> unblock to >> debian-release with a debdiff. >> >> Regards, >> >> Bart Martens >> > > > Hi Bart, > Thanks for the review , I'll try to fix it today > Regards, > -- > Josue Ortega > openfecks.wordpress.com > PGP key 4096R/AB4AFD3F 2012-08-20 > Fingerprintt = 7733 B328 D279 5F5B E232 5ADD 0150 9D5C AB4A FD3F > > > Hi Bart, I've fixed the debian/copyright and debian/rules as you told me , hoping that the files are right this time. Thanks :) Heres is the mentors adress for mp3diags: http://mentors.debian.net/package/mp3diags Regards -- Josue Ortega openfecks.wordpress.com PGP key 4096R/AB4AFD3F 2012-08-20 Fingerprintt = 7733 B328 D279 5F5B E232 5ADD 0150 9D5C AB4A FD3F