ompi icon indicating copy to clipboard operation
ompi copied to clipboard

Remove script wrappers in v6.0.x

Open jsquyres opened this issue 1 year ago • 3 comments

Hey @hppritcha is there a reason that we still need to keep carrying the script wrappers to v6.0.x and beyond?

They are not installed by default, but periodically someone does try and sometimes they do cause work (e.g., https://github.com/open-mpi/ompi/issues/12609). Does DOE still use the script wrappers for cross-compilation, or other reasons?

jsquyres avatar Jun 17 '24 19:06 jsquyres

FWIW, we are exploring the script wrappers for cross compilation in Conda-forge, which prompted #12609. This is motivated largely by the fact that cross compilation for mpich has been far simpler for us because mpich compiler wrappers are scripts instead of binaries. We do have cross compilation working with openmpi, so we will probably skip migrating to the script wrappers if they are going to go away anyway.

minrk avatar Jun 18 '24 09:06 minrk

@minrk Ok, great. Please keep us informed; we can keep the script wrappers (especially if they're working ok) -- my main motivation in filing this issue was that we probably haven't tested these scripts in years. If they're still working fine, that's good!

jsquyres avatar Jun 18 '24 14:06 jsquyres

pinged some folks about this.

hppritcha avatar Jun 19 '24 15:06 hppritcha