Opened 7 years ago

Closed 7 years ago

#4676 closed defect (fixed)

Hudson scripts do not capture assert() output during translation

Reported by: Francesco Casella Owned by: Martin Sjölund
Priority: critical Milestone: 1.13.0
Component: New Instantiation Version:
Keywords: Cc:

Description

See ticket:4660#comment:8

This should be fixed for the Hudson logs reporting failures of the new front end to be fully informative

Change History (6)

comment:1 by Martin Sjölund, 7 years ago

I believe it should be fixed in the NF by not printing to stdout. We have spent a lot of effort to remove print() statements and the like from the compiler so that you can query the information in scripting and OMPython..

comment:2 by Francesco Casella, 7 years ago

Owner: changed from Martin Sjölund to Per Östlund
Status: newassigned

comment:3 by Francesco Casella, 7 years ago

Component: Testing FrameworkNew Instantiation

comment:4 by Martin Sjölund, 7 years ago

Owner: changed from Per Östlund to Martin Sjölund
Status: assignedaccepted

comment:5 by Martin Sjölund, 7 years ago

I ran the following code and fixed some compilation errors for PR2080:

sed -i "s/ assert[(]\([^;]*\)[)]/ Error.assertion(\1, sourceInfo())/" Compiler/NFFrontEnd/*.mo

comment:6 by Francesco Casella, 7 years ago

Resolution: fixed
Status: acceptedclosed

The new Hudson logs in https://libraries.openmodelica.org/branches/newInst/Modelica_3.2.2/Modelica_3.2.2.html seem to be a lot more informative.

I'll close the ticket unless it is found that there are still unreported errors in those logs.

Note: See TracTickets for help on using tickets.