[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #50003] Build failure with IDA 2.8.1
From: |
Francesco Faccio |
Subject: |
[Octave-bug-tracker] [bug #50003] Build failure with IDA 2.8.1 |
Date: |
Sun, 8 Jan 2017 17:32:05 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:47.0) Gecko/20100101 Firefox/47.0 |
Follow-up Comment #6, bug #50003 (project octave):
@jwe
I think that in your changeset in configure.ac it should be:
diff -r 63a12df71848 configure.ac
--- a/configure.ac Sat Jan 07 20:12:16 2017 -0500
+++ b/configure.ac Sun Jan 08 18:20:14 2017 +0100
@@ -2156,7 +2156,7 @@
### Check for Sundials IDA library and header.
save_LIBS="$LIBS"
-LIBS="$SUNDIALS_NVECSERIAL_LIBS $BLAS_LIBS $FLIBS $LIBS"
+LIBS="$SUNDIALS_NVECSERIAL_LIBS $KLU_LIBS $LIBS"
OCTAVE_CHECK_LIB(sundials_ida, [Sundials IDA],
[Sundials IDA library not found. Solvers ode15i and ode15s will be
disabled.],
[ida/ida.h ida.h], [IDAInit],
As Carlo pointed out, we should require the latest version of IDA.
I just noticed that in IDA 2.9.0 the function IDAKLU accepts 4 input
arguments, while in IDA 2.8.1 it accepts only 3 inputs, so the macro
OCTAVE_CHECK_SUNDIALS_IDAKLU you wrote is already checking that the user is
working with the latest version of IDA.
This means that if we choose to disable ode15{i,s} when SUNDIALS has not been
configured with KLU enabled, then it is enough to use that macro.
If we choose to keep ode15{i,s} available without KLU (but I agree with Carlo
on the fact that the user should receive at least a warning in case she uses a
sparse Jacobian), then the user must have at least version 2.7.0 of IDA, but I
suggest to require the latest version anyway. The presence of the function
SparseSetMatToZero ensures that the user is working with IDA 2.9.0.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?50003>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/