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

Detect external libiconv and make libiconv and git non-buildable on macOS #710

Merged
merged 3 commits into from
Aug 15, 2023

Conversation

climbfuji
Copy link
Collaborator

Summary

Detect external libiconv and make libiconv and git non-buildable on macOS. This addresses the issue mentioned in #672 (comment).

Making git and libiconv non-buildable on macOS avoids running these commands manually as users.

Testing

Tested on my macOS laptop.

Applications affected

n/a

Systems affected

macOS systems

Dependencies

Issue(s) addressed

See #672 (comment)

Checklist

  • This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.

Copy link
Collaborator

@srherbener srherbener left a comment

Choose a reason for hiding this comment

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

Thanks for this update! Tested jedi-bundle on my Mac with this build. All expected tests passed.

@climbfuji
Copy link
Collaborator Author

Thanks @srherbener. I got the following ctest failures for jedi-bundle when using Rosetta2 x86_64. I think these are expected.

The following tests FAILED:
	 14 - test_util_signal_trap (Failed)
	355 - saber_test_error_covariance_training_bump_hdiag-nicas_2_1-1 (Failed)
	440 - saber_test_error_covariance_training_bump_hdiag-nicas_2_2-1 (Failed)
	1087 - ufo_test_tier1_test_ufo_qc_variableassignment (Failed)
	1211 - ufo_test_tier1_test_ufo_opr_gnssrorefmetoffice (Failed)
	1215 - ufo_test_tier1_test_ufo_opr_gnssrobendmetoffice_nopseudo (Failed)
	1223 - ufo_test_tier1_test_ufo_opr_groundgnssmetoffice (Failed)
	1276 - ufo_test_tier1_test_ufo_opr_scatwind_neutral_metoffice (Failed)
	1450 - fv3jedi_test_tier1_errorcovariance (Failed)
	1496 - fv3jedi_test_tier1_errorcovariance_bump (Failed)
	1559 - test_soca_errorcovariance (Failed)

Also, on the same machine using the native arch64 build, I got the following ctest failures (expected?):

The following tests FAILED:
	 14 - test_util_signal_trap (Failed)
	355 - saber_test_error_covariance_training_bump_hdiag-nicas_2_1-1 (Failed)
	440 - saber_test_error_covariance_training_bump_hdiag-nicas_2_2-1 (Failed)
	745 - test_ioda-engines-00-Strings-F (SEGFAULT)
	1071 - ufo_test_tier1_test_ufo_qc_average_obs_to_mod_levels (Failed)
	1087 - ufo_test_tier1_test_ufo_qc_variableassignment (Failed)
	1211 - ufo_test_tier1_test_ufo_opr_gnssrorefmetoffice (Failed)
	1215 - ufo_test_tier1_test_ufo_opr_gnssrobendmetoffice_nopseudo (Failed)
	1223 - ufo_test_tier1_test_ufo_opr_groundgnssmetoffice (Failed)
	1276 - ufo_test_tier1_test_ufo_opr_scatwind_neutral_metoffice (Failed)
	1450 - fv3jedi_test_tier1_errorcovariance (Failed)
	1496 - fv3jedi_test_tier1_errorcovariance_bump (Failed)
	1560 - test_soca_errorcovariance (Failed)
	1595 - test_soca_sqrtvertloc (Failed)
	1622 - test_soca_convertincrement (Failed)

I'll go ahead and merge the spack PR.

@climbfuji climbfuji marked this pull request as ready for review August 14, 2023 23:55
@climbfuji climbfuji self-assigned this Aug 15, 2023
@climbfuji climbfuji added the INFRA JEDI Infrastructure label Aug 15, 2023
@climbfuji climbfuji merged commit f02cf8a into JCSDA:develop Aug 15, 2023
6 checks passed
@climbfuji climbfuji deleted the feature/detect_external_libiconv branch August 15, 2023 01:34
@srherbener
Copy link
Collaborator

@climbfuji your list of test failures on the ARM based Mac looks like the one I get. I have one additional test failure on my Mac: ufo_test_tier1_test_ufo_function_orbitangle_ssmis which is due to a breech of a tolerance limit on a floating point value check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INFRA JEDI Infrastructure
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

3 participants