Opened 7 years ago

Last modified 7 years ago

#4587 new defect

Running Hudson PR with testsuite hash and force lastest generates invalid OpenModelica glue project

Reported by: adrpo Owned by: sjoelund.se
Priority: high Milestone: Future
Component: Build Environment Version:
Keywords: Cc:

Description

Whenever libraries change we might get issues with the testsuite.
I update the testsuite then I run the Hudson PR job with testsuite hash, force latest and PUSH and that somehow updates also the OMCompiler to the hash that is ONLY present on asap.openmodelica.org repository!
I guess we should check if the local hash is also present in the github repo.

Change History (2)

comment:1 Changed 7 years ago by Matthis Thorade <m.thorade@…>

Sometimes when I change multiple things in the BuildingSystems library in one single commit, then sometimes not all of the changes are reflected in the results.

Could that be related? If not, it should probably better go in a separate issue!?

Here is an example:
This pull request
https://github.com/UdK-VPT/BuildingSystems/pull/96
added some new models, e.g. this one:
https://libraries.openmodelica.org/branches/master/BuildingSystems/files/BuildingSystems_BuildingSystems.Fluid.Examples.Performance.PressureDrop.ParallelFlow.err
but the same PR also changed an existing model to fix an error which is somehow still present here:
https://libraries.openmodelica.org/branches/master/BuildingSystems/files/BuildingSystems_BuildingSystems.Fluid.FMI.Conversion.Validation.InletToAirDryAirCO2.err

comment:2 Changed 7 years ago by Matthis Thorade <m.thorade@…>

Sorry, I should have checked my PR before writing, for some reason the change I wanted to include was not included, so what I wrote in the previous is not correct.

Note: See TracTickets for help on using tickets.