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.

Sign up for GitHub

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

Jump to bottom

Clarify build procedures #818

Closed
GeorgeGayno-NOAA opened this issue Apr 26, 2023 · 0 comments · Fixed by #819
Closed

Clarify build procedures #818

GeorgeGayno-NOAA opened this issue Apr 26, 2023 · 0 comments · Fixed by #819
Assignees
@KateFriedman-NOAA @GeorgeGayno-NOAA
Labels
build

Comments

@GeorgeGayno-NOAA
Copy link
Collaborator

This repository uses submodules and users often forget to do a recursive clone or a git submodule update.

Improve the instructions or build script to prevent this.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Update the build script to do a submodule update
2544b92 
if necessary before building.

Update build instructions in README.md.

Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Clarify README file.
329ccb9 
Fixes ufs-community#818.
@GeorgeGayno-NOAA GeorgeGayno-NOAA mentioned this issue Apr 26, 2023
Merged
3 tasks
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Update rt.sh script to remove recursive clone which
4694b22 
can hang. Submodules are now handled by the build script.

Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Improve documentation per Kate's recommendation.
7279b73 
Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Minor format change to README.
6943c3b 
Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Remove redundant step from README.
072f732 
Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue Apr 26, 2023
@GeorgeGayno-NOAA
Minor format change to build script.
e09a220 
Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit to GeorgeGayno-NOAA/UFS_UTILS that referenced this issue May 1, 2023
@GeorgeGayno-NOAA
Merge branch 'develop' into feature/submod
8c177a3 
Fixes ufs-community#818.
GeorgeGayno-NOAA added a commit that referenced this issue May 1, 2023
@GeorgeGayno-NOAA
Improve repository build script (#819)
b4900c4 
Update the build_all.sh script to do a "submodule update" if
necessary before building.

Update build instructions in README.md.

Update the rt.sh script to remove the recursive clone as submodules
are now handled by the build script.

Fixes #818.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KateFriedman-NOAA KateFriedman-NOAA

@GeorgeGayno-NOAA GeorgeGayno-NOAA

Labels
build
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve repository build GeorgeGayno-NOAA/UFS_UTILS
2 participants
@KateFriedman-NOAA @GeorgeGayno-NOAA