<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 12 Jul 2019, 6:09 pm Chris Lamb, <<a href="mailto:lamby@debian.org">lamby@debian.org</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  * DIFFOSCOPE_FAIL_TESTS_ON_MISSING_TOOLS=''<br>
    → Fails; the required version is missing and unlisted.<br>
<br>
  * DIFFOSCOPE_FAIL_TESTS_ON_MISSING_TOOLS="foo bar" <br>
    → Fails; the required version is missing and unlisted.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">These shouldn't fail.  As in, the program is present, but the required version is missing.  That variable isn't holding versions after all.</div><div dir="auto"><br></div><div dir="auto">At least, having that fail in this case wasn't my intention when I wrote that thing.  If you went down that route it would make that variable/setting unusable in any circumstance where simply the required version is missing.</div><div dir="auto"><br></div><div dir="auto">Having said all of this, I think we should make the failing test able to run with either of the file versions.  `file` keeps changing and breaking our tests, but it would be awful to only support running the testsuite with the latest version available.</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"> <br>
</blockquote></div></div></div>