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

Ac263 support #1213

Merged
merged 3 commits into from
Sep 25, 2020
Merged

Ac263 support #1213

merged 3 commits into from
Sep 25, 2020

Conversation

marshallward
Copy link
Collaborator

@marshallward marshallward commented Sep 23, 2020

Additional changes were made to support older Autoconf versions (at
least as far back as 2.63).

  • AC_FC_LINE_LENGTH, added in 2.67, was backported
  • AC_OPENMP, bugged for Fortran until 2.69, tests against the C flag

Version restrictions are now explicit (2.63).

This may work with 2.62, when the m4_version_change macro was
introduced, but I have not yet tested it.

The Cray pointer macro's cache variable was also renamed.

Additional changes were made to support older Autoconf versions (at
least as far back as 2.63).

* AC_FC_LINE_LENGTH, added in 2.67, was backported
* AC_OPENMP, bugged for Fortran until 2.69, tests against the C flag

Version restrictions are now explicit (2.63) and

This may work with 2.62, when the m4_version_change macro was
introduced, but I have not yet tested it.

The Cray pointer macro's cache variable was also renamed.
Reduced two m4 version conditionals into a single block.
@marshallward
Copy link
Collaborator Author

This did not resolve all of @kshedstrom's issues using 2.63 on her machine, but at the least I would say it is "necessary but not sufficient".

As the window to investigate further may have closed, I think this may as well be merged and we hope it resolves the issue.

@adcroft adcroft merged commit 36ca92f into mom-ocean:dev/gfdl Sep 25, 2020
@marshallward marshallward deleted the ac263_support branch January 18, 2021 12:42
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