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

Fix warnings files #753

Merged
merged 249 commits into from
Jun 15, 2021
Merged

Fix warnings files #753

merged 249 commits into from
Jun 15, 2021

Conversation

byrnHDF
Copy link
Contributor

@byrnHDF byrnHDF commented Jun 15, 2021

Also addresses PR#751

byrnHDF added 30 commits January 7, 2021 07:15
byrnHDF and others added 21 commits June 3, 2021 14:27
ccmake runs iteratively, and the check_fortran_source_runs
macros were clobbering a single output file that did not
get updated on further configure iterations
@byrnHDF byrnHDF self-assigned this Jun 15, 2021
H5_CFLAGS="$H5_CFLAGS $(load_intel_arguments 18)"
fi

# intel <= 19
Copy link
Collaborator

Choose a reason for hiding this comment

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

Effect is to add -Wcheck for intel version 19 and lower? This is the reverse of the application of most of the compiler flags. It might be worth adding a comment explaining why this one is different.

@lrknox lrknox merged commit dfe4f29 into HDFGroup:develop Jun 15, 2021
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.

3 participants