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

Theia build broken #4

Closed
jmccreight opened this issue Mar 12, 2018 · 4 comments
Closed

Theia build broken #4

jmccreight opened this issue Mar 12, 2018 · 4 comments

Comments

@jmccreight
Copy link
Collaborator

From: Rocky Dunlap

Expected Behavior

That it configures and compiles under intel.

Current Behavior

git clone [email protected]:NCAR/wrf_hydro_nwm.git
cd trunk/NDHMS
./configure

Please select from following supported linux compilers
using either the number or key (not case sensitive):

Number    Key  Description
--------------------------------------------
     1    pgi  PGI parallel
     2  gfort  gfortran parallel
     3  ifort  intel parallel (incl. Theia)
     4   luna  ifort parallel (WCOSS Luna)
     0   exit  exit

Enter selection: 3

cp: cannot stat ‘arc/macros.mpp.ifort.theia’: No such file or directory
Configured: ifort on Theia
No compiler selected. Exiting

Possible Solution

cd trunk/NDHMS/arc
cp macros.theia macros.mpp.ifort.theia
cd ../
./configure ifort

Steps to Reproduce (for bugs)

See above, specific to theia.

Your Environment

  • Version of the code used:
  • Operating System and version:
  • Compiler and version:
  • Other relevant information:
@jmccreight
Copy link
Collaborator Author

I think the proposed solution is desirable for the repo because then the macro file name reflects "mpp" and "ifort" as do the other macro files. If this works I'll commit that change on the public repo.

@rsdunlapiv
Copy link
Contributor

The possible solution above worked for me on Theia. Do you plan to check in the renamed macro file so that the user does not need to do this step manually?

@jmccreight
Copy link
Collaborator Author

Yes! that's the plan.

@jmccreight
Copy link
Collaborator Author

and thanks for testing that.

jmccreight referenced this issue in jmccreight/wrf_hydro_nwm_public Nov 13, 2018
jmccreight referenced this issue in jmccreight/wrf_hydro_nwm_public Nov 13, 2018
laurareads pushed a commit to laurareads/wrf_hydro_nwm_public that referenced this issue Apr 3, 2019
New modis & snotel vignettes. Fix to geospatial.
@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
None yet
Projects
None yet
Development

No branches or pull requests

3 participants