Differential D49036

[phb][Uses]: Add missing Uses=mpi
ClosedPublic
Actions

Authored by fernape on Feb 17 2025, 7:25 PM.

Details

Reviewers

thierry

Commits

R9:198de88111ef: [phb][Uses]: Add missing Uses=mpi

Diff Detail

Repository

R9 FreeBSD doc repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 62489
Build 59373: arc lint + arc unit

Event Timeline

fernape requested review of this revision.Feb 17 2025, 7:25 PM

fernape created this revision.

Harbormaster completed remote builds in B62489: Diff 151122.Feb 17 2025, 7:25 PM

fernape added a reviewer: thierry.Feb 17 2025, 7:26 PM

Seems fine, but it would be better to write MPI_HOME set to $LOCALBASE for MPICH, even if MPICH is the default.

documentation/content/en/books/porters-handbook/uses/_index.adoc
1551–1552	Remark: MPI_HOME is set to $LOCALBASE only for MPICH.

Addressing feedback.

Harbormaster completed remote builds in B62505: Diff 151156.Feb 18 2025, 6:51 PM

fernape marked an inline comment as done.Feb 18 2025, 6:51 PM

Thanks!

This revision is now accepted and ready to land.Feb 18 2025, 6:58 PM

Thanks for the review!

Closed by commit R9:198de88111ef: [phb][Uses]: Add missing Uses=mpi (authored by fernape). · Explain WhyFeb 21 2025, 4:24 PM

This revision was automatically updated to reflect the committed changes.

fernape added a commit: R9:198de88111ef: [phb][Uses]: Add missing Uses=mpi.

Revision Contents
Changeset List

Path

Size

documentation/

content/

en/

books/

porters-handbook/

uses/

_index.adoc

42 lines

Diff 151122

View Options

documentation/content/en/books/porters-handbook/uses/_index.adoc

[phb][Uses]: Add missing Uses=mpiClosedPublicActions