Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update OMSimulator #8457

Merged
merged 2 commits into from Feb 22, 2022
Merged

Update OMSimulator #8457

merged 2 commits into from Feb 22, 2022

Conversation

lochel
Copy link
Member

@lochel lochel commented Jan 25, 2022

No description provided.

@lochel lochel added the COMP/OMSimulator Issue and pull request related to OMSimulator label Jan 25, 2022
@lochel lochel enabled auto-merge (squash) January 25, 2022 10:37
@lochel
Copy link
Member Author

lochel commented Feb 2, 2022

@adrpo after investigating this a bit with @mahge we concluded that the cmake version causes the problem. We should at least upgrade to 3.10.

@lochel
Copy link
Member Author

lochel commented Feb 22, 2022

@adrpo @mahge @sjoelund @arun3688 I hope that we can update OMSimulator again. I removed xerces which is very unfortunat, but apparently neccessary if we want to ship a up-to-date OMSimulator version with OpenModelica.

@lochel lochel merged commit 5e4e326 into OpenModelica:master Feb 22, 2022
@lochel lochel deleted the oms branch February 22, 2022 21:17
@mahge
Copy link
Contributor

mahge commented Feb 23, 2022

I think the agreement was to update the images for the CI after we branch out for release. It was planned, at the latest, by the end of February. So you can bring xerces back very soon. Hopefully we have not missed on anything significant for the upcoming release due to removing xerces.

@lochel
Copy link
Member Author

lochel commented Feb 23, 2022

End of February sounds good. I have already prepared the changes to re-introduce xerces and am just waiting for the new images to be in place. 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
COMP/OMSimulator Issue and pull request related to OMSimulator
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants