<p dir="ltr">Hi Chris!</p>
<p dir="ltr">I see. I'll be more careful with python coding style<br>
next time. (:</p>
<p dir="ltr">About the testsuite: Firstly I ran all tests, but some were naturally failing (like test_debian, I guess). Then I focused on getting to work at least the one test I was writting, since I don't know much about the project yet.</p>
<p dir="ltr">Thank you very much for your support and for your quick fixes. I hope to be able to contribute more in the future. :D</p>
<p dir="ltr">Juliana<br><br><br><br></p>
<br><div class="gmail_quote"><div dir="ltr">Em sex, 21 de jul de 2017 06:48, Chris Lamb <<a href="mailto:lamby@debian.org">lamby@debian.org</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tags 866120 + pending<br>
thanks<br>
<br>
Chris Lamb wrote:<br>
<br>
> Woo, thanks! First, am going to merge this into the existing<br>
> wishlist bug…<br>
<br>
… And I've now also merged the code into our Git repo. Thanks!<br>
<br>
I did make some changes after committing which might be useful<br>
to know/understand for next time. First two (easy) style fixups<br>
that should be fairly self-explanatory:<br>
<br>
  <a href="https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=9d89fdda0101915eedf931a4c9127918cdfea79c" rel="noreferrer" target="_blank">https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=9d89fdda0101915eedf931a4c9127918cdfea79c</a><br>
<br>
  <a href="https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=9123eb091c49db4d7685e36d33f539bcbecefae1" rel="noreferrer" target="_blank">https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=9123eb091c49db4d7685e36d33f539bcbecefae1</a><br>
<br>
Next, I had to add MissingFile support to your comparator; did you run<br>
the entire testsuite? *g*  In particular, the epub tests were failing<br>
for me before I added:<br>
<br>
  <a href="https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=2422e0f0f8a8ae66da0438f9fc64376faa154b53" rel="noreferrer" target="_blank">https://anonscm.debian.org/git/reproducible/diffoscope.git/commit/?id=2422e0f0f8a8ae66da0438f9fc64376faa154b53</a><br>
<br>
Anyway, thank you again for your contribution. Let me know if you have<br>
any questions on the above, but we all look forward to future patches<br>
from yourself…<br>
<br>
<br>
Best wishes,<br>
<br>
--<br>
      ,''`.<br>
     : :'  :     Chris Lamb, Debian Project Leader<br>
     `. `'`      <a href="mailto:lamby@debian.org" target="_blank">lamby@debian.org</a> / <a href="http://chris-lamb.co.uk" rel="noreferrer" target="_blank">chris-lamb.co.uk</a><br>
       `-<br>
</blockquote></div>