summaryrefslogtreecommitdiffstats
path: root/config/gnu-flags
Commit message (Collapse)AuthorAgeFilesLines
* Update copyright headers (#2184)Larry Knox2022-11-011-1/+0
| | | | | * Updated source file copyright headers to remove "Copyright by the Board of Trustees of the University of Illinois", which is kept in the top-level COPYING file.
* Re-promote some GNU warnings to errors (#1508)David Young2022-05-061-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Promote some GCC warnings back to errors and deal with build breakage. I tested configuring and building with GCC 8.3.0 with `.../configure --prefix=$HOME/wrk/install/manydsets --enable-build-mode=production --enable-symbols=-gdwarf-4 --enable-warnings-as-errors --enable-java --enable-hl --enable-fortran --enable-cxx --enable-mirror-vfd --enable-ros3-vfd --enable-direct-vfd` and with the same options but `--enable-build-mode=debug`. * Promote -Wrestrict to -Werror=restrict to help catch overlapping arguments to strcpy, memcpy, et cetera, at compile time. * NFCI: sort lines. * Committing clang-format changes * NFCI: remove whitespace at EOL. * Re-use temporary variable `shared`. * Remove merge oopsie. * Mention that no op codes are understood per review comment. * Change this back to the develop branch's way, since this change isn't integral to the PR. * Committing clang-format changes * NFCI: reduce differences with `develop` branch in the hope that that also kicks off CI. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Dana Robinson <43805+derobins@users.noreply.github.com>
* Fixed Spelling Errors (#1166)Scot Breitenfeld2021-12-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fixed missed closing of a dataset * fixed missed closing of a dataset * fixed typo in error return * Committing clang-format changes * minor edits * code format * Committing clang-format changes * code format * minor edit * switched from using MPI_count, to actual bytes written for H5FD_mpio_debug rw debugging * Committing clang-format changes * changed size_i in printf to reflect the I/O. * Committing clang-format changes * Fixed seg fault with xlf on BE with -qintsize=8 * fixed error function string * spelling corrections via codespell, added new spell check github actions * Committing clang-format changes * misc * misc * misc * misc * misc * misc * misc * misc * misc * misc * misc * misc * misc * misc * Committing clang-format changes * misc * misc * misc * misc * misc * misc * Committing clang-format changes * misc * work around for https://github.com/codespell-project/codespell/issues/2137 * misc * added missing file * misc * misc. * misc * switch to using Codespell with GitHub Actions * misc. * misc. * fixed more sp errors * Fix new typos found by codespell. * fixed proceed with precede * fixed variable in fortran test * fixed minnum * updated spelling list Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
* To reduce maintenance effort, delete the `noerror-` variants of the compiler ↵David Young2021-10-061-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | flags files (#1033) * Avoid maintenance headaches: delete the `noerror-` variants of the compiler flags files, since they essentially duplicate the `error-` files modulo the replacement of `-Werror=` with `-W` and any changes in comments. (I verified the duplication with a script.) For autoconf, reinstate the use of the `demote_errors` shell function to derive the `noerror-` content from the `error-` content. `demote_errors` replaces `-Werror=` with `-W` when `WARNINGS_AS_ERRORS` is `no`. Slightly reorder `configure.ac` so that the setting of `WARNINGS_AS_ERRORS` takes effect before the `error-` files are sourced. * Take a stab at updating the CMake files to match the changes I made to the autoconf files to remove `noerror-` files. I'm not much of a CMake user so these changes are quite rough. Looks like the duplication can be reduced with the introduction of a new macro. * Delete `noerror-` files from the MANIFEST. * Reduce duplication in the CMake files: perform the HDF5_ENABLE_WARNINGS_AS_ERRORS test once in the ADD_H5_FLAGS macro. * Add a release note.
* Organize and align compiler flags (#741)Allen Byrne2021-06-111-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * OESS-98 convert plugin option to FetchContent, add tests * Fixes for pkcfg files because of plugin option * OESS-98 fix tools test for plugins * Keep doxygen comments under 100 chars long - format hint * Whitespace * HDFFV-11144 - Reclassify CMake messages * HDFFV-11099/11100 added help text * Reworked switch statement to compare string instead * Fix typo * Update CDash mode * Correct name of threadsafe * Correct option name * Undo accidental commit * Note LLVM 10 to 11 format default changes * Update format plugin * Undo clang-format version 11 changes * One more correction * Update supported platforms * Revert whitespace changes * Correct whitespace * Changes from PR#3 * HDFFV-11213 added option to control gcc10 warnings diagnostics * HDFFV-11212 Use the new references correctly in JNI utility and tests * format source * Fix typo * Add new test file * HDFFV-11212 - update test and remove unused arg * Minor non-space formatting changes * Use H5I_INVALID_ID instead of "-1" * source formatting * add missing testfile, update jni function * Undo commit of debug code * remove mislocated file * Fix h5repack test for handling of fapls and id close * Update h5diff test files usage text * HDFFV-11212 add new ref tests for JNI export dataset * src format update * Remove blank line typo * src format typo * long double requires %Lg * Another long double foramt specifer S.B. %Lg * issue with t128bit test * Windows issue with h5dump and type. * Fix review issues * refactor function nesting and fix error checks * format fixes * Remove untested functions and javadoc quiet comments * Restore TRY block. * Change string append errors to memory exception * revert to H5_JNI_FATAL_ERROR - support functions need work * Add assertion error for h5util functions * remove duplicate function * format fix * Revert HD function error handling * Update copyright comments * GH #386 java folder copyright corrections * Whitespace * GH #359 implement and fix tools 1.6 API usage * remove excessive comments * Flip inits to correct ifdef section * rework ifdef to be simpler * format issue * Reformat ifdef inits * remove static attribute * format compliance * Update names * Revert because logic relies on float not being int * Changes noticed from creating merge of #412 * Double underscore change * Correct compiler version variable used * Remove header guard underscores * Whitespace cleanup * Split format source and commit changes on repo push * remove pre-split setting * Change windows TS to use older VS. * correct window os name * HDFFV-11212 JNI export util and Javadoc * Suggested review changes * Another change found * Committing clang-format changes * HDFFV-11113 long double in tools * HDFFV-11113 add note * Disable long double tests for now * HDFFV-11228 remove arbitrary CMake warning groups. * Make each flag a string * Some Javadoc warning fixes * Updated javadoc fixes * # WARNING: head commit changed in the meantime HDFFV-11229 Fix long double usage in tools and java Changed h5dump and h5ls to just print 128-bit for long double type. Added test and file for dataset and attributes with all three float types. * Committing clang-format changes * HDFFV-11229 Add release note * HDFFV-11229 windows testfile needed * fix typo * Remove non supported message text * HDFFV-11229 - change ldouble test to check both native and general * HDFFV-11229 add second file compare * HDFFV-11229 fix reference file * HDFFV-11229 autotools check two refs * HDFFV-11229 revert back to removal of NATIVE_LDOUBLE in tools output * Committing clang-format changes * Update release note * Update attribute type of ref file * Change source of ninja for macs * try port instead of brew * Recommended is to use brew. * Undo non long double changes * remove unneeded file * Fix perl and doxygen CMake code * Add "option" command for clang options * Rework CMake add_custom to use the BYPRODUCTS argument * Add stamp files to BYPRODUCTS * Only one copy of file exists * Fix custom cmmand depends targets * Fix fortran custom command DEPENDS * Add LD_LIBRARY_PATH to tests * Add custom target for DEPENDS * Add h5detect conditionaly to generated target DEPENDS * Correct DEPENDS targets * Parallel builds need the mpi compiler for pkgconfig scripts. * install only if MPI build * Fortran target depends * Remove incorrect source attribute * doxygen adjustments * doxygen build updates * Correct version * Correct function version - function has been merged to 1.12 * Correct version string for map functions * Cleanup warnings for tools debug builds * TRILAB-227 - fix indexing for h5diff selections * Correct location of pos to index function call * TRILAB-227 Enable test * Quote subset args * Use MATCHES because of AppleClang * if blocks needed for build modes * Update list of DT platforms * VS2019 correctly displays float values * revert VS2019 change * Issue #669 remove version from pkgcfg filename * remove version from h5cc script * Java reference functions updated enabled fortran in cmake gcc action yaml file java reference test changed to correctly test refs jni reference functions that create ids changed to use jni id logging * Correct BYPRODUCTS argument * Correct more genereated files BYPRODUCTS * BYPRODUCTS must have unique locations * Fix typo * Fix fortran configure checks * Rework H5_PAC_C_MAX_REAL_PRECISION setting logic * Add note about fortran configure change * Adds a quick for for some egregious chunk_info badness (#722) * Fixes issue with ccmake that prevents building Fortran (#723) 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 * Fix conflicts with merge * Move MAX_PRECISION back to HDF5UseFortran.cmake * Use STREQUAL to test macro argument * Move C language test to ConfigureChecks from HDF5UseFortran * MAX_PRECISION defines must be defined * Organize flags and align autotools and cmake * Fix comment in no-error-general warnings files. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Dana Robinson <43805+derobins@users.noreply.github.com>
* Update license url part2 (#333)Larry Knox2021-02-171-1/+1
| | | | | | * Modify temporary rpath for testing in java example scripts. * Update URL in source file Copyright headers for web copy of COPYING file - files not in src or test.
* Add option to control gcc 10 warnings diagnostics (#321)Allen Byrne2021-02-081-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * OESS-98 convert plugin option to FetchContent, add tests * Fixes for pkcfg files because of plugin option * OESS-98 fix tools test for plugins * Keep doxygen comments under 100 chars long - format hint * Whitespace * HDFFV-11144 - Reclassify CMake messages * HDFFV-11099/11100 added help text * Reworked switch statement to compare string instead * Fix typo * Update CDash mode * Correct name of threadsafe * Correct option name * Undo accidental commit * Note LLVM 10 to 11 format default changes * Update format plugin * Undo clang-format version 11 changes * One more correction * Update supported platforms * Revert whitespace changes * Correct whitespace * Changes from PR#3 * HDFFV-11213 added option to control gcc10 warnings diagnostics
* Add compiler flags for GCC 10, along with updating warnhist script to ↵Quincey Koziol2020-12-211-0/+10
| | | | accommodate them. (#217)
* Add else choice to error flags option (#128)Allen Byrne2020-11-251-0/+3
|
* Minor/solaris gcc fix (#126)Dana Robinson2020-11-251-4/+8
| | | | | | | * Adds gnu99 as the standard when building with gcc on Solaris * Adds a note to RELEASE.txt and improves comments For Solaris + gcc fix (HDFFV-11191)
* Revise gnu-*flags and cmake/HDF*CompilerFlags.cmake files to add warningLarry Knox2020-05-181-60/+9
| | | | | | | flags for GCC compilers version 4.8 and above. Removed files from gnu-warnings that only apply to versions < 4.8. Consolidated warnings from versions < 4.8 that apply to versions >= 4.8 into the 4.8 warnings files.
* Restrict errors to gcc 4.8 and above.Allen Byrne2020-04-241-0/+2
|
* TRILAB-244 separate CXX warnings and errors from CAllen Byrne2020-04-231-19/+19
|
* Don't add general warnings flags for unsupported old versions of gcc andLarry Knox2020-04-201-2/+4
| | | | | | g++ (older than gcc/g++ 4.2). Correct gnu-cxxflags to determine warnings flags to be added based on C++ compiler version instead of C compiler version.
* Update all load_gcc_arguments to load_gnu_arguments.Larry Knox2020-04-091-11/+11
| | | | Fixes TRILAB-212.
* TRILAB-192 add c++ and fortran warnings build systems one fileAllen Byrne2020-04-051-26/+26
|
* Merge pull request #2402 in HDFFV/hdf5 from ↵David Young2020-03-231-74/+70
|\ | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:break-out-warnings to develop * commit '2f09d8f3c8a4a44932c1b8ffb0f6ea2219059a5c': Move the GNU warnings configuration under config/gnu-warnings/ and trim the filename prefixes. Update config/gnu-flags to suit. By default, demote warnings promoted to errors back to warnings. Incorporate the H5_ECFLAGS into the library CFLAGS. Now a bunch of errors will occur. Next commit will demote the warnings promoted to errors back to warnings again. Break out warnings into more files that autoconf and CMake can share. This change temporarily disables the warnings that were promoted to errors, but I will add those warnings back as warnings, not errors, in the next commit.
| * Move the GNU warnings configuration under config/gnu-warnings/ and trimDavid Young2020-03-191-35/+35
| | | | | | | | | | | | | | the filename prefixes. Update config/gnu-flags to suit. I also made changes to config/cmake/HDFCompilerFlags.cmake, but I'm not sure they're correct.
| * By default, demote warnings promoted to errors back to warnings.David Young2020-02-251-1/+19
| |
| * Break out warnings into more files that autoconf and CMake can share. ThisDavid Young2020-02-251-70/+48
| | | | | | | | | | change temporarily disables the warnings that were promoted to errors, but I will add those warnings back as warnings, not errors, in the next commit.
* | Modfy expression to match mpicc build with Intel as intel compiler, notLarry Knox2020-02-271-1/+1
|/ | | | gcc.
* Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2020-01-241-1/+3
|\ | | | | | | | | | | | | | | | | | | | | * commit '590aaff33046df99a4d88ba59e4b461e060b36e4': Optimized the floating point comparisons a little bit. Fix for failing h5diff tests involving floating-point compares. Removed H5_DEC_ENUM Revert "Revert "Moved -Wunsuffixed-float-constants to the developer warnings."" Revert "Moved -Wunsuffixed-float-constants to the developer warnings." Moved -Wunsuffixed-float-constants to the developer warnings. Fixed a bug in testpar/t_cache.c concerning checking expected vs. actual cache entry reads and writes.
| * Revert "Revert "Moved -Wunsuffixed-float-constants to the developer warnings.""Dana Robinson2020-01-231-1/+3
| | | | | | | | This reverts commit 6486e06545bf637d46e18a787395542f4fca143a.
* | Create common build system files for warningsAllen Byrne2020-01-171-15/+31
|/
* Changed -Wformat-truncation's level from 2 to 1.Dana Robinson2020-01-011-1/+1
|
* Yanked -Wc++-compat from the flags used to build the C library in bothDana Robinson2019-12-041-1/+4
| | | | the Autotools and CMake.
* Revert "Make a squash commit of 'Quiet some warnings by adjusting warnings ↵David Young2019-11-271-87/+11
| | | | level and fixing some code.' (commit 5c911d8baf3)"
* Always warn on maybe-uninitialized. -Wincompatible-pointer-types was notDavid Young2019-11-221-2/+2
| | | | available until GCC 5, so enable it only if that's the GCC version we're using.
* Only promote maybe-uninitialized warnings to errors on GCC 8. Even onDavid Young2019-11-221-1/+7
| | | | | GCC 8, there may be false positives at low optimization levels? I need to check.
* Only use -Werror=cast-function-type with GCC 8 and later.David Young2019-11-221-1/+1
|
* Put all of the -W options back into the order I found them in so that it'sDavid Young2019-11-221-29/+23
| | | | easier to compare old and new config/gnu-flags.
* Promote decleration-after-statement warnings to errors.David Young2019-11-131-1/+1
|
* Make errors of some more warnings. Move disabled warnings toDavid Young2019-11-131-31/+43
| | | | | DEVELOPER_WARNING_CFLAGS. Put just one warning option on a line, and sort some of the options.
* Change some GCC warnings to errors. Fix code to quiet some warnings.David Young2019-11-051-11/+75
|
* Fixed a couple of typos.Dana Robinson2019-08-011-1/+1
|
* Restored warnings for older gcc so jam can continue to creak along.Dana Robinson2019-07-161-2/+12
|
* Removed support for gcc 4.3 and earlier.Dana Robinson2019-07-141-11/+1
|
* Fix errant quotesDana Robinson2019-07-101-1/+1
|
* * Fixed a bug in the gcc warning parsingDana Robinson2019-07-101-8/+8
| | | | | * Changed version parsing from == gcc 4.x to < to handle ancient compilers.
* Reworked the config/gnu-flags file to be more organized and robust.Dana Robinson2019-07-101-82/+82
|
* Converted duplicated flags code to a set of if statements.Dana Robinson2019-07-101-435/+117
|
* Added GCC9 flags to CMake buildQuincey Koziol2019-07-031-1/+0
|
* Add support for GCC9, update warnhist script, and clean up warnings.Quincey Koziol2019-07-031-10/+61
|
* Move the -Wformat-nonliteral warning to the developer flags. Fix bugsQuincey Koziol2019-06-291-2/+5
| | | | I introduced in the last commit.
* Updated configure & CMake compiler flags for GCC 8.x, along with correspondingQuincey Koziol2019-06-281-11/+61
| | | | | changes to warnhist script (and some extra improvements for condensing C++ and Java warnings), and fixed a bunch of warnings.
* Move -Wnormalized down into GCC 6.x flagsQuincey Koziol2019-06-221-3/+3
|
* Add support for GCC 7.x warnings, update warnhist script to account for them,Quincey Koziol2019-06-211-3/+47
| | | | clean up warnings.
* HDFFV-10776 missing compiler flagsAllen Byrne2019-04-301-9/+9
|
* Correct typo in comment in config/gnu-flags.Larry Knox2018-07-241-1/+1
|
* Apply Cygwin pathches from Marco Atzeri.Larry Knox2018-07-181-1/+13
| | | | Commit Joe Lee's typo corrections for src/H5MF.c.