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

gsi-ncdiag to authoritative spack repository #494

Closed
aerorahul opened this issue Mar 10, 2023 · 6 comments
Closed

gsi-ncdiag to authoritative spack repository #494

aerorahul opened this issue Mar 10, 2023 · 6 comments
Assignees

Comments

@aerorahul
Copy link

aerorahul commented Mar 10, 2023

gsi-ncdiag is required in the various GSI uses
gsi-ncdiag is in the NOAA-EMC fork of spack, but not in the authoritative spack repo.
https://github.com/NOAA-EMC/spack/blob/jcsda_emc_spack_stack/var/spack/repos/jcsda-emc/packages/gsi-ncdiag/package.py

Can a PR be opened into the authoritative spack repo for gsi-ncdiag so NOAA/NCEP/NCO can fetch and install from the authoritative repo instead of the fork. This significantly speeds up processing the installation request.

Is your feature request related to a problem? Please describe.
No problem. Just improving process for getting software installed on operational platforms.

Describe the solution you'd like
gsi-ncdiag is in the NOAA-EMC fork of spack, but not in the authoritative spack repo.

Additional context
This request of installation of gsi-ncdiag is being held up on WCOSS2
I also opened a parallel issue in the NOAA-EMC fork of spack at JCSDA/spack#239

@ulmononian
Copy link
Collaborator

@aerorahul i'll get a PR in for this

@ulmononian
Copy link
Collaborator

@aerorahul PR in here: spack/spack#35999

@ulmononian
Copy link
Collaborator

@aerorahul PR in here: spack/spack#35999

@aerorahul aforementioned spack PR was merged.

@ulmononian ulmononian self-assigned this Mar 13, 2023
@ulmononian
Copy link
Collaborator

can this be closed @aerorahul @climbfuji?

@climbfuji
Copy link
Collaborator

Yes for me.

@aerorahul
Copy link
Author

Thank you @ulmononian for opening this PR and following up.

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

No branches or pull requests

3 participants