| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| | |
Merge-request: !1422
|
| |
| |
| |
| | |
Closes #17404
|
|\ \
| | |
| | |
| | | |
Merge-request: !1424
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Test was failing in case dist macro contained
a + symbol which is valid but must be escaped
for using the string as a regex.
Fixes #17328
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1418
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Refactoring in commit v3.10.0-rc1~115^2 (Clean up iwyu code to not be
one big if statement, 2017-08-28) incorrectly changed the logic to run
only one lint tool at a time. Restore support for running all tools
specified on the command-line.
|
| | | |
| | | |
| | | |
| | | | |
These do not need to be declared in the header.
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | | |
Merge-request: !1408
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
When a STATIC_LIBRARY cycle is detected we don't add any STATIC_LIBRARY target
from the cycle to the `_autogen` target dependencies.
Closes #17389
|
|\ \ \ \ \
| |_|_|_|/
|/| | | |
| | | | | |
Merge-request: !1415
|
| | |_|/
| |/| |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In commit v3.5.0-rc1~8^2~2 (Help: Clarify policy `CMP0040`
documentation, 2016-01-28) the documentation was clarified to indicate
that the target must be defined in the current directory. Do the same
for the text of the policy warning itself.
Fixes: #17399
|
|\ \ \ \
| |_|/ /
|/| | |
| | | | |
Merge-request: !1407
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
We do this for `cl`, so we should do it for `clang-cl`.
Fixes: #17394
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1406
|
| |/ /
| | |
| | |
| | |
| | |
| | | |
Explicitly return a value from `main` in our test sources.
Fixes: #17391
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1405
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Since commit v3.9.0-rc1~224^2~1 (FindMPI: MPIEXEC handling improvements,
2017-04-21) the `ProcessorCount` module is being used to initialize
`MPIEXEC_MAX_NUMPROCS`. However, this leads to the logical cores being
counted rather than the physical ones, and some MPI implementations like
OpenMPI will error if mpiexec is called with that number. Switch it to
the number of physical cores using `cmake_host_system_information`.
This ensures that if `MPIEXEC_MAX_NUMPROCS` is being used to set up MPI
tests with CTest or similar that the tests won't spuriously fail due to
OpenMPI refusing to start the application.
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1398
|
| |/ / |
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1397
|
| | |/
| |/|
| | |
| | |
| | | |
Value keywords are actually UNDEFINED rather than set to the empty
string when they are not found in the argument list.
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1400
|
| | |/
| |/|
| | |
| | | |
Fixes: #17381
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1401
|
| |/ /
| | |
| | |
| | | |
Fixes: #17380
|
|\ \ \
| |_|/
|/| |
| | | |
Merge-request: !1393
|
| |/
| |
| |
| |
| | |
The Flang compiler neither supports nor documents -fbounds-check leading
to -Wunused-command-line-argument warnings with the default Debug flags.
|
| | |
|
|\ \
| | |
| | |
| | | |
Merge-request: !1390
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The `TargetMachinePlatform` setting tells CUDA what `--machine {32,64}`
flag to pass to nvcc. While CUDA automatically chooses the proper
default for the target architecture, it does not reflect this in the
user-visible IDE settings. Set it explicitly to fix the user-visible
setting.
Fixes: #17355
|
|\ \
| | |
| | |
| | | |
Merge-request: !1385
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This supports libraries given as full path and filters libraries given
in CMAKE_<LANG>_STANDARD_LIBRARIES.
Fixes: #17351
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1388
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The `Microsoft.VisualStudio.Component.VC.Tools.x86.x64` component is
not the only way a VS instance may provide the `cl` compiler tool.
For example, VS 2017 Express Edition does not install that component.
Instead search for the tools directly on disk within an instance.
Suggested-by: Rich Chiodo <rchiodo@microsoft.com>
Fixes: #17349
|
| | | | |
|
| | |/
| |/|
| | |
| | | |
This also adds a missing conversion to unix slashes in one code path.
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1386
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This property is meant to be set on the consumers of imported targets,
not the imported targets themselves.
Fixes: #17348
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The change in commit v3.8.0-rc1~276^2 (Allow NO_SYSTEM_FROM_IMPORTED on
imported INTERFACE libraries, 2016-11-21) was incorrect. The property
is not meant to be set on imported targets at all. It is meant to be
set on their consumers that compile sources. Since INTERFACE libraries
have no sources to compile, the property is not needed on them.
Revert most of that change. Unfortunately we must still tolerate
project code setting NO_SYSTEM_FROM_IMPORTED on INTERFACE libraries
because they were allowed by CMake 3.8 and 3.9.
Issue: #17348
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1377
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
`xmlrpc-c-config $modules --cflags` may report no `-I` flags if the
headers are in a standard location like `/usr/include`. In this case
it is okay for `XMLRPC_INCLUDE_DIRS` to be empty.
Ideally the `--cflags` output should be used as hints for a `find_path`
call to really find the headers as we do for other find modules, but
simply assuming no explicit include directories are needed is good
enough for now.
Fixes: #17347
|
| |/ /
| | |
| | |
| | |
| | | |
We always have `execute_process` now, so drop our ancient `exec_program`
code paths that are never used anymore.
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1373
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The LLVM/Clang installer on Windows provides a `LLVM/bin` directory
containing `clang.exe` and `clang++.exe` command-line tools that have a
GNU-like command-line but target the MSVC ABI (instead of MinGW). We
do not support this combination, so diagnose and reject it explicitly.
Tell users what to do to use the `clang-cl.exe` tool instead.
Issue: #16439
|
|\ \ \
| | | |
| | | |
| | | | |
Merge-request: !1375
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Regex [+-~] passes all characters between + and ~ and not only
+, - and ~
|