Skip to content

v12: Fix for run scripts and spack#817

Merged
sdrabenh merged 3 commits intofeature/sdrabenh/gcm_v12from
feature/v12-scm-setup-fix
Mar 12, 2026
Merged

v12: Fix for run scripts and spack#817
sdrabenh merged 3 commits intofeature/sdrabenh/gcm_v12from
feature/v12-scm-setup-fix

Conversation

@mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Mar 4, 2026

In testing Spack builds of GEOS, it was found that in at least some ways of running with Spack, LD_LIBRARY_PATH (or DYLD_LIBRARY_PATH) does not exist. But our run scripts seem to assume it does.

This PR updates the scripts to be more generic.

@mathomp4 mathomp4 self-assigned this Mar 4, 2026
@mathomp4 mathomp4 requested a review from a team as a code owner March 4, 2026 16:03
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Mar 4, 2026
Copy link
Contributor

@FlorianDeconinck FlorianDeconinck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sdrabenh sdrabenh merged commit 09af5ca into feature/sdrabenh/gcm_v12 Mar 12, 2026
8 of 10 checks passed
@sdrabenh sdrabenh deleted the feature/v12-scm-setup-fix branch March 12, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants