<div dir="ltr"><div class="gmail_default" style="font-family:tahoma,sans-serif;color:rgb(0,0,0)">* Hilko Bengen:<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-m_3928821464032662458gmail-im">> As i don't want to start pushing without your ack, I just created a<br>
> new branch debian/samuel with my changes, so you can have a look at<br>
> them before i commit to debian/master, i appreciate any feedback.<br>
<br>
</span>I don't quite see the point in fuzzing around with debian/changelog<br>
before actually preparing an upload<br>
(c36804cf7e02e90361901e718c95e<wbr>fe4ed87d5d3), the other changes look good<br>
to me.<br>
<br>
You may want to collapse related changes such as the bump to DH 10<br>
(f175818f6f05e89a0e9a1a1ad6a54<wbr>77d0c4b43f9) and removal of --parallel<br>
(f74d5ced73d1647b7bf16cb3c7734<wbr>aad2b720e6f) into one commit.</blockquote><div><br></div><div>Done, pushed to debian/master and deleted debian/samuel.<br></div><div>The thing with d/changelog is a bad practice i was doing while packaging with git, that became pretty clear since the first rebase conflict i've got on d/changelog :).<br></div><div>From now on, i'll update changelog separately.<br></div><div> <br></div><div>* Gianfranco:<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-m_3928821464032662458gmail-im">>You may want to collapse related changes such as the bump to DH 10<br>
>(f175818f6f05e89a0e9a1a1ad6a5<wbr>477d0c4b43f9) and removal of --parallel<br>
>(f74d5ced73d1647b7bf16cb3c773<wbr>4aad2b720e6f) into one commit.<br>
<br>
<br>
</span>can you please also drop dh-autoreconf from build-dependencies?<br>
debhelper 10 runs it by default, for this reason it already have an explicit dependency<br>
on it<br>
<a href="https://packages.debian.org/unstable/debhelper" rel="noreferrer" target="_blank">https://packages.debian.org/un<wbr>stable/debhelper</a></blockquote><div><br></div><div>Done, thanks for pointing that out.<br></div><div>You may have noticed that i used my key to sign the changelog instead of yours, because you're not an uploader and them we would have to a Team Upload. I don't know if you're planning to add yourself as an uploader, if so, please feel free to go ahead.<br></div><div><br></div><div>* All:<br><br></div><div>I started doing some bug triage, already closed 2 bugs on BTS and a few of them on Launchpad.<br></div><div><br>There are still a lot of lintians to take care of in order to get a nice packaging, though, most of them looks like an upstream problem with autogenerated files: manpages and .desktop file. We should probably patch them and send upstream.<br><br></div><div>If anyone is planning to help, i think the most important problems are these ones:<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">W: nmap-dbgsym: debug-file-with-no-debug-symbols usr/lib/debug/.build-id/3a/38e7101667fa48176801e2dfa6764aa4f6b397.debug<br>W: nmap-dbgsym: debug-file-with-no-debug-symbols usr/lib/debug/.build-id/8a/99c26fb602336b1ae73c3a57e605f58e8c61f9.debug<br>W: nmap-dbgsym: debug-file-with-no-debug-symbols usr/lib/debug/.build-id/c1/53135488ace5425d9b4d78ef75d7e6b2decf9d.debug<br></blockquote></div></div></div><div class="gmail_extra"><br><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0)" class="gmail_default">​I didn't have time to look further into this, i hope this to be a false-positive, as at a first glance it looks like we're using the -g flag.<br></div><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0);display:inline" class="gmail_default">​CXXFLAGS missing (-fPIE): g++ -c -I../libdnet-stripped/include -Wdate-time -D_FORTIFY_SOURCE=2 -I../nbase -I../nsock/include -g -O2 -fdebug-prefix-map=/home/tmp/nmap-7.31=. -fstack-protector-strong -Wformat -Werror=format-security -Wall  -fno-strict-aliasing   -DHAVE_CONFIG_H -DNPING_NAME=\"Nping\" -DNPING_URL=\"<a href="https://nmap.org/nping\">https://nmap.org/nping\</a>" -DNPING_PLATFORM=\"x86_64-pc-linux-gnu\" -D_FORTIFY_SOURCE=2 EchoHeader.cc -o EchoHeader.o</div></blockquote><div><br><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0);display:inline" class="gmail_default">​"blhc --all" reports we're not using the -fPIE flag, i still didn't check why and if we will have problems using it.<br><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">P: nmap source: no-dep5-copyright<br></blockquote><div><br></div><div>Probably the least important of all, but it'd be nice to have nmap being dep5 compliant, especially considering the importance of the package. <br></div></div>​</div><br clear="all"><div><div class="gmail-m_3928821464032662458gmail_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><div style="font-family:tahoma,sans-serif;color:rgb(0,0,0);display:inline" class="gmail_default">​​</div></span></span></div></div></div></div></div></div></div>
</div></div>