Opened 7 years ago

Closed 22 months ago

#2038 closed defect (fixed)

Add Casadi to default build

Reported by: Christian Andersson Owned by: Tove Bergdahl
Priority: major Milestone: Pending
Component: Build Version: trunk
Keywords: Cc:

Description

As of ticket:1900 the casadi test are run by default when invoking jm_tests. However, CasADi is not currently built by default resulting in lots of errors when running the test suite. Maybe it should be built by default in order to be consistent?

Change History (12)

comment:1 Changed 7 years ago by Fredrik Magnusson

Just like the tests, it should only be built automatically on Windows and Linux.

comment:2 Changed 7 years ago by jakesson

Owner: set to Jesper Mattsson
Status: newassigned

comment:3 Changed 7 years ago by jakesson

Milestone: 1.8.x1.9.x

comment:4 Changed 7 years ago by jakesson

Milestone: 1.9.x

CasADi still does not compile on Mac out of the box.

comment:5 Changed 6 years ago by Tove Bergdahl

Does CasADi still not compile on Mac? Ticket needs to be updated with current status.

comment:6 in reply to:  5 Changed 6 years ago by Fredrik Magnusson

Replying to tove:

Does CasADi still not compile on Mac? Ticket needs to be updated with current status.

I do not know the details, but I believe that the CasADi developers consider the Mac issue to be resolved, whereas we do not. I also believe that no recent effort has been made from our side to consider the current possibilities. See https://github.com/casadi/casadi/issues/201

comment:7 Changed 6 years ago by Jesper Mattsson

Needs discussion.

comment:8 Changed 6 years ago by Jesper Mattsson

Milestone: 1.12.x
Owner: changed from Jesper Mattsson to Tove Bergdahl

Change makefiles so that:

  • make base
    • Builds what make currently does, i.e. all except CasADi
  • make casadi
    • Builds CasADi
  • make
    • Builds all
    • Equivalent to make base casadi
  • make install_base
    • Installs what make install currently does, i.e. all except CasADi
  • make install_casadi
    • Installs CasADi, like it does now (right? or does it build as well?)
  • make install
    • Installs all
    • Equivalent to make install_base install_casadi

And all the install targets should of course depend on their corresponding build target.

comment:9 Changed 6 years ago by Jesper Mattsson

Discuss with toivo concerning effects on his and blennernas' changes.

comment:10 Changed 6 years ago by Tove Bergdahl

Milestone: 1.12.x1.13.x

comment:11 Changed 6 years ago by Tove Bergdahl

Milestone: 1.13.xPending

CasADi can not yet be installed on Mac so there is still need for an independent make target. Move to pending until this has been solved.

comment:12 Changed 22 months ago by Jesper Mattsson

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.