Here are new patches. These use SAGE_NUM_THREADS
if it is set, and otherwise try to extract a number from MAKE
. (My method for doing this is probably not ideal, but the options This is done in sage-env. Running sage -b
should use this setting now, also.
I don't know how to get the number of threads from
make -j16 ptestlong
so I removed that from the "to do" list in the ticket description.
In the file sage-ptest, I removed the "FIXME" comment in
try:
# FIXME: Nice, but <NUMTHREADS> should immediately follow '-tp' etc.,
# i.e., be the next argument. We might have file or directory
# names that properly convert to an int...
numthreads = int(argv[1])
infiles = argv[2:]
except ValueError: # can't convert first arg to an integer: arg was probably omitted
numthreads = 1
The script sage-ptest doesn't get a "tp" argument; it is instead called by sage-sage, and the way it is called, the first argument to sage-ptest is precisely what ever came after "-tp". So I don't think anything needs fixing. If we ever rewrite sage-sage (#21) to properly parse arguments, we can make sure that "-tp" has a default numerical argument of zero.