summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'ninja-no-rsp-for-rc'Brad King2016-06-285-0/+24
|\ | | | | | | | | 15b3f6f0 ninja, rc: ignore CMAKE_NINJA_FORCE_RESPONSE_FILE for RC files
| * ninja, rc: ignore CMAKE_NINJA_FORCE_RESPONSE_FILE for RC filesBen Boeckel2016-06-275-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit v3.6.0-rc1~174^2 (Ninja: Honor CMAKE_NINJA_FORCE_RESPONSE_FILE for compile rules, 2016-04-06), Ninja learned to look for `CMAKE_NINJA_FORCE_RESPONSE_FILE` in the current scope or the environment in order to force response file usage for all compilation rules. However, on Windows, the RC compiler goes through cmcldeps which does a `replace(output, output + ".dep.obj")` on the command line. However, with a response file (which we name `output + ".rsp"`), the response file path is replaced instead causing the compiler to (correctly) complain that the response file `output + ".dep.obj.rsp"` does not exist. What needs to happen is for cmcldeps to look through the response file, replace *its* contents and place it in the `output + ".dep.obj.rsp"` file. Also add a test which actually compiles an RC file into a library and executable for all generators on Windows and additionally test `CMAKE_NINJA_FORCE_RESPONSE_FILE` for Ninja generators. Fixes #16167.
* | Merge topic 'vs-tool-override'Brad King2016-06-232-1/+128
|\ \ | | | | | | | | | | | | ed05f11d VS: Add a VS_TOOL_OVERRIDE source file property
| * | VS: Add a VS_TOOL_OVERRIDE source file propertyGilles Khouzam2016-06-202-1/+128
| | | | | | | | | | | | | | | | | | | | | | | | This property allow to specify a specific Visual Studio tool for a source file overriding the default tool behavior. For example, a `.resw` file being processed as a `PriResource` file. This has the advantage of being able to teach CMake to process new file types without code modifications.
* | | Merge topic 'vs-sdk-refs'Brad King2016-06-231-0/+3
|\ \ \ | | | | | | | | | | | | | | | | 35a6cac2 VS: Add VS_SDK_REFERENCES target property to reference external SDKs
| * | | VS: Add VS_SDK_REFERENCES target property to reference external SDKsGilles Khouzam2016-06-221-0/+3
| |/ / | | | | | | | | | | | | Allow one to specify external SDK references such as `Microsoft.AdMediatorWindows81, Version=1.0`.
* | | Merge topic 'xcode-bundle-extension'Brad King2016-06-231-1/+0
|\ \ \ | |/ / |/| | | | | | | | e36e5a26 Xcode: Properly handle BUNDLE_EXTENSION
| * | Xcode: Properly handle BUNDLE_EXTENSIONGregor Jasny2016-06-181-1/+0
| | |
* | | Merge topic 'cmake-trace-source'Brad King2016-06-204-0/+9
|\ \ \ | | | | | | | | | | | | | | | | e63151ff cmake: Add an option to control what files needs to be traced
| * | | cmake: Add an option to control what files needs to be tracedAlex Turbov2016-06-174-0/+9
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Even in relatively small projects using `--trace` (and `--trace-expand`) may produce a lot of output. When developing a custom module usually one is interested in output of only a few particular modules. Add a `--trace-source=<file>` option to enable tracing only a subset of source files. The final output would be only from requested modules, ignoring anything else not matched to given filename(s).
* | | Merge topic 'cpack-deb-test-fixes'Brad King2016-06-203-10/+13
|\ \ \ | |/ / |/| / | |/ | | | | 6107fab4 CPack/Deb: Fix test to actually test the preinst script 302391ba CPack/Deb: Fix tests for SELinux-enabled systems
| * CPack/Deb: Fix test to actually test the preinst scriptDomen Vrankar2016-06-201-2/+2
| | | | | | | | | | | | | | The test regex variable name was invalid and did not test preinst script. Reported-by: Patrick <mail6543210@yahoo.com.tw>
| * CPack/Deb: Fix tests for SELinux-enabled systemsDomen Vrankar2016-06-203-10/+13
| | | | | | | | Suggested-by: Patrick <mail6543210@yahoo.com.tw>
* | Merge topic 'link_what_you_use'Brad King2016-06-1713-0/+62
|\ \ | | | | | | | | | | | | | | | a0902efa Help: Add notes for topic 'link_what_you_use' 96242f80 Add options to run `ldd -u -r` as a "link-what-you-use" tool
| * | Add options to run `ldd -u -r` as a "link-what-you-use" toolBill Hoffman2016-06-1713-0/+62
| | | | | | | | | | | | | | | | | | | | | Create a LINK_WHAT_YOU_USE target property and corresponding CMAKE_LINK_WHAT_YOU_USE variable to enable this behavior. Extend link commands by running `ldd -u -r` to detect shared libraries that are linked but not needed.
* | | Merge topic 'cleanup-streams'Brad King2016-06-162-4/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | ed5fa48d cmXMLWriter: use ifstream from KWSys 24ab29b8 Prefer istringstream and ostringstream over stringstream. ab8b77dd Remove redundant arguments from fstream constructors eb79fa72 Access std::ios_base with std::ios
| * | | Remove redundant arguments from fstream constructorsDaniel Pfeifer2016-06-142-4/+2
| | | | | | | | | | | | | | | | Don't pass the default value of the openmode parameter explicitly.
* | | | Parser: Issue file open error messages through dedicated APIStephen Kelly2016-06-134-20/+4
|/ / /
* | | Merge topic 'fix-Message-API-ISP-violation'Brad King2016-06-135-0/+25
|\ \ \ | | | | | | | | | | | | | | | | 0a4af073 cmake: Issue message independent of cmMakefile definition
| * | | cmake: Issue message independent of cmMakefile definitionStephen Kelly2016-06-125-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The makefile is only used when called by the cmMessageCommand, so inline the use of it there. It otherwise creates an undesirable dependency on cmMakefile for issuing messages in the cmake instance, a violation of the Interface Segregation Principle. https://en.wikipedia.org/wiki/Interface_segregation_principle This also makes it more explicit that the variable definitions only affect the message() command. If an AUTHOR_WARNING is issued for any other reason, it is not affected. To affect that, it is necessary to set the cache variable instead of the regular variable. This is an unfortunate interface quirk, but one which can't be fixed easily now.
* | | | Merge topic 'find-lib32'Brad King2016-06-1315-18/+47
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 896ad251 Teach find_library and find_package to search lib32 paths (#11260)
| * | | | Teach find_library and find_package to search lib32 paths (#11260)Daniel Scharrer2016-06-1015-18/+47
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Add a ``FIND_LIBRARY_USE_LIB32_PATHS`` global property analogous to the ``FIND_LIBRARY_USE_LIB64_PATHS`` property. This helps find commands on multilib systems that use ``lib32`` directories and either do not have ``lib`` symlinks or point ``lib`` to ``lib64``.
* | | | Add missing braces around statements.Daniel Pfeifer2016-06-102-10/+17
| | | | | | | | | | | | | | | | | | | | Apply fixits of clang-tidy's readability-braces-around-statements checker.
* | | | PseudoMemcheck: revise style with clang-format.Daniel Pfeifer2016-06-101-14/+8
|/ / / | | | | | | | | | | | | | | | Rename memtester.cxx.in to memtester.cxx, run clang-format, then restore the original name. Fix the @_retval@ placeholder that was broken by clang-format.
* | | Merge topic 'watcom-workaround'Brad King2016-06-101-1/+3
|\ \ \ | | | | | | | | | | | | | | | | ceebac93 Tests: Restore fix to Plugin test on Watcom compiler
| * | | Tests: Restore fix to Plugin test on Watcom compilerBrad King2016-06-091-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | The fix in commit v3.4.0-rc1~127^2 (Tests: Fix Plugin test on Watcom compiler, 2015-09-03) was reverted accidentally by automated .c_str() removal. Restore it and add a comment explaining why.
* | | | Merge topic 'java-export-targets'Brad King2016-06-0912-12/+199
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | 0bd91ad4 UseJava: Fix race condition creating java class list 89df91b9 Help: Add notes for topic 'java-export-targets' 95d84369 Tests: Add test for exported JARs 5341c0d8 UseJava: Add infrastructure to export targets d91ec044 Tests/Java: Clean up style of Java test code
| * | | Tests: Add test for exported JARsMatthew Woehlke2016-06-0710-0/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a unit test to test the new functions to export JAR targets. The test builds three sub-projects: two that generate and export a JAR (one does a build-directory-only export, one an install-only export), and a third that consumes the first two as imported targets.
| * | | Tests/Java: Clean up style of Java test codeMatthew Woehlke2016-06-072-12/+12
| |/ /
* | | Merge topic 'add-FindVulkan'Brad King2016-06-081-1/+1
|\ \ \ | | | | | | | | | | | | | | | | b8a00b67 Tests/FindVulkan: Run clang-format
| * | | Tests/FindVulkan: Run clang-formatBrad King2016-06-081-1/+1
| | | |
* | | | Merge topic 'add-FindVulkan'Brad King2016-06-084-0/+58
|\ \ \ \ | |/ / / | | | | | | | | | | | | adf4df28 Add FindVulkan.cmake.
| * | | Add FindVulkan.cmake.Matthäus G. Chajdas2016-06-084-0/+58
| | | | | | | | | | | | | | | | This adds FindVulkan with corresponding tests.
* | | | Merge topic 'find_path-in-framework'Brad King2016-06-084-0/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 188baef0 find_path: Fix location of <dir/header.h> in a framework on OS X
| * | | | find_path: Fix location of <dir/header.h> in a framework on OS XHarry Mallon2016-06-074-0/+8
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | After finding it in `foo.Framework/Headers/dir/header.h`, we should report the `foo.Framework/Headers` directory, not `foo.Framework/Headers/dir`, because the former is what actually contains the path the caller wishes to include.
* | | | Tests/FindOpenCL: Run clang-format to fix styleBrad King2016-06-071-4/+5
|/ / /
* | | Remove c_str() calls from stream arguments.Daniel Pfeifer2016-06-061-2/+1
| | | | | | | | | | | | | | | | | | Mostly automated: git grep -l '.c_str() <<' | xargs sed -i 's|\.c_str() <<| <<|g'
* | | Merge topic 'productbuild'Brad King2016-06-061-0/+11
|\ \ \ | |/ / |/| | | | | | | | | | | | | | 63e5eb5f Help: Add notes for 'productbuild' topic 2e3c67d1 productbuild: Add new productbuild cpack generator. 50a3d340 PackageMaker: factor out common code for creating pkg files.
| * | productbuild: Add new productbuild cpack generator.Clinton Stimpson2016-06-031-0/+11
| | | | | | | | | | | | This cpack generator basically replaces the obsolete PackageMaker generator.
* | | Merge topic 'fix-TARGET_PROPERTY-LOCATION-crash'Brad King2016-06-033-0/+14
|\ \ \ | | |/ | |/| | | | | | | f500a784 Fix crash on $<TARGET_PROPERTY:...,LOCATION> genex (#16134)
| * | Fix crash on $<TARGET_PROPERTY:...,LOCATION> genex (#16134)Brad King2016-06-023-0/+14
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Policy CMP0026 deprecated the LOCATION property, and we have long provided a $<TARGET_FILE:...> generator expression. However, if a project tries to use $<TARGET_PROPERTY:...,LOCATION> we should at least not crash. The compatibility implementation of the LOCATION property uses cmGlobalGenerator::CreateGenerationObjects to create the structures needed to evaluate the property before generation starts. The implementation assumed that accessing the property could only be done during configuration (via the typical get_property command use case). The $<TARGET_PROPERTY:...,LOCATION> genex causes the LOCATION property to be accessed during generation. Calling CreateGenerationObjects during generation blows away all the objects currently being used for generation and is not safe. Add a condition to call it only when configuration is not finished.
* | FindOpenCL: Add an imported targetMatthäus G. Chajdas2016-06-024-0/+44
|/
* Merge topic 'remove-needless-copies'Brad King2016-05-271-1/+1
|\ | | | | | | | | | | 27ead963 Remove unnecessary local copies. 618fb23f Pass arguments that are not modified as const&.
| * Pass arguments that are not modified as const&.Daniel Pfeifer2016-05-261-1/+1
| | | | | | | | | | | | | | Use clang-tidy's performance-unnecessary-value-param checker to find value parameter declarations of expensive to copy types that are not modified inside the function. Ignore findings in kwsys. After applying the fix-its, manually change `const T&` to `T const&`.
* | Remove redundant c_str() calls.Daniel Pfeifer2016-05-262-9/+9
|/ | | | | Run clang-tidy's readability-redundant-string-cstr checker. Ignore findings in kwsys.
* try_compile: Optionally forward custom platform variables to test projectBrad King2016-05-252-0/+27
| | | | | | | | | Add a `CMAKE_TRY_COMPILE_PLATFORM_VARIABLES` variable to specify a list of custom variables to be forwarded to a `try_compile` test project. This will be useful for platform information modules or toolchain files to forward some platform-specific set of variables from the host project (perhaps set in its cache) to the test project so that it can build the same way.
* Merge topic 'standard-include-directories'Brad King2016-05-254-0/+11
|\ | | | | | | | | | | | | c1340827 Add a variable to specify language-wide system include directories 44199097 cmMakefile: Optimize AddSystemIncludeDirectories for empty set a896043b GHS: Compute include directories consistently with other generators
| * Add a variable to specify language-wide system include directoriesBrad King2016-05-254-0/+11
| | | | | | | | | | | | | | Create a `CMAKE_<LANG>_STANDARD_INCLUDE_DIRECTORIES` variable to specify system include directories for for `<LANG>` compiler command lines. This plays a role for include directories as the existing `CMAKE_<LANG>_STANDARD_LIBRARIES` variable does for link libraries.
* | Merge topic 'cpack-deb-different-package-names'Brad King2016-05-2419-13/+43
|\ \ | | | | | | | | | | | | adbd3985 CPack/Deb possibility to change package name
| * | CPack/Deb possibility to change package nameDomen Vrankar2016-05-2319-13/+43
| |/ | | | | | | | | | | | | | | This patch preserves backward compatibility of deb package names with previous CMake versions but similarly to CPack/RPM allows to change package name format and supports DEB-DEFAULT setting that produces proper Debian package names.