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 , 7 years ago
comment:2 by , 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.
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