summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'kwiml-test-gcc-format-security' into releaseBrad King2017-02-271-1/+1
|\
| * KWIML: Add -Wno-format-security to tolerate -Werror=format-securityOrion Poplawski2017-02-271-1/+1
| | | | | | | | | | | | | | | | | | When compiling KWIML tests with `-Werror=format-security` we get error: -Wformat-security ignored without -Wformat Therefore we should disable both `format` and `format-security` together.
* | Merge branch 'ExternalProject-checkout-clarify' into releaseBrad King2017-02-241-1/+1
|\ \
| * | ExternalProject: Run `git checkout` with `--` to clarify argumentsBrad King2017-02-241-1/+1
| |/ | | | | | | | | | | | | Fix the case when the tag name to be checked out also happens to match a path name. Fixes: #16678
* | Merge branch 'object-library-compile-pdb' into releaseBrad King2017-02-242-1/+2
|\ \
| * | Fix COMPILE_PDB_NAME when used on an OBJECT libraryBrad King2017-02-242-1/+2
| |/ | | | | | | | | | | | | | | When `COMPILE_PDB_NAME` is used without `COMPILE_PDB_OUTPUT_DIRECTORY` we cannot fall back on the link `PDB_OUTPUT_DIRECTORY` for an object library because it has no link step. Fixes: #16674
* | Merge branch 'ninja-no-full-path' into releaseBrad King2017-02-241-4/+3
|\ \
| * | Revert "Ninja: Use full path for all source files"Brad King2017-02-241-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit v3.7.0-rc1~275^2 (Ninja: Use full path for all source files, 2016-08-05). Unfortunately using absolute paths can cause incorrect rebuilds due to ninja limitations. The ninja manual [1] explains: > ... using absolute paths, your depfile may result in a mixture of > relative and absolute paths. Paths used by other build rules need > to match exactly. Passing an absolute path to a source file to the compiler while using a relative path in the ninja build manifest can cause such mixture and lead to incorrect rebuilds. Simply revert the change for now. Note that there was a follow-up to the original change in commit v3.7.0-rc2~10^2 (Ninja: Fix RC language depfile generation with cmcldeps, 2016-10-13). We don't need to revert that because that change made the relevant code cleverly adapt to whatever variable we use to reference the source file. [1] https://ninja-build.org/manual.html#_deps Fixes: #16675 Issue: #13894
| * | Merge branch 'fix-ctest_update-svn' into release-3.7Brad King2017-02-151-1/+4
| |\ \
* | \ \ Merge branch 'vcxproj-indentation' into releaseBrad King2017-02-231-0/+1
|\ \ \ \
| * | | | VS: Fix .vcxproj indentationBrad King2017-02-231-0/+1
| | |_|/ | |/| | | | | | | | | | | | | | | | | | In commit v3.8.0-rc1~87^2~1 (VS: added support for C#, 2017-01-09) we removed what looked like a no-op streaming operation but in fact it is responsible for applying indentation. Restore the line.
* | | | Merge branch 'clang-format-attrs' into releaseBrad King2017-02-229-44/+60
|\ \ \ \
| * | | | clang-format.bash: Use Git attributes to mark files for formattingBrad King2017-02-229-32/+27
| | | | |
| * | | | Define a custom Git attribute to mark sources using our C styleBrad King2017-02-221-5/+15
| | | | |
| * | | | Define a custom Git attribute to mark source files as generatedBrad King2017-02-222-10/+21
| | | | |
* | | | | Merge branch 'doc-cmake-language-makefile-vars' into releaseBrad King2017-02-221-0/+6
|\ \ \ \ \
| * | | | | Help: Clarify 'make-style' variable references in cmake-language(7)Michael Maltese2017-02-221-0/+6
| | |/ / / | |/| | | | | | | | | | | | | Fixes: #16661
* | | | | Merge branch 'FindCUDA-fix-init' into releaseBrad King2017-02-221-0/+1
|\ \ \ \ \
| * | | | | FindCUDA: Fix PTX selection with multiple architecturesBjoern Thiel2017-02-221-0/+1
| |/ / / /
* | | | | Merge branch 'vs2017-fallback-sdk' into releaseBrad King2017-02-212-0/+38
|\ \ \ \ \
| * | | | | VS2017: If Win 8.1 SDK is not available, use Win 10 SDKBrad King2017-02-212-0/+38
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We try to choose the Windows SDK version based on the version of Windows targeted by the build. However, if using VS 2017 without the Windows 8.1 SDK installed then we must fall back to the Windows 10 SDK even when targeting an older version of Windows. Inspired-by: gnaggnoyil <gnaggnoyil@gmail.com>
* | | | | Merge branch 'ExternalProject-update-docs' into releaseBrad King2017-02-211-2/+9
|\ \ \ \ \
| * | | | | ExternalProject: Clarify documented role of `INSTALL_DIR`Brad King2017-02-211-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It does not actually configure the project to install to the given location. That must be done by passing the `<INSTALL_DIR>` placeholder to the external project configuration step.
| * | | | | ExternalProject: Clarify documented role of `INSTALL_COMMAND`Wilfried Goesgens2017-02-211-1/+5
| |/ / / / | | | | | | | | | | | | | | | | | | | | It is for the install step of the external project and does not affect installation of the calling project.
* | | | | Merge branch 'cuda-separable-bool' into releaseBrad King2017-02-212-2/+4
|\ \ \ \ \
| * | | | | CUDA: Fix boolean interpretation of CUDA_SEPARABLE_COMPILATIONBrad King2017-02-202-2/+4
| |/ / / / | | | | | | | | | | | | | | | | | | | | If the property is explicitly set to a non-true value we should not treat it as true.
* | | | | Merge branch 'FindGTK2-old-sigc++' into releaseBrad King2017-02-211-1/+4
|\ \ \ \ \
| * | | | | FindGTK2: handle old libsigc++ versionsBen Boeckel2017-02-201-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Old versions of `libsigc++` do not have the version macros inside of its `sigc++config.h` header. Assume nothing about such headers and report version "zero". Fixes: #16654
* | | | | | Merge branch 'fix-CMP0017-path-case' into releaseBrad King2017-02-171-1/+1
|\ \ \ \ \ \
| * | | | | | Windows: Fix inconsistent behavior on changes to case of path to cmakeBrad King2017-02-171-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit v3.4.0-rc1~333^2 (Merge branch 'upstream-kwsys' into update-kwsys, 2015-07-15) we brought in upstream KWSys commit 86a24794 (SystemTools: Fix GetActualCaseForPath drive letter case handling, 2015-07-09). This caused our path processing to convert drive letters to upper-case and exposed an existing bug in our implementation of CMP0017. Policy CMP0017 is responsible for ensuring that modules included from a builtin module only load other builtin modules and cannot be overridden by a file in `CMAKE_MODULE_PATH`. If there is a case difference in the drive letter (or other path components) then the path to the including module may not match our builtin module directory in a simple string comparison. This means builtin modules may not be recognized as such, and they may not reliably include their builtin dependencies. For example, if a project provides a `Platform/Windows` module in `CMAKE_MODULE_PATH` it can break inclusion of our builtin `Platform/Windows` module, leading to strange behavior. Fix this by comparing the path to the including module to our builtin module directory using a function that is aware of case-insensitivity of paths on Windows. Fixes: #16648, #16622
* | | | | | Merge branch 'FindHDF5-fix-header-langs' into releaseBrad King2017-02-171-3/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | FindHDF5: Fix search for language-specific header fileBrad King2017-02-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit v3.6.0-rc1~85^2 (HDF5: Refactor the use of compiler wrappers, 2016-04-04) converted code of the form if(${LANGUAGE} MATCHES ...) to if(LANGUAGE MATCHES ...) However, `LANGUAGE` is a foreach() loop variable and not a normal variable so auto-dereference does not occur. Restore the explicit `${}` syntax and use the new name of the loop variable that has changed since then too. Fixes: #16651
| * | | | | FindHDF5: Restore HDF5_<LANG>_INCLUDE_DIR advanced cache entry markBrad King2017-02-171-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit v3.6.0-rc1~72^2 (HDF5: Rework component searching to correctly find HL for all bindings, 2016-05-12) renamed the language loop variable used to construct the name of `HDF5_<LANG>_INCLUDE_DIR` but forgot to update it in the `mark_as_advanced` call. Fix it now. Issue: #16651
* | | | | Merge branch 'autogen_fix_3.8' into releaseBrad King2017-02-151-2/+2
|\ \ \ \ \
| * | | | | Autogen: Fix headers not skippedSebastian Holtermann2017-02-151-2/+2
| | | | | |
* | | | | | Merge branch 'fix-ctest_update-svn' into releaseBrad King2017-02-151-1/+4
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | |
| * | | | | ctest_update: Fix svn log and external loadingBrad King2017-02-151-1/+4
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.7.0-rc1~132^2 (CTestSVN: Fix segfault when CTEST_UPDATE_VERSION_ONLY is enabled, 2016-09-12) we do not properly extract svn log or externals. After updating we erase the information that was loaded before updating and can no longer log the changes between revisions to extract authors. Fix this by only loading the repository information once, whether needed by `NoteOldRevision`, `NoteNewRevision`, or both. Fixes: #12630, #16646
* | | | | Merge branch 'cuda-msvc-flags' into releaseBrad King2017-02-157-16/+17
|\ \ \ \ \
| * | | | | CUDA: Fix default compiler flags on WindowsBrad King2017-02-152-11/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the default values of `CMAKE_CUDA_FLAGS[_<CONFIG>]` on Windows to make the host compiler flags match those produced for C++ by the `Platform/Windows-MSVC` module. This makes the flags consistent with those used for C++.
| * | | | | CUDA: Fix test cases to not override CUDA flagsBrad King2017-02-155-5/+5
|/ / / / / | | | | | | | | | | | | | | | Append to `CMAKE_CUDA_FLAGS` instead of overwriting it.
* | | | | Merge branch 'cuda-no-Os' into releaseBrad King2017-02-141-1/+1
|\ \ \ \ \
| * | | | | CUDA: Do not use non-existent -Os flag for nvccBrad King2017-02-141-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | Fix the CUDA MinSizeRel configuration flags to avoid using the `-Os` flag that nvcc does not support.
* | | | | Merge branch 'cuda-with-c' into releaseBrad King2017-02-146-0/+51
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | CUDA: Link to standard system libraries when linking as CUDABrad King2017-02-143-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Windows with MSVC-like host compilers we must honor the standard libraries chosen by the `Platform/Windows-MSVC` module. Otherwise C code linked into the CUDA binary that expects to have these libraries available may not link.
| * | | | Tests: Add case for CUDA with C but not C++Brad King2017-02-144-0/+35
| | | | | | | | | | | | | | | | | | | | An executable using CUDA and C should link as CUDA.
* | | | | Merge branch 'backport-clang-format-cuda' into releaseBrad King2017-02-1420-88/+79
|\ \ \ \ \ | |/ / / /
| * | | | Tests: Run clang-format on CUDA codeBrad King2017-02-1416-86/+59
| | | | |
| * | | | Modules: Run clang-format on CUDA codeBrad King2017-02-141-1/+1
| | | | |
| * | | | clang-format.bash: Format CUDA source files tooBrad King2017-02-143-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add `.clang-format` configuration files for Cuda test directories that use `Standard: Cpp11`. Otherwise clang-format splits the triple angle brackets used for CUDA kernels.
* | | | | Merge branch 'execute_process-default-encoding' into releaseBrad King2017-02-132-5/+16
|\ \ \ \ \