Skip to content

Fix OMSimulator submodule update when directory layout changes#287

Merged
AnHeuermann merged 1 commit into
OpenModelica:masterfrom
AnHeuermann:fix-oms-install
Jun 12, 2026
Merged

Fix OMSimulator submodule update when directory layout changes#287
AnHeuermann merged 1 commit into
OpenModelica:masterfrom
AnHeuermann:fix-oms-install

Conversation

@AnHeuermann

Copy link
Copy Markdown
Member

Issue

I changed the layout of OMSimulator submodules in OpenModelica/OMSimulator-3rdParty#98. This broke the library testing CI:

https://test.openmodelica.org/jenkins/blue/organizations/jenkins/LibraryTesting%2FLibraryTest/detail/LibraryTest/11099/pipeline/39/

CMake Error at 3rdParty/CMakeLists.txt:146 (add_subdirectory):
  The source directory

    /var/lib/jenkins/ws/OpenModelicaLibraryTestingWork/OMSimulator/3rdParty/libzip

  does not contain a CMakeLists.txt file.


CMake Error at 3rdParty/CMakeLists.txt:147 (add_library):
  add_library cannot create ALIAS target "oms::3rd::libzip" because target
  "zip" does not already exist.

Changes

Clean the working tree (including nested git repos) before running submodule update, so that directories that were replaced by submodules (or vice versa) don't block initialization.

Clean the working tree (including nested git repos) before running
submodule update, so that directories that were replaced by submodules
(or vice versa) don't block initialization.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@AnHeuermann AnHeuermann self-assigned this Jun 12, 2026
@AnHeuermann AnHeuermann enabled auto-merge (squash) June 12, 2026 10:31
@AnHeuermann AnHeuermann merged commit d357aff into OpenModelica:master Jun 12, 2026
10 of 11 checks passed
@AnHeuermann AnHeuermann deleted the fix-oms-install branch June 12, 2026 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant