Skip to content

Fix master - #239

Merged
sjoelund merged 2 commits into
masterfrom
fix-master
Sep 23, 2026
Merged

sjoelund merged 2 commits into
masterfrom
fix-master

Conversation

@sjoelund

Copy link
Copy Markdown
Member

Related Issues

Purpose

Approach

`c_example` and `dsimple_test` are the only MUMPS targets that need the
complete MPI symbol set, and nothing uses them. They fail to link on
el10:

    undefined reference to `mpi_wtime_'
    undefined reference to `dmumps_ana_driver_'

Drop them from `all` here rather than in the MUMPS CMakeLists, so that
the fix survives moo becoming a submodule.

Assisted-by: Claude Opus 5 (1M context)
#237 turned the vendored ColPack/ into a submodule at the same path. A
checkout that goes back to a commit with the vendored files then fails:
the submodule's work tree stays behind as untracked files, and git will
not overwrite them

    error: The following untracked working tree files would be
    overwritten by checkout:
        ColPack/.gitignore

Jenkins' `git clean -fdx` leaves it alone (it is a nested repository),
so one build of a PR on the new 3rdParty broke every later build on
that agent of a PR still on the old one.

The submodule keeps its name, so an existing clone reuses
.git/modules/ColPack. Not `colpack`: on the macOS and Windows file
systems that is still the same directory.

Assisted-by: Claude Opus 5.5
@sjoelund
sjoelund merged commit 7dae8c3 into master Sep 23, 2026
6 checks passed
@sjoelund
sjoelund deleted the fix-master branch September 23, 2026 08:15
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