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

Update build for new GTG method #821

Closed
WalterKolczynski-NOAA opened this issue Jun 1, 2022 · 1 comment
Closed

Update build for new GTG method #821

WalterKolczynski-NOAA opened this issue Jun 1, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@WalterKolczynski-NOAA
Copy link
Contributor

Expected behavior
Those with GTG access should be able to build post with GTG.

Current behavior
UPP does not build with GTG.

Machines affected
All

To Reproduce

  1. Run checkout with -o to include GTG code (authorized users only).
  2. Run build_upp.sh.

Context
UPP recently updated the way in which GTG is built. Instead of the manual copying of source code to overwrite the normal versions, there is a compile option that will include the GTG code (see NOAA-EMC/UPP/pull/435).

Detailed Description
The compile_upp.sh script within UPP now has a -g option to compile with the GTG module.

Possible Implementation
Will need a new option in build_upp.sh and build_all.sh to build to enable an operational code that builds with GTG (similar to what checkout.sh uses).

@WalterKolczynski-NOAA WalterKolczynski-NOAA added the bug Something isn't working label Jun 1, 2022
@WalterKolczynski-NOAA WalterKolczynski-NOAA self-assigned this Jun 1, 2022
@WalterKolczynski-NOAA
Copy link
Contributor Author

OBE. GTG was removed from workflow checkout in PR #1642.

@WalterKolczynski-NOAA WalterKolczynski-NOAA closed this as not planned Won't fix, can't repro, duplicate, stale Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant