Skip to content

TSMP-PDAF Makefiles: Fixes for ensuring safe parallel compilation - #87

Open
jjokella wants to merge 4 commits into
tsmp-pdaf-patchedfrom
tsmp-pdaf-patched-parallel-build
Open

jjokella wants to merge 4 commits into
tsmp-pdaf-patchedfrom
tsmp-pdaf-patched-parallel-build

Conversation

@jjokella

@jjokella jjokella commented Oct 8, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

jjokella added a commit to HPSCTerrSys/TSMP2 that referenced this pull request Oct 19, 2026
@kvrigor  I am starting this as PDAF-related change to build on #89 

- pdaf-compilations use `$(MAKE)` instead of `make`
- pdaf-compilations use `-j ${PARALLEL_JOBS}`
- split `clean` target to avoid race condition

- pdaf itself has been updated with some fixes aiming to ensure safe
parallel compilation, which has so far worked HPSCTerrSys/pdaf#87

for testing / may be reverted
- relax CMake version condition
- add some `make` -> `$(MAKE)` replacements in eCLM externals (see eCLM
branch `master-parallel-build`)

## Compilation times eCLM-PDAF
Significant reduction that would be nice to have in the `master`
- ca. 11 minutes w/o parallel PDAF-compilation (but eCLM parallel as far
as I understand)
- ca. 8 minutes with parallel PDAF-compilation (`-j 16`)
Trying to avoid error in parallel build:

```
Fatal Error: Cannot open module file ‘mod_clm_statistics.mod’ for
reading at (1): No such file or directory
```

This branch has not been deployed

No deployments
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