Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove script wrappers in v6.0.x #12625

Open
jsquyres opened this issue Jun 17, 2024 · 3 comments
Open

Remove script wrappers in v6.0.x #12625

jsquyres opened this issue Jun 17, 2024 · 3 comments

Comments

@jsquyres
Copy link
Member

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., #12609). Does DOE still use the script wrappers for cross-compilation, or other reasons?

@minrk
Copy link
Contributor

minrk commented Jun 18, 2024

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.

@jsquyres
Copy link
Member Author

@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!

@hppritcha
Copy link
Member

pinged some folks about this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants