Re: [Patch] Update the INSTALL file for SWIG bindings (Re: svn commit: r1883333 - /subversion/site/staging/docs/release-notes/1.14.html)

Yasuhito FUTATSUKI wrote on Sat, 14 Nov 2020 14:52 +0900:
> +++ subversion/bindings/swig/INSTALL (working copy)
> @@ -141,7 +141,15 @@
> - Make sure that Subversion's ./configure script sees your installed SWIG!
> + If you are using the distribution tarball and you want to use the language
> + bindings C source files shipped with it, you might need to pass the
> + --without-swig option to configure script to avoid detecting and checking
> + SWIG on your system. A Makefile generated by configure will prevent
> + building the language bindings if the configure script detect unsuitable
> + version of SWIG.

I don't dispute the accuracy of this paragraph, but I think this API
isn't autotools-idiomatic.

Generally, I'd expect --without-foo to short-circuit the probe for foo
and assume foo isn't found; i.e., if my system has an unsuitable
version of foo, I'd the default behaviour (given neither --with-foo nor
--without-foo) and the behaviour given --without-foo to be identical:
namely, behave as though foo isn't available (even if /usr/bin/foo
exists and is perfectly suitable).

In particular, if my system has an unsuitable version of swig,
I wouldn't expect passing --without-swig to change configure's behaviour.

Cheers,

Daniel
Received on 2020-11-16 18:17:01 CET