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

Mvclmbldtry2 #88

Merged
merged 5 commits into from
Mar 9, 2016
Merged

Mvclmbldtry2 #88

merged 5 commits into from
Mar 9, 2016

Conversation

jedwards4b
Copy link
Contributor

mv clm build into build_libraries
prepare to move tests into python by looking for TESTNAME_script and
using the python case.test template if its not found.

Instead of having case.test_build and case.build in the case directory and
throwing an error if the user trys case.build, just replace case.build with case.test_build

@jgfouca jgfouca self-assigned this Mar 9, 2016
@@ -166,10 +166,6 @@ def case_build(caseroot, case=None, testmode=False, sharedlib_only=False, model_
case = Case() if case is None else case
testcase = case.get_value("TESTCASE")
cimeroot = case.get_value("CIMEROOT")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

case_build still takes testmode as an optional argument. Does it still need it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that it doesn't.

@jgfouca
Copy link
Contributor

jgfouca commented Mar 9, 2016

Review complete.

jgfouca added a commit that referenced this pull request Mar 9, 2016
@jgfouca jgfouca merged commit 274e93c into ESMCI:master Mar 9, 2016
@jedwards4b jedwards4b deleted the mvclmbldtry2 branch March 28, 2016 18:23
jayeshkrishna pushed a commit that referenced this pull request Aug 16, 2016
Implementation of PIOc_strerror() and pio_strerror() to get strings associated with error codes
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