<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"></div><div dir="auto"><div>Hi Andreas, <br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 10 Mar, 2020, 9:21 PM Andreas Tille, <<a href="mailto:andreas@an3as.eu" target="_blank">andreas@an3as.eu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Fine.  That works.  Sorry when I have given a bit short-cutted advise.<br>
I've completed the long description.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Oh sorry I missed. Actually I changed it at first but it didn't work so I went back and forth a few times and forgot to complete it. </div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="gmail_default" style="font-family:arial,helvetica,sans-serif"></span>In my commit 6cff0c2377861eb179d4ec06514df75df1323510 I tried to fix this<br>
but did not test which I'll leave you for an exercise. ;-)<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks, this worked, but initially it didn't compile. The error was that it couldn't find any *.1 files in debian/idba and debian/idba-extra.<br>However, when I renamed the two folders to idba-man and idba-extra-man, it suddenly worked. Do you have any ideas why this was so?</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:Arial,Helvetica,sans-serif"> </span><br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">When doing so I stumbled upon the script run-unittest.py which I think<br>
should not end up in /usr/bin.  I installed it into docs and your second<br>
exercise could be to check whether it really does some sensible testing<br>
at all.  As far as I can see it is just triggering the build time test<br>
but it does not look as if it be useful to install on users machines.<br>
Even if it would make sense the name /usr/bin/run-unittest.py is way to<br>
generic and should be avoided.  Same with /usr/bin/scan.py - please<br>
check whether this script makes sense in /usr/bin it should be probably<br>
renamed (to something like idba_scan.py).  Otherwise it should also go<br>
to /usr/share/doc/idba (or left out fully).<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Yes, it should definitely not go to debian/bin, sorry.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Line 17 in debian/rules [1] says, "for the moment the role of these scripts is totally unknown but they do not seem to be necessary," and proceeds to copy run-unittest.py, scan.py, validate_blat and validate_blat_parallel to usr/lib/idba.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I tried commenting out these lines, and the package seems to work just fine. They seem to be required only during compilation and testing, so they probably shouldn't be installed as binaries in the final user installation. Maybe we could let them be in the lib folder? </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I'm not sure, however. Please take a look and let me know if I should change it to docs.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> I have also changed the test files now. Only a few necessary files are<br>
> included, and the rest are generated during testing.<br>
<br>
That's really good!  The only think I'm missing now is that we should<br>
always mention the origin where the data were obtained from (may be in<br>
a script calling wget for downloading).  It makes also sense to drop<br>
some note about the license of these files.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I have added a file called data-source.sh in debian/tests/test-data. Is this the right way to do it?</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Also, I obtained the data from their research papers about these tools. What kind of license would be applicable in this case?</div></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">And where do I get the license from?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> Let me know if I missed anything, otherwise please review and sponsor these<br>
> changes.<br>
<br>
What you did is basically correct and extremely helpful to finalise the<br>
autopkgtest.  Please inspect my recent commits and try to provide the<br>
missings I mentioned above in this mail.</blockquote><div><br><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"><span class="gmail_default" style="font-family:arial,helvetica,sans-serif"></span>I added a few examples by extending the debian/createmanpages script.<br>
I'd perfectly accept some kind of "lazyness" here.  I would not mind<br>
uploading the current incomplete status despite of the lintian warnings.<br>
I usually decide according to the "importance" of the tool that needs a<br>
manpage.  My motivation was to get "the first three in list" and also<br>
the ones we are using in autopkgtest.  Feel free to decide whether you<br>
want a complete set of manpages or not.</blockquote><div><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Okay. I think the set of manpages we have should be enough, given that the other tools aren't too important.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks for the intro to dh_install, man pages and documentation about packages. </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Please let me know if any more work is required on this package.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Regards,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Pranav<br></div></div></div></div></div></div>
</div><div hspace="streak-pt-mark" style="max-height:1px"><img alt="" style="width:0px;max-height:0px;overflow:hidden" src="https://mailfoogae.appspot.com/t?sender=aYmFsbGFuZXlwcmFuYXZAZ21haWwuY29t&type=zerocontent&guid=4a2dd5b8-26e7-4996-afa0-46996c7ee4d4"><font color="#ffffff" size="1">ᐧ</font></div>