summaryrefslogtreecommitdiffstats
path: root/Modules
Commit message (Collapse)AuthorAgeFilesLines
* Remove `//------...` horizontal separator commentsBrad King2016-05-093-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | Modern editors provide plenty of ways to visually separate functions. Drop the explicit comments that previously served this purpose. Use the following command to automate the change: $ git ls-files -z -- \ "*.c" "*.cc" "*.cpp" "*.cxx" "*.h" "*.hh" "*.hpp" "*.hxx" | egrep -z -v "^Source/cmCommandArgumentLexer\." | egrep -z -v "^Source/cmCommandArgumentParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmDependsJavaLexer\." | egrep -z -v "^Source/cmDependsJavaParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmExprLexer\." | egrep -z -v "^Source/cmExprParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmFortranLexer\." | egrep -z -v "^Source/cmFortranParser(\.y|\.cxx|Tokens\.h)" | egrep -z -v "^Source/cmListFileLexer\." | egrep -z -v "^Source/cm_sha2" | egrep -z -v "^Source/(kwsys|CursesDialog/form)/" | egrep -z -v "^Utilities/(KW|cm).*/" | xargs -0 sed -i '/^\(\/\/---*\|\/\*---*\*\/\)$/ {d;}' This avoids modifying third-party sources and generated sources.
* Help clang-format wrap after braces on long initializer listsBrad King2016-05-061-1/+3
| | | | | | Add a long comment inside a few braced initializer lists in order to convince clang-format to break after the opening brace and format the list without indenting every value past the opening brace.
* Merge topic 'clang-cl-showincludes'Brad King2016-05-041-2/+2
|\ | | | | | | | | 66de0866 Ninja: Fix clang-cl /showIncludes prefix detection
| * Ninja: Fix clang-cl /showIncludes prefix detectionBrad King2016-05-031-2/+2
| | | | | | | | | | | | | | | | | | We run MSVC-like compilers with the `/showIncludes` option and match the output to extract the corresponding message. Fix the matching to support compilers like `clang-cl` that print the message on the first line such that it is not preceded by a newline. LLVM-Issue: https://llvm.org/bugs/show_bug.cgi?id=27226
* | Merge topic 'ExternalProject-git-recursive-init'Brad King2016-05-031-2/+2
|\ \ | | | | | | | | | | | | 79410eeb ExternalProject: Initialize Git submodules recursively and on update (#16083)
| * | ExternalProject: Initialize Git submodules recursively and on update (#16083)Ilya Kulakov2016-05-031-2/+2
| |/
* | Merge topic 'FindBoost-no-system-env-paths'Brad King2016-05-031-2/+2
|\ \ | | | | | | | | | | | | 5a9b2b3b FindBoost: Fix Boost_NO_SYSTEM_PATHS to not consider environment (#15702)
| * | FindBoost: Fix Boost_NO_SYSTEM_PATHS to not consider environment (#15702)Brad King2016-05-021-2/+2
| |/ | | | | | | | | | | Previously we added only NO_CMAKE_SYSTEM_PATH to find command calls. Add NO_SYSTEM_ENVIRONMENT_PATH too so that paths found from system environment variables are not considered either.
* | FindHDF5: Add another debug suffixFARBOS Arnaud2016-05-021-2/+2
|/ | | | | HDF5 1.8.16 (and likely others) names its debug libraries with a `_debug` suffix intsead of `d`.
* Merge topic 'clang-format-include-order'Brad King2016-05-022-2/+1
|\ | | | | | | | | | | | | | | | | | | | | e1c77472 Format include directive blocks and ordering with clang-format 180538c7 Source: Stabilize include order 0e7bca92 Utilities/Release: Stabilize include order in WiX custom action eb817be0 Tests: Stabilize include order in MFC, VSXaml, and VSWinStorePhone eda313b4 Tests: Stabilize include order in StringFileTest 7110b754 CursesDialog: add missing cmState include d7a5f255 Modules: Remove unused CMakeTestWatcomVersion.c file
| * Format include directive blocks and ordering with clang-formatBrad King2016-04-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sort include directives within each block (separated by a blank line) in lexicographic order (except to prioritize `sys/types.h` first). First run `clang-format` with the config file: --- SortIncludes: false ... Commit the result temporarily. Then run `clang-format` again with: --- SortIncludes: true IncludeCategories: - Regex: 'sys/types.h' Priority: -1 ... Commit the result temporarily. Start a new branch and cherry-pick the second commit. Manually resolve conflicts to preserve indentation of re-ordered includes. This cleans up the include ordering without changing any other style. Use the following command to run `clang-format`: $ git ls-files -z -- \ '*.c' '*.cc' '*.cpp' '*.cxx' '*.h' '*.hh' '*.hpp' '*.hxx' | egrep -z -v '(Lexer|Parser|ParserHelper)\.' | egrep -z -v '^Source/cm_sha2' | egrep -z -v '^Source/(kwsys|CursesDialog/form)/' | egrep -z -v '^Utilities/(KW|cm).*/' | egrep -z -v '^Tests/Module/GenerateExportHeader' | egrep -z -v '^Tests/RunCMake/CommandLine/cmake_depends/test_UTF-16LE.h' | xargs -0 clang-format -i This selects source files that do not come from a third-party. Inspired-by: Daniel Pfeifer <daniel@pfeifer-mail.de>
| * Modules: Remove unused CMakeTestWatcomVersion.c fileBrad King2016-04-291-1/+0
| | | | | | | | | | This file has not been used since commit v2.8.10~218^2~7^2 (Watcom: Simplify compiler version detection, 2012-08-13).
* | Features: Record features for VS 2015 Update 2Brad King2016-04-281-1/+4
|/
* Merge topic 'intel-compiler-features'Brad King2016-04-286-3/+247
|\ | | | | | | | | | | | | | | | | | | | | 21b1fa5c Help: Add notes for topic 'intel-compiler-features' 9addce99 Features: Record standard flags for Intel C/C++ on Windows 15a6c950 WCDH: Add Intel to list of supported compilers 36f32ede Features: Record standards and features for Intel C on UNIX d028b948 Features: Specify minimum version Intel C++ 12.1 be910f00 Features: Record standards and features for Intel C++ on UNIX 7f401ae4 Features: Detect C default dialect on MSVC-like compilers
| * Features: Record standard flags for Intel C/C++ on WindowsBrad King2016-04-282-32/+44
| | | | | | | | | | Select the `-std=` or `-Qstd=` flag based on whether Intel is GNU-like or MSVC-like, respectively.
| * WCDH: Add Intel to list of supported compilersRobert Maynard2016-04-281-2/+3
| |
| * Features: Record standards and features for Intel C on UNIXRobert Maynard2016-04-282-0/+73
| | | | | | | | | | Record features for Intel C 12.1 and above. Skip this for now on Windows (where Intel C simulates MSVC).
| * Features: Specify minimum version Intel C++ 12.1Robert Maynard2016-04-281-22/+18
| | | | | | | | | | | | Versions below 12.1 do not provide enough information to properly detect if compiling with c++98 or c++0x enabled so remove them from the supported list.
| * Features: Record standards and features for Intel C++ on UNIXLevi Morrison2016-04-282-0/+158
| | | | | | | | Skip this for now on Windows (where Intel C++ simulates MSVC).
| * Features: Detect C default dialect on MSVC-like compilersBrad King2016-04-281-1/+5
| | | | | | | | | | | | These compilers default to C 90 but do not define __STDC__. While MSVC itself has no option to change the dialect, compilers that document compatibility with MSVC and define _MSC_VER may (e.g. Intel).
* | ExternalProject: Add option to perform a shallow Git clone (#15291)Brad King2016-04-271-3/+20
| | | | | | | | Inspired-by: Ilya Kulakov <kulakov.ilya@gmail.com>
* | FindBISON: Fix BISON_TARGET macro for special characters in path (#16072)Brad King2016-04-201-4/+4
| | | | | | | | | | Use the VERBATIM option to add_custom_command so that the command is escaped correctly.
* | FindFLEX: Fix FLEX_TARGET macro for special characters in path (#16072)Brad King2016-04-201-2/+2
| | | | | | | | | | Use the VERBATIM option to add_custom_command so that the command is escaped correctly.
* | Merge topic 'coverage_collect_delete'Brad King2016-04-201-2/+6
|\ \ | | | | | | | | | | | | 5de122df CTestCoverageCollectGCOV: improve DELETE option
| * | CTestCoverageCollectGCOV: improve DELETE optionZack Galbreath2016-04-191-2/+6
| |/ | | | | | | | | | | | | | | | | | | | | | | | | The DELETE option to ctest_coverage_collect_gcov now properly removes all the .gcov files that were created by this function. Previously it left behind any files that were excluded by CTEST_CUSTOM_COVERAGE_EXCLUDE. This option now also deletes the following files/directory that are created by ctest_coverage_collect_gcov: data.json coverage_file_list.txt the uncovered/ directory
* | Merge topic 'boost-import-default'Brad King2016-04-201-2/+2
|\ \ | | | | | | | | | | | | 2c1b720e FindBoost: Define Boost::boost for all boost versions
| * | FindBoost: Define Boost::boost for all boost versionsRoger Leigh2016-04-191-2/+2
| |/
* | BundleUtilities: Add optional parameter to ignore specified filesRoman Wüger2016-04-191-37/+115
|/
* Merge topic 'FindMPI-intel-compiler'Brad King2016-04-181-2/+13
|\ | | | | | | | | | | 2a99fae1 FindMPI: Recognize `.lib` file names for specification of link libraries c4417b09 FindMPI: Recognize /LIBPATH for specification of linker paths.
| * FindMPI: Recognize `.lib` file names for specification of link librariesDominic Meiser2016-04-151-0/+11
| | | | | | | | | | The Intel MPI compiler wrappers link against static MPI libraries simply by listing the libraries (no `-l`).
| * FindMPI: Recognize /LIBPATH for specification of linker paths.Dominic Meiser2016-04-151-2/+2
| | | | | | | | The Intel MPI wrappers use this form of -L to specify library locations.
* | FindGit: Add new 'GitHub for Windows' user directoryFrank Park2016-04-151-2/+4
|/ | | | | | | | | | | | | GitHub for Windows now installs in $ENV{LOCALAPPDATA}/Github/PortableGit*/cmd instead of $ENV{LOCALAPPDATA}/Github/PortableGit*/bin Search the new location first but keep the old one in case people have older versions installed.
* FindMPI: Improve documentation formattingChristoph Grüninger2016-04-111-12/+15
|
* Merge topic 'GetPrerequisites-more-dumpbin-paths'Brad King2016-04-111-0/+11
|\ | | | | | | | | | | a1c40291 GetPrerequisites: Look for VS tools using environment variables c79f88b0 GetPrerequisites: add path of VS 10,11,12,14 in order to find dumpbin.exe
| * GetPrerequisites: Look for VS tools using environment variablesBrad King2016-04-081-0/+7
| | | | | | | | | | | | The `VS*COMNTOOLS` environment variables specify locations of VS tools and are set during the VS installation. Use them in addition to the hard-coded default install locations.
| * GetPrerequisites: add path of VS 10,11,12,14 in order to find dumpbin.exeFARBOS Arnaud2016-04-081-0/+4
| |
* | MSVC: Drop space in `/DNDEBUG` flag for consistency (#16052)Brad King2016-04-072-9/+9
|/ | | | | | | | We define `NDEBUG` without a space after the `-D` option for most compilers. Remove the space for MSVC (and Intel Fortran) for consistency. The MS compiler technically does not document that the `-D` argument may be separated from its value, though every version to date supports it.
* Merge topic 'branch_coverage_working_dir'Brad King2016-04-071-2/+2
|\ | | | | | | | | c5ff34cc CTestCoverageCollectGCOV: specify base dir for GLOB_RECURSE
| * CTestCoverageCollectGCOV: specify base dir for GLOB_RECURSEZack Galbreath2016-04-041-2/+2
| | | | | | | | | | | | Consistently glob for .gcda files in the binary directory. Previously the behavior of this function depended on the current working directory that it was called from.
* | Merge topic 'branch_coverage_glob'Brad King2016-04-071-8/+70
|\ \ | | | | | | | | | | | | | | | 8ea1b0df CTestCoverageCollectGCOV: Improve documentation 425d7646 CTestCoverageCollectGCOV: Honor CTEST_EXTRA_COVERAGE_GLOB
| * | CTestCoverageCollectGCOV: Improve documentationZack Galbreath2016-04-071-8/+13
| | | | | | | | | | | | Clean up prose and mention support for CTEST_EXTRA_COVERAGE_GLOB.
| * | CTestCoverageCollectGCOV: Honor CTEST_EXTRA_COVERAGE_GLOBZack Galbreath2016-04-071-0/+57
| | | | | | | | | | | | | | | | | | | | | Teach CTestCoverageCollectGCOV to honor the CTEST_EXTRA_COVERAGE_GLOB variable. When this variable is set, this module will glob for matching source files that were not covered and include them in the resulting tar file.
* | | Merge topic 'UseJava-cmake-command'Brad King2016-04-071-1/+1
|\ \ \ | | | | | | | | | | | | | | | | fa382ae8 UseJava: use CMAKE_COMMAND instead of direct cmake call
| * | | UseJava: use CMAKE_COMMAND instead of direct cmake callBrad King2016-04-061-1/+1
| | | | | | | | | | | | | | | | Author: dancing-leaves <dancing-leaves@users.noreply.github.com>
* | | | Merge topic 'ExternalProject-TLS_VERIFY-fixes'Brad King2016-04-071-10/+16
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | 4d8c988c ExternalProject: Fix file download script with CMAKE_TLS_{VERIFY,CAINFO} 23a71e4e ExternalProject: Tell Git not to verify certs only if TLS_VERIFY is OFF
| * | | ExternalProject: Fix file download script with CMAKE_TLS_{VERIFY,CAINFO}Brad King2016-04-061-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid generating incorrect code such as set(CMAKE_TLS_VERIFY set(CMAKE_TLS_VERIFY 1)) when one of these variables is set in the calling project.
| * | | ExternalProject: Tell Git not to verify certs only if TLS_VERIFY is OFFBrad King2016-04-061-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 272779ce (ExternalProject: Allow TLS_VERIFY for git clones, 2016-04-01) we pass the `-c http.sslVerify=false` option to `git clone` even if no explicit `TLS_VERIFY` option was set. This changes behavior because we used to use the default Git behavior by default. Revise the logic to preserve the old default behavior by passing the new option only if `TLS_VERIFY` was explicitly passed as `OFF`. While at it, also honor `CMAKE_TLS_VERIFY` if the explicit `TLS_VERIFY` option is not given.
* | | | Merge topic 'FindOpenSSL-names-per-dir'Brad King2016-04-061-0/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | b1484403 FindOpenSSL: Prefer libs early in search path regardless of name (#15887)
| * | | | FindOpenSSL: Prefer libs early in search path regardless of name (#15887)Hannes Mezger2016-04-051-0/+10
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Add NAMES_PER_DIR to all find_library invocations so that we consider all possible names in each search directory before moving on to the next directory. Otherwise we may not find self-built libraries first even if they appear early in the search path.
* | | | Merge topic 'mingw-w64-Fortran-platform'Brad King2016-04-061-6/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | 8c4f100a Fortran: Fix platform id detection on mingw-w64 66fa6143 CMakeDetermineFortranCompiler: Modernize conventions