Re: new ODE functions

[Bill Greene]

I ran into a small problem while trying to build this so thought

I would pass it along.

My version of Sundials/IDA happened to be configured with the Lapack

dense solvers as one of the options. So while running configure, the step

OCTAVE_CHECK_LIB(sundials_ida, SUNDIALS_IDA,

failed because the link step couldn't find those Lapack functions.

I tried replacing 

LIBS="$KLU_LDFLAGS $KLU_LIBS $LIBS"

with

LIBS="$KLU_LDFLAGS $KLU_LIBS $LAPACK_LIBS $BLAS_LIBS $LIBS"

but that didn't work (apparently) because $LAPACK_LIBS $BLAS_LIBS

weren't yet defined.

Bill

On Fri, Dec 16, 2016 at 1:18 PM, John W. Eaton <address@hidden> wrote:

On 12/16/2016 12:40 PM, Rik wrote:

On 29 Nov 2016, at 15:56, c. <address@hidden> wrote:

I will now start working more thoroughly on improving the coding style,
but I would really appreciate if someone could look in detail at this
code, especially the autoconf part, and suggest changes/improvements.

I made a number of style fixes for the C++ code.

The Debian packages for Sundials do not have KLU enabled.  I installed my own local copy with it enabled, but it might be nice if we could disable whatever features KLU provides if it is not found while still allowing the rest to build.  I could work on that if you agree.

jwe