Hi Antti,



>Not changed yet, but inquiry sent to 
>http://lists.alioth.debian.org/cgi-bin/mailman/private/pkg-qemu-devel/2015-August/006604.html
>so if they have good suggestions, then one more upload to mentors
>will be needed.


I can't/won't read, but ok, it is good as-is, let me know if you will need a 
new upload.

>> 6) d/watch file: you might want to use the Debian redirector instead of 
>> pointing directly to sf
>> https://qa.debian.org/cgi-bin/fakeupstream.cgi?upstream=sf/qemuctl
>
>Ok, didn't know such thing was in place. Qemuctl is not listed
>in re-director so adding it there would require another ticket into BTS.
>To avoid this hassle, I'm leaving the current URL because it is working 
>ok with uscan. 


Actually it is listed, but as said on irc, if the sf link is automatically 
translated by uscan
maybe it is better to leave it

[snip]
>dget -x 
>http://mentors.debian.net/debian/pool/main/q/qemuctl/qemuctl_0.3.1-4.dsc


last two nitpicks and issues:

1) there is no need to call the files under debian directory
"qemuctl.manpages" as long as you have only one binary package "manpages" is 
good enough.

(note: I'm not asking to change that)


2) you might find useful to remove the "debian/dirs" file, because it should be
useless since you already have your install file.


3) the license of the package seems to be wrong:

accoding to licensecheck and manual checks I did, the files have a license
GPL-3+, not GPL-2+
(and thanks for promptly replying to the feedbacks)



please let me know how do you feel about the last three things,
and I'll do the final checks&upload.


cheers,

Gianfranco


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
https://lists.debian.org/393021171.1415269.1439072646966.javamail.ya...@mail.yahoo.com

Reply via email to