<div dir="ltr"><div><div><div>Hi Afif,<br><br></div>Thanks for your feedback. I've made the changes that you suggested. <br></div><div><br></div><div>Also, I've changed the upstream source to the linux-rdma github repository and updated the code to 0.4.10.</div><div><br></div>Regarding the copyright, BSD should not have been in there and has been removed. COPYING is identical to /usr/share/common-licenses/GPL-2. The GPL-2 license section in copyright now contains the GPL-2 notice and a pointer to /usr/share/common-licenses/GPL-2. Please let me know if that is not the right thing to do.<br><br></div>Thanks, again!<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"><div><div dir="ltr"><div><div dir="ltr">Brian T. Smith<br>System Fabric Works<br>Senior Technical Staff<br><a href="mailto:bsmith@systemfabricworks.com" target="_blank">bsmith@systemfabricworks.com</a><br>(512) 293-4472<br></div><div>GPG Key: B3C2C7B73BA3CD7F<br></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Thu, Nov 9, 2017 at 11:56 PM, Afif Elghraoui <span dir="ltr"><<a href="mailto:afif@debian.org" target="_blank">afif@debian.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
على الخميس  9 تشرين الثاني 2017 ‫11:19، كتب Brian Smith:<br>
> Hello Afif,<br>
><br>
> Sorry for cross-posting. I wasn't sure if the forwarding from pkg-ofed<br>
> is yet configured.<br>
<br>
</span>No worries. It turns out not to be the reason why I'm getting duplicate<br>
messages, so I'm restoring the cross-posting.<br>
<span class=""><br>
><br>
> To answer your question, I am subscribed to debian-hpc.<br>
><br>
<br>
</span>Ok, so I can just reply-list now. :)<br>
<span class=""><br>
> I really appreciate your reviewing the qperf package and look forward<br>
> to your feedback. Thanks!<br>
><br>
<br>
</span>Here we go.<br>
<br>
debian/control<br>
==============<br>
<br>
> Depends: ${misc:Depends},<br>
>  ${shlibs:Depends},<br>
>  libibverbs1,<br>
>  librdmacm1<br>
<br>
${shlibs:Depends} will handle the shared library dependencies for you,<br>
along with minimal versions where possible. You shouldn't set those<br>
shared library dependencies manually unless absolutely necessary. If I<br>
build after removing those two explicit lines, I get the final package<br>
appropriately having:<br>
<br>
Depends: libc6 (>= 2.15), libibverbs1 (>= 1.1.6), librdmacm1 (>= 1.0.19)<br>
<br>
debian/rules<br>
============<br>
With debhelper compat 10, parallel and autoreconf are enabled by default<br>
[1], so you can simply use `dh $@`.<br>
<br>
<br>
debian/changelog<br>
================<br>
The changelog gets updated once per upload. Everything as of now is part<br>
of the initial release, so you should just get rid of the everything<br>
else there. I'll change the target distribution once it's ready to go,<br>
right before I build the version to upload.<br>
<br>
<br>
debian/copyright<br>
================<br>
* The source code indicates GPL 2, not GPL 2+. You have correctly<br>
labeled it GPL-2, but the text you included has the "or (at your option)<br>
any later version" clause. That should be removed for this package.<br>
<br>
* The BSD option is for a 2-clause license, not 3. Please correct this;<br>
and it's best to use the exact wording they give for the license text.<br>
<br>
debian/tests<br>
============<br>
Since the debian/tests/control file here defines no tests, I don't see a<br>
reason for this directory to exist.<br>
<br>
debian/docs<br>
===========<br>
The NEWS file is empty; I wouldn't install an empty file.<br>
<br>
<br>
Everything else looks good to me. Let me know when I should have another<br>
look--and no need to upload to mentors; I'm working with the git<br>
repository. Thanks for working on this package!<br>
<br>
regards<br>
Afif<br>
<br>
1. <a href="https://lists.debian.org/debian-devel/2016/04/msg00018.html" rel="noreferrer" target="_blank">https://lists.debian.org/<wbr>debian-devel/2016/04/msg00018.<wbr>html</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Afif Elghraoui | عفيف الغراوي<br>
<a href="http://afif.ghraoui.name" rel="noreferrer" target="_blank">http://afif.ghraoui.name</a><br>
<br>
______________________________<wbr>_________________<br>
Pkg-ofed-devel mailing list<br>
<a href="mailto:Pkg-ofed-devel@lists.alioth.debian.org">Pkg-ofed-devel@lists.alioth.<wbr>debian.org</a><br>
<a href="http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-ofed-devel" rel="noreferrer" target="_blank">http://lists.alioth.debian.<wbr>org/cgi-bin/mailman/listinfo/<wbr>pkg-ofed-devel</a></font></span></blockquote></div><br></div>