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: Adrian Pop Owned by: Martin Sjölund
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 by Matthis Thorade <m.thorade@…>, 7 years ago

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 by Matthis Thorade <m.thorade@…>, 7 years ago

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.