-
Notifications
You must be signed in to change notification settings - Fork 17
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
Run warnhist on MAPL #2512
Comments
It does work with CMake and I think it's useful for Fortran also. :-) Try the improved version that I just put up in a PR, which added a "warning density" calculation for builds (i.e. <warnings> / <lines of code>): HDFGroup/hdf5#3910 |
@qkoziol Ooh. New version! I'll give it a try. Also: do you know if |
Cool - the updated version has been merged into the develop branch now, so you could pull it from there if you'd like. I haven't tested with other Fortran compilers than gfortran. But if it doesn't work, you can post the build output here and I can see how difficult it is to handle them. |
@qkoziol I'm attaching a couple logs for you to peruse. First up GNU: and
Cool! Though, I will say Next up is Intel: and One other thing is we do build MAPL with the NAG Fortran compiler as well, so I might try to get you that output as well just so you can take a look at it. |
This issue has been automatically marked as stale because it has not had recent activity. If there are no updates within 7 days, it will be closed. You can add the "long term" tag to prevent the Stale bot from closing this issue. |
Whoops. This slipped for me. I need to test and run... |
This issue has been automatically marked as stale because it has not had activity in the last 60 days. If there are no updates within 7 days, it will be closed. You can add the ":hourglass: Long Term" label to prevent the stale action from closing this issue. |
While following around issues on GitHub I hit this reply by @qkoziol where he mentioned
warnhist
which is a script the HDF5 Group use.So, let's try it out on MAPL!
Possible fun:
The text was updated successfully, but these errors were encountered: