Skip to content

Commit

Permalink
Feature 1778 debug (#1785)
Browse files Browse the repository at this point in the history
* Per #1778, please see #1778 (comment) for details. Basically, when doing development, compile with the -g debug option. Otherwise, remove it by default.

* Per #1778, update stale URL's in the README and configure.ac file. Also, change the default MET version from 8.1 to development.
  • Loading branch information
JohnHalleyGotway committed May 8, 2021
1 parent 6c9b78e commit 50b77c0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 9 deletions.
8 changes: 4 additions & 4 deletions met/README
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ This is the main directory for the Model Evaluation Tools source code release.
================================================================================

For questions, please:
- Refer to the MET User's Guide in the /doc sub-directory.
- Refer to the MET website: http://www.dtcenter.org/met/users
- Refer to the MET User's Guide: https://met.readthedocs.io/en/latest
- Refer to the MET website: http://dtcenter.org/community-code/model-evaluation-tools-met
- Send mail to met_help@ucar.edu.

Dependencies
Expand Down Expand Up @@ -52,7 +52,7 @@ Dependencies

- It is suggested that the following tools be used in conjunction with MET:
- Unified Post-Processor
- http://www.dtcenter.org/wrf-nmm/users
- http://dtcenter.org/community-code/unified-post-processor-upp

NOTE: The required libraries should be compiled with the same set of compilers
to be used in compiling MET.
Expand All @@ -63,7 +63,7 @@ How to compile and run?
For more detailed instructions on building MET and for a list of supported
platforms and compilers, please refer to the MET User's Guide in the doc/
sub-directory, or the MET Online Tutorial:
http://www.dtcenter.org/met/users/support/online_tutorial
http://dtcenter.org/community-code/model-evaluation-tools-met/online-tutorial

- Set the $CXX and $F77 environment variables to specify the C++ and FORTRAN
compilers to be used. If not set, configure will search for compilers and the
Expand Down
13 changes: 8 additions & 5 deletions met/configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# Process this file with autoconf to produce a configure script.

AC_PREREQ([2.69])
AC_INIT([MET], [m4_esyscmd_s(echo ${MET_BUILD_VERSION:-8.1})], [met_help@ucar.edu], [], [http://www.dtcenter.org/met/users])
AC_INIT([MET], [m4_esyscmd_s(echo ${MET_BUILD_VERSION:-development})], [met_help@ucar.edu], [], [http://dtcenter.org/community-code/model-evaluation-tools-met])
AC_CONFIG_SRCDIR([src/tools/tc_utils/tc_dland/tc_dland.cc])
AC_CONFIG_HEADERS([config.h])

Expand Down Expand Up @@ -1041,11 +1041,14 @@ CPPFLAGS=$CPPFLAGS' -DMET_BASE="\"$(pkgdatadir)\""'

AC_SUBST(FC_LIBS, [-lgfortran])

# The CXXFLAGS default to "-O2 -g". The optimization is causing
# problems so just set it to "-g" if the user hasn't overridden it
# themselves.
# For GNU compilers, CFLAGS, CXXFLAGS, and FFLAGS default to "-O2 -g".
# The CXXFLAGS "-O2" optimization has caused problems in the past.
# For Intel compilers, "-g" slows down runtimes considerably (MET #1778).
# For development, retain the "-g" option. Otherwise, discard it.

: ${CXXFLAGS="-g"}
AM_COND_IF([ENABLE_DEVELOPMENT], [: ${CFLAGS="-g -O2"}], [: ${CFLAGS="-O2"}])
AM_COND_IF([ENABLE_DEVELOPMENT], [: ${CXXFLAGS="-g"} ], [: ${CXXFLAGS=""}] )
AM_COND_IF([ENABLE_DEVELOPMENT], [: ${FFLAGS="-g -O2"}], [: ${FFLAGS="-O2"}])

# Checks for programs.

Expand Down

0 comments on commit 50b77c0

Please sign in to comment.