Skip to content

Update WRF-3.8.0_netCDF-Fortran_separate_path.patch#3702

Merged
boegel merged 1 commit intoeasybuilders:developfrom
andreas-h:patch-1
Nov 11, 2016
Merged

Update WRF-3.8.0_netCDF-Fortran_separate_path.patch#3702
boegel merged 1 commit intoeasybuilders:developfrom
andreas-h:patch-1

Conversation

@andreas-h
Copy link
Copy Markdown
Contributor

I believe this is a typo preventing the NETCDFFPATH variable to be set correctly.

I believe this is a typo preventing the `NETCDFFPATH` variable to be set correctly.
@boegel boegel modified the milestones: 3.1, 3.0 Nov 10, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 10, 2016

@andreas-h hmm, sure looks like a typo indeed, but I'm a bit puzzled as to how we missed this up until now?

What part of WRF is broken without this fix?

@andreas-h
Copy link
Copy Markdown
Contributor Author

It's indeed very strange. WRF3.8.1 builds fine without this fix on my installation, but during my ongoing work to compile WRF with Chemistry enabled (some extra environment variables set), some Makefile of the chemistry code started complaining that it couldn't find the netCDF-Fortran library ... I'm awfully behind on this project; hopefully I can continue end of next week (and then also answer your question about the other PR).

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 10, 2016

@andreas-h OK, thanks for the info. I'll make sure this doesn't break the WRF 3.8 installation on my end, but I don't expect it to. If it's fine, I'll merge this ASAP, thanks!

@boegel
Copy link
Copy Markdown
Member

boegel commented Nov 11, 2016

tested, works, so going in, thanks @andreas-h!

@boegel boegel merged commit 75b51d7 into easybuilders:develop Nov 11, 2016
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

Successfully merging this pull request may close these issues.

2 participants