<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Ok, here's my review of the package:<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">What needs to be done in order for the upload to be approved (by me, i know there are other DDs who doesn't mind some problems that i'm considering here):<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">a) Bump DH level to 10: You're currently using DH 9.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">b) Declare a proper DH version on d/control: Please have a look at [1], and bare in mind that you will be using DH 10 in your case.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">c) Remove the watchfile: It is pointing to code.google, which is obsolete and won't have any new releases (besides upstream is non-active for at last 7 years).<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">d) Remove double GPL-2+ definition: More information here[2].<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">e) Remove debhelper template from d/copyright: Looks like you forgot to have a look at line 6.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">f) Add yourself on d/copyright: After all, you did changes to the packaging and now you can add yourself as one of the copyright holders of debian/*<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">h) Open an ITP bug and close it using d/changelog: Please read this[3].<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">i) Remove trailing whitespaces: I can see a lot of them on d/control and on on d/rules, you can either use $ wrap-and-sort -a or enable trailing whitespace highlighting on vim[4], i recommend you to use wrap-and-sort to fix this.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">j) Remove dh comments from d/rules: Self explanatory.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">k) Remove d/docs: None of the files there is useful, the only one that is arguable (is a doc in fact) is README.txt, but a manpage should make it useless.<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">l) Patch Makefile: The makefile is missing CPPFLAGS, LFDLAGS and the CFLAGS is being overriden (which causes a useless -dbgsym package), please have a look at[5] for hints on how to fix that. After that you may also add "<span class="gmail-st">export DEB_BUILD_MAINT_OPTIONS = hardening=+all" at d/rules in order to enable bindnow.</span><br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">m) Rename dnsmap-bulk.sh to dnsmap-bulk: We are encouraged not to ship script files with language extensions, you can read more about this here[6].<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">n) Remove d/dirs: it is not needed to put usr/bin there and usr/share/dnsmap is already being used on d/dnsmap.install, more info[7].<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">o) Bonus: Create a manpage: I consider this important, shouldn't be so trouble since one can use info already provided on README.txt.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">That's all, i know that there are a lot of points, please don't be discouraged by my review, most of these problems are newbie ones and i believe you won't be committing them again.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">If you have any questions about how to do any of these changes, or feel like not having time to do them, please reply and i can help you. If you're planning to become a DM/DD, its better if you do the changes yourself.<br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">After all these changes are made, i can have another look at the package just to confirm that i didn't missed anything (i'm pretty sure i not).<br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)"><br>[1]<a href="https://www.debian.org/doc/manuals/maint-guide/dreq.en.html#control" target="_blank">https://www.debian.org/doc/<wbr>manuals/maint-guide/dreq.en.<wbr>html#control</a><br>[2]<a href="https://lintian.debian.org/tags/dep5-copyright-license-name-not-unique.html" target="_blank">https://lintian.debian.org/<wbr>tags/dep5-copyright-license-<wbr>name-not-unique.html</a><br>[3]<a href="https://www.debian.org/doc/manuals/developers-reference/pkgs.html#newpackage" target="_blank">https://www.debian.org/doc/<wbr>manuals/developers-reference/<wbr>pkgs.html#newpackage</a><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">[4]Put this in your .vimrc: inoremap <F2> <CR><C-R>=repeat(' ',col([line('.')-1,'$'])-col('<wbr>.'))<CR><C-O>:.retab<CR><br>[5]<a href="http://eriberto.pro.br/blog/?p=2222">http://eriberto.pro.br/blog/?p=2222</a><br>[6]<a href="https://www.debian.org/doc/debian-policy/ch-files.html">https://www.debian.org/doc/debian-policy/ch-files.html</a> ['10.4 Scripts' paragraph]<br>[7]<a href="https://www.debian.org/doc/manuals/maint-guide/dother.en.html#dirs">https://www.debian.org/doc/manuals/maint-guide/dother.en.html#dirs</a><br><br></div><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Regards, and happy new year.<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><span style="font-family:verdana,sans-serif"><span style="color:rgb(102,102,102)">Samuel Henrique <samueloph></span></span></div></div></div></div></div></div></div>
<br><div class="gmail_quote">2016-12-30 23:43 GMT-02:00 Samuel Henrique <span dir="ltr"><<a href="mailto:samueloph@gmail.com" target="_blank">samueloph@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Sure, i would like to do that!<br><br></div><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">Just give me 14 hours and i will reply here with my review (i'm gonna do it tomorrow).<br></div></div><div class="gmail_extra"><br clear="all"><div><div class="m_-6655423342194793410gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><span style="font-family:verdana,sans-serif"><span style="color:rgb(102,102,102)">Samuel Henrique <samueloph></span></span></div></div></div></div></div></div></div><div><div class="h5">
<br><div class="gmail_quote">2016-12-30 19:40 GMT-02:00 Gianfranco Costamagna <span dir="ltr"><<a href="mailto:locutusofborg@debian.org" target="_blank">locutusofborg@debian.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<span><br>
>hello guys, i have packaged dnsmap and then adjusted some d/rules things by watching to the kali package, is someone interested in checking if everything is ok?<br>
<br>
<br>
</span>there are some improvements that I would like to add, but I prefer somebody else to do the review :)<br>
<br>
Samuel, can you please give it a look?<br>
<br>
I see something like 4 things I would like to see fixed before sponsoring :p<br>
<span class="m_-6655423342194793410HOEnZb"><font color="#888888"><br>
<br>
G.<br>
</font></span></blockquote></div><br></div></div></div>
</blockquote></div><br></div></div>