Re: PATCH: configure-time setting of MAXJOB revisited

Re: PATCH: configure-time setting of MAXJOB revisited

[Sven Wischnowsky]

jarausch@igpm.rwth-aachen.de wrote:

> Hi,
> current cvs (14.4.2000  7:00 GMT) fails due to
> gcc -c -I.  -DHAVE_CONFIG_H -O3 -fstrict-aliasing  -o builtin.o builtin.c
> In file included from zsh.mdh:32,
>                  from builtin.c:30:
> jobs.epro:8: `MAXJOB' undeclared here (not in a function)
> builtin.c: In function `checkjobs':
> builtin.c:3159: `MAXJOB' undeclared (first use in this function)
> 
> What's gone wrong?

Clint just made MAXJOB configurable, you need to re-run autoconf.

Bye
 Sven


--
Sven Wischnowsky                         wischnow@informatik.hu-berlin.de

Re: PATCH: configure-time setting of MAXJOB revisited

[jarausch]

Hi,
current cvs (14.4.2000  7:00 GMT) fails due to
gcc -c -I.  -DHAVE_CONFIG_H -O3 -fstrict-aliasing  -o builtin.o builtin.c
In file included from zsh.mdh:32,
                 from builtin.c:30:
jobs.epro:8: `MAXJOB' undeclared here (not in a function)
builtin.c: In function `checkjobs':
builtin.c:3159: `MAXJOB' undeclared (first use in this function)

What's gone wrong?

Thanks,
Helmut.

-- 
Helmut Jarausch
Lehrstuhl fuer Numerische Mathematik
Institute of Technology, RWTH Aachen
D 52056 Aachen, Germany

PATCH: configure-time setting of MAXJOB revisited

[Clint Adams]

This returns the previous behavior as default and allows more flexibility.

Index: configure.in
===================================================================
RCS file: /cvsroot/zsh/zsh/configure.in,v
retrieving revision 1.4
diff -u -r1.4 configure.in
--- configure.in	2000/04/13 22:25:04	1.4
+++ configure.in	2000/04/13 23:05:07
@@ -498,27 +498,6 @@
  
 AC_CHECK_HEADER([sys/ptem.h], AC_DEFINE(WINSIZE_IN_PTEM))
 
-dnl Try to get a sane value for MAXJOB
-linux_tasks_defines_mtpu=no
-AC_MSG_CHECKING(what to set MAXJOB to)
-if test $ac_cv_header_linux_tasks_h = yes; then
-AC_EGREP_CPP(yes,
-[#include <linux/tasks.h>
- #ifdef MAX_TASKS_PER_USER
- yes
- #endif
-], linux_tasks_defines_mtpu=yes)
-fi
-
-if test $linux_tasks_defines_mtpu = yes; then
-AC_DEFINE(MAXJOB, MAX_TASKS_PER_USER)
-AC_DEFINE(NEED_LINUX_TASKS_H)
-AC_MSG_RESULT(${msg}MAX_TASKS_PER_USER)
-else
-AC_DEFINE(MAXJOB, 50)
-AC_MSG_RESULT(${msg}50)
-fi
-
 dnl -------------------
 dnl CHECK FOR LIBRARIES
 dnl -------------------
@@ -1371,6 +1350,46 @@
       AC_DEFINE(GETPWNAM_FAKED)
     fi
 fi
+
+AC_MSG_CHECKING(what to set MAXJOB to)
+dnl Do you want to alter the maximum job table size?
+undefine([max_jobtable_size])dnl
+AC_ARG_ENABLE(max-jobtable-size,
+[  --enable-max-jobtable-size=MAX   Limit job table size to MAX],
+
+[if test x$enableval = xyes; then
+
+  if test $ac_cv_header_linux_tasks_h = yes; then
+   AC_EGREP_CPP(yes,
+   [#include <linux/tasks.h>
+    #ifdef MAX_TASKS_PER_USER
+    yes
+    #endif
+   ],
+   maxj=max)
+
+   if test x$maxj = xmax; then
+    AC_DEFINE(MAXJOB, MAX_TASKS_PER_USER)
+    AC_DEFINE(NEED_LINUX_TASKS_H)
+    AC_MSG_RESULT(${msg}MAX_TASKS_PER_USER)
+   else
+    AC_DEFINE(MAXJOB, 256)
+    AC_MSG_RESULT(${msg}256)
+   fi
+  fi
+
+ elif test x$enableval = xno; then
+ AC_DEFINE(MAXJOB,512)
+ AC_MSG_RESULT(${msg}512)
+ else
+ AC_DEFINE_UNQUOTED(MAXJOB,$enableval)
+ AC_MSG_RESULT(${msg}${enableval}) 
+fi],
+[
+AC_DEFINE(MAXJOB, 50)
+ AC_MSG_RESULT(${msg}50) 
+])
+
 
 dnl ---------------
 dnl dynamic loading