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

Problem with symlink artifact produced by compile_offline_NoahMP.sh #17

Closed
tjmills opened this issue Apr 4, 2018 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@tjmills
Copy link

tjmills commented Apr 4, 2018

compile_offline_NoahMP.sh produces two symlinks in the trunk/NDHMS source directory when run: LandModel and LandModel_cpl.

If the directory is copied without careful attention to these symlinks, they are copy as the target folders. The compile_offline_NoahMP.sh script attempts to delete these files when re-run, but if they exist as directories the script fails.

This may affect compile_offline_Noah.sh as well.

FIX

https://github.com/NCAR/wrf_hydro_nwm_public/blob/master/trunk/NDHMS/compile_offline_NoahMP.sh#L39

should be changed to rm -rf LandModel LandModel_cpl

ADDITIONAL ISSUES
Under no circumstances should artifacts be produced in the source directory. This issue is only one example of the dangers of polluting the source directory with artifacts. A new issue will be opened to address this

@tjmills tjmills added the bug Something isn't working label Apr 4, 2018
jmccreight pushed a commit to jmccreight/wrf_hydro_nwm_public that referenced this issue Nov 13, 2018
@rcabell
Copy link
Collaborator

rcabell commented May 4, 2022

For artifact-free, the CMake build systems allows out-of-tree builds.

@rcabell
Copy link
Collaborator

rcabell commented May 4, 2022

Closing due to corner-case, lack of recent activity, and upcoming focus on CMake

@rcabell rcabell closed this as completed May 4, 2022
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

2 participants