summaryrefslogtreecommitdiffstats
path: root/Source
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'delete_copy_assign'Brad King2019-02-1835-1/+126
|\ | | | | | | | | | | | | ae5e97a005 Delete some default constructors and assignment operators Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2968
| * Delete some default constructors and assignment operatorsAlbert Astals Cid2019-02-1535-1/+126
| | | | | | | | | | | | They are unused, but if someone used them they would lead to problems since they would copy the internal raw pointers and the destructor would cause double delete
* | CMake Nightly Date StampKitware Robot2019-02-181-1/+1
| |
* | CMake Nightly Date StampKitware Robot2019-02-171-1/+1
| |
* | CMake Nightly Date StampKitware Robot2019-02-161-1/+1
| |
* | Merge topic 'systools-cleanup'Brad King2019-02-155-24/+3
|\ \ | | | | | | | | | | | | | | | | | | bd20cc29a2 cmSystemTools: Remove redundant cmCopyFile() and Split() Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2966
| * | cmSystemTools: Remove redundant cmCopyFile() and Split()Vitaly Stakhovsky2019-02-155-24/+3
| | |
* | | Merge topic 'remove_return_void'Kyle Edwards2019-02-151-3/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 30faf72625 Remove return from void function Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2969
| * | | Remove return from void functionAlbert Astals Cid2019-02-141-3/+3
| | |/ | |/|
* | | Merge topic 'ninja-swift-library-name'Brad King2019-02-153-0/+11
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 157570b5a2 Add placeholder for Swift's library name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2902
| * | | Add placeholder for Swift's library nameSaleem Abdulrasool2019-02-133-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows us to set the proper link name for the Swift library (soname). Because this needs to be passed to the object being compiled, we need to create a new placeholder so that it can be sent along to the frontend. Default to the target name unless it is explicitly provided.
* | | | Merge topic 'update-kwsys'Brad King2019-02-152-12/+4
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | f510aafbfc Merge branch 'upstream-KWSys' into update-kwsys a930de85d8 KWSys 2019-02-14 (e270ce9f) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2967
| * | | Merge branch 'upstream-KWSys' into update-kwsysBrad King2019-02-142-12/+4
| | |/ | |/| | | | | | | | | | * upstream-KWSys: KWSys 2019-02-14 (e270ce9f)
* | | Merge topic 'vs2017-sdk81'Brad King2019-02-151-1/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5c26e3c5e3 VS: Fix validation of Windows 8.1 SDK Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Dedmen Miller <dedmenmiller@googlemail.com> Merge-request: !2962
| * | | VS: Fix validation of Windows 8.1 SDKBrad King2019-02-141-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The check added by commit 0a29a31161 (VS2017: Verify Windows 8.1 SDK before using it, 2017-04-25, v3.8.1~2^2) used the wrong path to `windows.h` within the SDK, leading to it never being detected. Fixes: #18923
* | | | Merge topic 'try_compile-expand-compile-defs'Brad King2019-02-151-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cde2596a19 try_compile: Restore expansion of ;-list in COMPILE_DEFINITIONS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2965
| * | | | try_compile: Restore expansion of ;-list in COMPILE_DEFINITIONSBrad King2019-02-151-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The quoting added by commit 8c5221fb1f (try_compile: Preserve special characters in COMPILE_DEFINITIONS, 2019-01-21, v3.14.0-rc1~108^2~3) broke the case that the `COMPILE_DEFINITIONS` value contains a `;`. Without the quoting the `;` would be generated literally in an unquoted argument in the test `CMakeLists.txt` file and would then be expanded. With quoting the `;` is preserved, which is not the old behavior. Fix this by expanding the `;`-list ahead of time. Add test cases for behavior with both `#` and `;`. This was noticed with the PGI compiler where we set `CMAKE_CXX*_STANDARD_COMPILE_OPTION` to values like `--c++17;-A`. The symptom had also been observed while preparing commit ef8f237686 (ParseImplicitIncludeInfo: add SunPro Fortran and PGI compiler, Cray fix, 2019-01-29, v3.14.0-rc1~26^2~2) but was not recognized at the time as a regression. Revert the workaround added by that commit. Fixes: #18919
* | | | CMake Nightly Date StampKitware Robot2019-02-151-1/+1
| |/ / |/| |
* | | Merge topic 'fortran-submodule-names'Brad King2019-02-147-11/+76
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d80ecba5c2 Fortran: Fix submodule file names across compilers 72057d9c15 Fortran: Thread compiler id through to internal Fortran parser 7ae329e2ed Fortran: Factor out .mod and .smod file name construction Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Michael Hirsch, Ph.D. <michael@scivision.co> Merge-request: !2958
| * | | Fortran: Fix submodule file names across compilersBrad King2019-02-147-1/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The naming convention for submodule files varies across compilers. Add a table to the compiler information modules and thread the information through to the Fortran module dependency parser. Fill out the table for compiler ids known to support Fortran submodules. Fixes: #18746
| * | | Fortran: Thread compiler id through to internal Fortran parserBrad King2019-02-145-5/+26
| | | |
| * | | Fortran: Factor out .mod and .smod file name constructionBrad King2019-02-142-6/+21
| |/ /
* | | Merge topic 'fix-legacy-implicit-includes'Brad King2019-02-142-7/+29
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6fc3382944 Update logic for sysroot in detected implicit include directories 2ad14ef4ea cmAlgorithms: Add cmHasPrefix to match existing cmHasSuffix 557b2d6e65 Fix regression in -I/usr/include exclusion logic 017598a444 macOS: Fix addition of <sdk>/usr/include to default implicit include dirs Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2957
| * | | Update logic for sysroot in detected implicit include directoriesBrad King2019-02-131-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 5990ecb741 (Compute implicit include directories from compiler output, 2018-12-07, v3.14.0-rc1~108^2) the values of the `CMAKE_<LANG>_IMPLICIT_INCLUDE_DIRECTORIES` variables are computed from a real compiler invocation. In this case the paths under the sysroot should already have the sysroot prefix so we should no longer have to add the sysroot prefix. However, it is also possible for project code to add its own paths to `CMAKE_<LANG>_IMPLICIT_INCLUDE_DIRECTORIES` without the sysroot prefix and expect the historical addition of the sysroot prefix to be preserved. Try to account for both cases by conditionally adding the sysroot prefix on implicit include directories that do not already have it.
| * | | cmAlgorithms: Add cmHasPrefix to match existing cmHasSuffixBrad King2019-02-131-0/+8
| | | |
| * | | Fix regression in -I/usr/include exclusion logicBrad King2019-02-131-6/+16
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The change in commit 15ad830062 (Refactor exclusion of -I/usr/include to avoid per-language values, 2019-01-21, v3.14.0-rc1~108^2~4) caused the exclusion to apply to Fortran, but it was only meant for C, CXX, and CUDA. The purpose of the change was to prepare for the value of `CMAKE_<LANG>_IMPLICIT_INCLUDE_DIRECTORIES` to be computed from the actual compiler instead of hard-coded. We need to preserve exclusion of `-I/usr/include` if the compiler has any implicit include directory that looks intended to replace it, e.g. `<sdk>/usr/include` on macOS. Fixes: #18914
* | | Merge topic 'fix-pointer-truncation'Brad King2019-02-142-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 76a5ac2100 cmListFileLexer: Add missing include to avoid possible pointer truncation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2956
| * | | cmListFileLexer: Add missing include to avoid possible pointer truncationZsolt Parragi2019-02-132-2/+2
| |/ / | | | | | | | | | | | | | | | | | | The `cmsys/Enconding.h` include had a typo in its surrounding ifdef, possibly causing a missing function declaration (`cmsysEncoding_DupToWide`). As this is C code, this resulted in the code compiling, but with a truncated return value, possibly causing crashes.
* | | Merge topic 'update-kwsys'Brad King2019-02-147-17/+29
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ec13bd5230 Tests: Disable test that builds KWSys if not also building CMake ded4bc136e Merge branch 'upstream-KWSys' into update-kwsys f3999a21c8 KWSys 2019-02-12 (44676809) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2953
| * | | Merge branch 'upstream-KWSys' into update-kwsysBrad King2019-02-127-17/+29
| | | | | | | | | | | | | | | | | | | | * upstream-KWSys: KWSys 2019-02-12 (44676809)
* | | | CMake Nightly Date StampKitware Robot2019-02-141-1/+1
| |_|/ |/| |
* | | Merge topic 'names'Brad King2019-02-1311-270/+187
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | fc8b90af2c Create and use `cmGeneratorTarget::Names` Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2936
| * | | Create and use `cmGeneratorTarget::Names`Saleem Abdulrasool2019-02-1111-270/+187
| | | | | | | | | | | | | | | | | | | | | | | | Rather than taking a number of out parameters for the various names, create a structure that is reused for both `GetLibraryNames` and `GetExecutableNames`. Replace uses according to the new interface.
* | | | Merge topic 'systools-stdstring2'Brad King2019-02-137-74/+78
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 080a79ca4a cmSystemTools: More methods accept `std::string` params Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2952
| * | | | cmSystemTools: More methods accept `std::string` paramsVitaly Stakhovsky2019-02-117-74/+78
| |/ / /
* | | | Merge topic 'cmcurses-fix-solaris-lambda-warning'Brad King2019-02-131-5/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c28abdb5ac cmCursesMainForm: Fix lambda return type deducing warning Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2955
| * | | | cmCursesMainForm: Fix lambda return type deducing warningArtur Ryt2019-02-121-5/+6
| | |/ / | |/| | | | | | | | | | | | | | It was issued by sparc compiler on Solaris system See !2947
* | | | Merge topic 'genex-GENEX_EVAL-fix-recursion'Brad King2019-02-132-10/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e429e9af42 genex: Fix erroneous handling of recursion for $<GENEX_EVAL:> Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: yrHeTaTeJlb <jjj.bo@yandex.ru> Merge-request: !2954
| * | | | genex: Fix erroneous handling of recursion for $<GENEX_EVAL:>Marc Chevrier2019-02-132-10/+4
| | |_|/ | |/| | | | | | | | | | Fixes: #18894
* | | | CMake Nightly Date StampKitware Robot2019-02-131-1/+1
| |/ / |/| |
* | | Merge topic 'clangcl-fixes'Brad King2019-02-123-2/+17
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 091afa7342 Tests: Teach tests when to treat clang-cl as MSVC 006768903c Work around clang-cl breakage on make_unique/std::forward d625dfcdf9 Avoid compiling CMake itself as C++17 with Clang's MSVC ABI Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2573
| * | | Work around clang-cl breakage on make_unique/std::forwardZsolt Parragi2019-02-112-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Clang on Windows with the MSVC ABI produces link errors of the form `unresolved std::_Iosb<int>::{app,_Openmode} in ...`. Use a temporary variable to forward as lvalue rather than rvalue to work around the problem.
| * | | Avoid compiling CMake itself as C++17 with Clang's MSVC ABIZsolt Parragi2019-02-111-0/+13
| | |/ | |/| | | | | | | | | | | | | GUID functions in ATL cause compilation errors with this compiler. Add the offending case to our check for C++17 support so that it is not used by default when it does not work.
* | | Merge topic 'progress-stdstring'Brad King2019-02-1222-50/+58
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 950c099d83 cmake: Progress functions use `std::string` param Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2948
| * | | cmake: Progress functions use `std::string` paramVitaly Stakhovsky2019-02-1122-50/+58
| |/ /
* | | Merge topic 'modernize-cmCursesMainForm'Brad King2019-02-121-62/+48
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | f6291eee25 cmCursesMainForm: Modernize with STL and ranged-for loops Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2947
| * | | cmCursesMainForm: Modernize with STL and ranged-for loopsArtur Ryt2019-02-101-62/+48
| | | |
* | | | Merge topic 'cmCTestRunTest-compress'Brad King2019-02-122-78/+11
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 948c55857e cmCTestRunTest: Remove duplicated compression logic Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2946
| * | | | cmCTestRunTest: Remove duplicated compression logicRegina Pfeifer2019-02-102-78/+11
| |/ / /
* | | | CMake Nightly Date StampKitware Robot2019-02-121-1/+1
| |/ / |/| |