<div dir="ltr"><div>Hi Graham,<br><br></div>Thanks for the thorough check.<br><div class="gmail_extra"> <div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>1.  You changed the build-dependency on debhelper to (>= 10), but<br>
debian/compat needs to be bumped to 10 as well for this to have any<br>
effect.<br></blockquote><div><br></div><div>Done.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>2.  You also changed the build-dependency on cmake to (>= 3.0.2),<br>
please mention the reason in debian/changelog.<br></blockquote><div><br></div><div>Done (upstream requires this as of 1.8.4 due to build system changes). <br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>3.  Please also mention bumping to debhelper 10, Standards-Version<br>
4.0.0 and the changes to debian/copyright in debian/changelog<br>
(preferably one line each).<br></blockquote><div><br></div><div>Done.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>4.  As per the Package Tracker for chemps2 [1], the Multiarch Hinter<br>
reports 4 issues.  If you are happy with its suggested changes, please<br>
make them.<br></blockquote><div><br></div><div>Done. Very good idea.<br><br></div><div>One question though: libchemps2-dev installs headers in<br>/usr/include/chemps2/*.h. If you install two different architecture<br>packages and then delete one, will the headers remain?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>5.  Also per the Package Tracker, the Build Log Hardening Checker<br>
reports 75 (of 77) compiler flags hidden.  This can be fixed by<br>
removing the line "set (CMAKE_VERBOSE_MAKEFILE  OFF)" from<br>
CMakeLists.txt.  This can be done in a patch for now, and possibly<br>
included in your next upstream release.<br></blockquote><div><br></div><div>Fixed in a different way (by passing an argument to make), see [2].<br></div><div><br></div><div><div>Let me know if you have questions or if you disagree with the option chosen to fix 5.<br></div><div><br></div><div>Best wishes,<br></div>Sebastian</div><div><br>[1] <a href="https://tracker.debian.org/pkg/chemps2" rel="noreferrer" target="_blank">https://tracker.debian.org/<wbr>pkg/chemps2</a><br>[2] <a href="https://anonscm.debian.org/cgit/debichem/packages/chemps2.git/commit/?id=d0ed8f9c785eaf3108b7c14d37bd678b166a5755">https://anonscm.debian.org/cgit/debichem/packages/chemps2.git/commit/?id=d0ed8f9c785eaf3108b7c14d37bd678b166a5755</a><br></div></div><br></div></div>