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

Reply via email to