<div dir="ltr">mh, no.  I think some logic needs tweaking, as file is definitely present (it's an hard dependency of diffoscope), and that test should just be skipped.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 11, 2019 at 11:42 PM Chris Lamb <<a href="mailto:lamby@debian.org">lamby@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Steve Langasek wrote:<br>
<br>
> At the very least, it seems this should be a versioned test dep on file (>=<br>
> 5.37), but perhaps it should also be a versioned runtime dependency. I<br>
> haven't looked to see what the impact is of the wrong version of 'file' when<br>
> DIFFOSCOPE_TESTS_FAIL_ON_MISSING_TOOLS is not set.<br>
<br>
I think we need to add "file" to the DIFFOSCOPE_TESTS_MISSING_TOOLS<br>
list in debian/tests/pytest. Mattia, can you confirm?<br>
<br>
<br>
Regards,<br>
<br>
-- <br>
      ,''`.<br>
     : :'  :     Chris Lamb<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>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">regards,<br>                        Mattia Rizzolo<br><br>GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.<br>more about me:  <a href="https://mapreri.org" target="_blank">https://mapreri.org</a>                             : :'  :<br>Launchpad user: <a href="https://launchpad.net/~mapreri" target="_blank">https://launchpad.net/~mapreri</a>                  `. `'`<br>Debian QA page: <a href="https://qa.debian.org/developer.php?login=mattia" target="_blank">https://qa.debian.org/developer.php?login=mattia</a>  `-<br></div>