llnl / llnl/smith

Investigate hdf5+mpi

Open
#1,088 1 comment 0 reactions 1 assignee View on GitHub

@chapman39 is already working on this.

Since Feb 29, 2024.

build
Dominant language
C++
Stars
245
Forks
36
Avg merge
8d 23h
Merged PRs (30d)
4

Description

PETSc cannot build with `+hdf5` due to requiring it built with mpi. Serac set `hdf5~mpi` a long time ago, so it may not be an issue anymore. This would in turn allow LiDO to build `petsc+hdf5` as well.

```diff
- depends_on("hdf5@1.8.21:+hl~mpi")
+ depends_on("hdf5@1.8.21:+hl+mpi")
```

https://github.com/LLNL/serac/blob/develop/scripts/spack/packages/serac/package.py#L114-L117

https://github.com/spack/spack/blob/develop/var/spack/repos/builtin/packages/petsc/package.py#L189

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.