bugGNU Octave - Bugs: bug #51904, SUNDIALS configuration test for...

 
 

bug #51904: SUNDIALS configuration test for sizeof(realtype) is not cross-compile-safe

Submitter:  Mike Miller <mtmiller>
Submitted:  Thu 31 Aug 2017 04:50:41 PM UTC
   
 
Category:  Configuration and Build System Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Fixed Assigned to:  mtmiller
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sat 02 Sep 2017 07:05:14 PM UTC, comment #4: 

@Mike: I see you have fixed it in http://hg.octave.org/mxe-octave/rev/8b731cde3aaa. Thanks.

Avinoam Kalma <avinoam>
Group Member
Fri 01 Sep 2017 07:11:59 PM UTC, comment #3: 

Please file a new bug about that. The mxe-octave/src/sundials-ida.mk file will have to be updated to build against the SuiteSparse KLU library for that to work.

Mike Miller <mtmiller>
Group Member
Fri 01 Sep 2017 08:36:57 AM UTC, comment #2: 

Thanks. Now the cross compilation works, but...


checking ida/ida.h usability... yes
checking ida/ida.h presence... yes
checking for ida/ida.h... yes
checking for IDAInit in -lsundials_ida... yes
checking for ida/ida.h... (cached) yes
checking ida.h usability... no
checking ida.h presence... no
checking for ida.h... no
checking whether SUNDIALS IDA is configured with double precision realtype... y
es
checking ida/ida_klu.h usability... no
checking ida/ida_klu.h presence... no
checking for ida/ida_klu.h... no
checking ida_klu.h usability... no
checking ida_klu.h presence... no
checking for ida_klu.h... no
checking whether SUNDIALS IDA is configured with IDAKLU enabled... no
configure: WARNING: SUNDIALS IDA library not configured with IDAKLU, ode15i and
 ode15s will not support the sparse Jacobian feature


Seems that  ida_klu.h is missing, so ode15s will not have a full functionality.

Avinoam Kalma <avinoam>
Group Member
Thu 31 Aug 2017 09:27:05 PM UTC, comment #1: 

I fixed this on the default branch

https://hg.savannah.gnu.org/hgweb/octave/rev/eb6602171d59

and tested with my local mxe-octave setup, works for me.

Mike Miller <mtmiller>
Group Member
Thu 31 Aug 2017 04:50:41 PM UTC, original submission:  

The configure test for whether SUNDIALS is built for double-precision floating point values is not cross-compile-safe. However, since it only relies on compile-time constants, it should be possible to convert it to a AC_COMPILE_IFELSE instead of a AC_RUN_IFELSE, which will make it cross-compile-safe. In other words there is no reason to use a runtime assert in order to test something that can be determined at compile time. I'll take a look at fixing this test.

Mike Miller <mtmiller>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by avinoam (Posted a comment)
  • -email is unavailable- added by avinoam
  • -email is unavailable- added by mtmiller (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-31 mtmiller StatusNone Fixed
        Open/ClosedOpen Closed
    2017-08-31 avinoam Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code