build: cautiously allow parallel builds.

Over two years ago, I disabled parallel building with a
.NOTPARALLEL: directive in the Makefile, due to some rules
that depended on order of updates of their prerequisites.

Today the situation is better. It looks like there are just a
few rules that have to be fixed, all in the area of executing
tests.

Let us not enable that by default, though, to protect
downstream users from themselves.  Some operating system
distribution builds optimistically run make -j N on all the
packages, and then have to deal with sporadic breakages when
some of those packages have parallel build bugs. We can
prevent TXR from ever being implicated in such a breakage by
turning off parallel building even if make -j requests it.

There will now be a ./configure --parallelmake option
to allow parallel builds, off by default.  The --maintainer
configuration mode will flip that default; configuring in
maintainer mode will enable parallel builds, unless explicitly
disabled with --no-parallelmake.

* Makefile (.NOTPARALLEL): Conditionally assert, if
parallelmake is false.
(tst/tests/014/dgram-stream.ok,
tst/tests/014/socket-basic.ok): Enforce order between these
two tests. They clash in their use of network ports so cannot
run at the same time.
(test.clean): test.clean must finish executing before tests;
enforce order.

* configure (parallelmake, parallelmake_given): New variables.
(help text): Add parallelmake option to help.
(gen_config_make): Generate the parallelmake make variable
into config.make.
New script section, defaulting parallelmake to y in maintainer
mode.

1 files changed, 19 insertions, 0 deletions

diff --git a/configure b/configure
index 1fbce3b5..b65bbaf5 100755
--- a/configure
+++ b/configure
@@ -141,6 +141,8 @@ bindir='bin'
 datadir='share/txr'
 mandir='share/man'
 maintainer=
+parallelmake=
+parallelmake_given=
 make=
 cross=
 compiler_prefix=
@@ -332,6 +334,13 @@ maintainer [$maintainer]
   the parser.l and parser.y files will have no effect. In maintainer mode,
   also, c90 is used if compiling the code as C.
 
+parallelmake [$parallelmake]
+
+  Boolean. If set to 'y', it specifies that parallel building with make -j
+  is permitted. Otherwise the Makefile asserts no parallelism with
+  a .NOTOPARALLEL: directive. The above maintainer mode also implies
+  parallel building being permited.
+
 prefix [$prefix]
 
   Specifies root directory where the software will ultimately be installed and
@@ -886,6 +895,9 @@ termios_define := $termios_define
 # do we compile in debug support?
 debug_support := $debug_support
 
+# allow parallel make?
+parallelmake := $parallelmake
+
 # EXE suffix
 EXE := $exe
 
@@ -1777,6 +1789,13 @@ if [ $maintainer ] ; then
 fi
 
 #
+# Check for parallel build
+#
+if [ $maintainer ] && ! [ $parallelmake_given ] ; then
+  parallelmake=y
+fi
+
+#
 # sys/wait.h
 #