Fix WRFDA build failure caused by duplicate da_residual.inc - #2367
Open
the-1000th-summer wants to merge 2 commits into
Open
Fix WRFDA build failure caused by duplicate da_residual.inc#2367the-1000th-summer wants to merge 2 commits into
the-1000th-summer wants to merge 2 commits into
Conversation
Collaborator
|
The regression test results: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TYPE: bug fix
KEYWORDS: WRFDA, build, symbolic link, duplicate filename, da_residual.inc
SOURCE: Deng Hao (Sun Yat-sen University)
DESCRIPTION OF CHANGES:
Problem:
The WRFDA source tree contains two files with the same basename:
var/da/da_minimisation/da_residual.incvar/da/da_tools/da_residual.incThe file under
da_minimisationis empty, while the file underda_toolscontains the implementation used byda_tools.f90.During the legacy WRFDA build process, files from the different
var/da/da_*directories are linked into the commonvar/builddirectory. Both source files therefore map to:
var/build/da_residual.incWith GNU
ln, this causes the link-generation step to fail with:ln: will not overwrite just-created './da_residual.inc'The current
da_minimisation.f90usesda_calculate_residual.incand does not appear to use the emptyda_minimisation/da_residual.incfile.Solution:
Removed the unused empty file:
var/da/da_minimisation/da_residual.incThe implementation in
var/da/da_tools/da_residual.incis unchanged.ISSUE:
Fixes #2364
LIST OF MODIFIED FILES:
D var/da/da_minimisation/da_residual.inc
TESTS CONDUCTED:
Reproduced the duplicate-link failure before the change when
compiling WRFDA 4D-Var.
Removed the empty duplicate file, cleaned the source tree,
reconfigured WRFDA, and rebuilt it using:
./clean -a
./configure wrfda
./compile all_wrfvar
The duplicate
da_residual.inclink error no longer occurred,and the WRFDA build completed successfully.
Test environment:
The official Jenkins regression tests were not run locally.
RELEASE NOTE:
Removed an unused empty WRFDA include file that caused a duplicate
filename conflict during the legacy WRFDA build link-generation stage.