<div dir="ltr">Thanks for the PR. Makes sense.<div>I've merged it.</div><div><br></div><div>Best.<br><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 14, 2019 at 4:47 PM Klemens Nanni <<a href="mailto:kn@openbsd.org" target="_blank">kn@openbsd.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">The last three commits already made configure recommend installing<br>
Python 3 packages and look for versioned executables, however with a low<br>
priority.<br>
<br>
This is a problem on systems such as OpenBSD 6.5 with a default Python<br>
version at 2.7, where 3.7 flavored Python packages get installed with<br>
a "-3" binary suffix.  That is, when both rst2man and rst2man-3 are<br>
installed at configure time, the lower version will be picked unless<br>
explicitly passed through `--with-feature' arguments.<br>
<br>
Regardless of this specific case, trying more specificly versioned tool<br>
names first seems correctly in line with recent development and less<br>
error prone, so change it accordingly.<br>
---<br>
 <a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> | 6 +++---<br>
 1 file changed, 3 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a> b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
index fe4225069..fbfea3e45 100644<br>
--- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
+++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
@@ -41,7 +41,7 @@ AC_ARG_WITH([rst2man],<br>
   AS_HELP_STRING([--with-rst2man=PATH], [Location of rst2man (auto)]),<br>
   [RST2MAN="$withval"],<br>
   AC_CHECK_PROGS(RST2MAN,<br>
-    [rst2man rst2man.py rst2man-3.6 rst2man-3],<br>
+    [rst2man-3.6 rst2man-3 rst2man rst2man.py],<br>
     [no]))<br>
 if test "x$RST2MAN" = "xno"; then<br>
   AC_MSG_ERROR(<br>
@@ -52,7 +52,7 @@ AC_ARG_WITH([sphinx-build],<br>
   AS_HELP_STRING([--with-sphinx-build=PATH], [Location of sphinx-build (auto)]),<br>
   [SPHINX="$withval"],<br>
   AC_CHECK_PROGS(SPHINX,<br>
-    [sphinx-build sphinx-build-3.6 sphinx-build-3],<br>
+    [sphinx-build-3.6 sphinx-build-3 sphinx-build],<br>
     [no]))<br>
 if test "x$SPHINX" = "xno"; then<br>
   AC_MSG_ERROR(<br>
@@ -63,7 +63,7 @@ AC_ARG_WITH([rst2html],<br>
   AS_HELP_STRING([--with-rst2html=PATH], [Location of rst2html (auto)]),<br>
   [RST2HTML="$withval"],<br>
   AC_CHECK_PROGS(RST2HTML,<br>
-     [rst2html rst2html.py rst2html-3.6 rst2html-3],<br>
+     [rst2html-3.6 rst2html-3 rst2html rst2html.py],<br>
      "no"))<br>
<br>
 if test "x$RST2HTML" = "xno"; then<br>
-- <br>
2.21.0<br>
<br>
<br>
_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org" target="_blank">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" rel="noreferrer" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br>
</blockquote></div>