summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Tests/CompileDefinitions: Avoid spaces in defines on VS 6Stephen Kelly2013-05-244-4/+18
| | | | | | | | The VS 6 IDE does not like spaces in definition values so CMake drops them and warns. The Tests/CompileDefinitions test C code that looks for the dropped definitions already knows to skip them, but CMake still warns. Silence the warnings by avoiding such values in the first place on VS 6.
* Merge topic 'geh-cleanup-identifiers'Brad King2013-05-246-0/+50
|\ | | | | | | | | 7e24997 GenerateExportHeader: Generate only C identifiers as defines
| * GenerateExportHeader: Generate only C identifiers as definesStephen Kelly2013-05-246-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The variables in this module are used to configure a header file with defines whose name depends on the name of the target. As valid names of targets may be invalid for use as defines, convert the names of the defines used to C identifiers first. This is already done in C++ code for the DEFINE_SYMBOL property. This is not as simple as ensuring that the BASE_NAME is a C identifier, because most of the define names are configurable, and because use of a BASE_NAME which is not a C identifier, such as 4square can become a C identifier by specifying a prefix in the generate_export_header macro.
* | Merge topic 'geh-MODULE-export'Brad King2013-05-241-5/+3
|\ \ | | | | | | | | | | | | 7ffef50 GenerateExportHeader: Allow use of of this macro with MODULEs.
| * | GenerateExportHeader: Allow use of of this macro with MODULEs.Stephen Kelly2013-05-231-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The significant issue with MODULEs is that on Windows, the exported symbols must be dllexported and they are not imported. In other export macro implementations this is done by defining an export macro outside of any ifdef which depends on definitions set on the command line. However, with cmake we already expect the DEFINE_SYMBOL to be defined, so the regular EXPORT macro can be used by such plugins.
* | | Merge topic 'geh-add-missing-test'Brad King2013-05-241-0/+1
|\ \ \ | | | | | | | | | | | | | | | | 74f24b2 Tests/Module/GenerateExportHeader: Test exported free-function
| * | | Tests/Module/GenerateExportHeader: Test exported free-functionStephen Kelly2013-05-241-0/+1
| | | |
* | | | Merge topic 'geh-output-tweak'Brad King2013-05-241-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | bb412d0 GenerateExportHeader: Add newlines to separate the compiler output.
| * | | | GenerateExportHeader: Add newlines to separate the compiler output.Stephen Kelly2013-05-231-2/+2
| |/ / / | | | | | | | | | | | | Make the error logs more readable.
* | | | Merge topic 'language-generator-expressions'Brad King2013-05-2413-8/+200
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 3241014 Add $<LINK_LANGUAGE> generator expression
| * | | | Add $<LINK_LANGUAGE> generator expressionStephen Kelly2013-05-2413-8/+200
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They can't be used when evaluating link libraries, but they can be used for include directories and compile definitions. Later they can be used for compile options.
* | | | | Merge topic 'fix-include_directories-whitespace-handling'Brad King2013-05-241-1/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | a97240b Tests/IncludeDirectories: Avoid shared library with no symbols
| * | | | | Tests/IncludeDirectories: Avoid shared library with no symbolsBrad King2013-05-231-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parent commit added a empty_entry_test shared library to the IncludeDirectories test. Some toolchains fail to create a shared library with no user-defined symbols, so provide a dummy symbol.
* | | | | | Merge topic 'Memchecker-Mac-fix'Brad King2013-05-241-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0aed09f Tests: ignore GuardMalloc messages on all Apple build, not just XCode ones
| * | | | | | Tests: ignore GuardMalloc messages on all Apple build, not just XCode onesRolf Eike Beer2013-05-231-1/+1
| | | | | | |
* | | | | | | CMake Nightly Date StampKitware Robot2013-05-241-1/+1
| |_|_|_|/ / |/| | | | |
* | | | | | Merge topic 'xcode-attributes-variant'Brad King2013-05-231-2/+33
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 332350b Xcode: Support XCODE_ATTRIBUTE_ with [variant=<config>] (#12532)
| * | | | | | Xcode: Support XCODE_ATTRIBUTE_ with [variant=<config>] (#12532)Cédric OCHS2013-05-221-2/+33
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit c519bb2b (XCode: Also qoute [] as needed to set build-configurations, 2011-04-05) we escape "[]" conditions for XCODE_ATTRIBUTE_ settings. However, we need to handle the "variant" condition with a special case to map it only into the settings for the matching configuration.
* | | | | | Merge topic 'xcode-shared-linker-flags-per-config'Brad King2013-05-231-17/+13
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | bcda47d Xcode: Honor CMAKE_(MODULE|SHARED)_LINKER_FLAGS_<CONFIG> (#14161)
| * | | | | Xcode: Honor CMAKE_(MODULE|SHARED)_LINKER_FLAGS_<CONFIG> (#14161)Matthew Bentham2013-05-221-17/+13
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Refactor lookup of CMAKE_(EXE|MODULE|SHARED)_LINKER_FLAGS(|_<CONFIG>) variables to ensure all combinations are handled. Use the helper method AddConfigVariableFlags to simplify the implementation.
* | | | | CMake Nightly Date StampKitware Robot2013-05-231-1/+1
| |_|_|/ |/| | |
* | | | Merge topic 'join-genex'Brad King2013-05-227-51/+145
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | a7ba452 Add the JOIN generator expression. 96ec314 Make it possible for any genex to have arbitrary content at the end. bd638ee Rename the method determining if a genex accepts arbitrary content. dc742fe Extract the ProcessArbitraryContent method.
| * | | Add the JOIN generator expression.Stephen Kelly2013-05-166-3/+61
| | | | | | | | | | | | | | | | | | | | | | | | This generator expression joins a list with a separator. The separator may contain arbitrary content, such as commas, which is ordinarily a delimiter in the generator expression syntax.
| * | | Make it possible for any genex to have arbitrary content at the end.Stephen Kelly2013-05-161-1/+14
| | | |
| * | | Rename the method determining if a genex accepts arbitrary content.Stephen Kelly2013-05-161-6/+8
| | | | | | | | | | | | | | | | | | | | The meaning of this will be expanded to generator expressions with more than a single parameter.
| * | | Extract the ProcessArbitraryContent method.Stephen Kelly2013-05-162-41/+62
| | | |
* | | | Merge topic 'LinkLine-typedef'Brad King2013-05-221-2/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 776225d cmTarget: drop the unused local typedef LinkLine
| * | | | cmTarget: drop the unused local typedef LinkLineRolf Eike Beer2013-05-221-2/+0
| | |/ / | |/| |
* | | | Merge topic 'wizard-cleanup'Brad King2013-05-221-7/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | eca523f wizard: simplify control flow f1d27bf wizard: fix warnings
| * | | | wizard: simplify control flowRolf Eike Beer2013-05-221-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Checking if a byte is 0 before checking it for something else doesn't do anything useful. Also one can be sure that it can't be anything wanted if the value was set to 0 immediately before.
| * | | | wizard: fix warningsRolf Eike Beer2013-05-211-2/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | .../Source/cmakewizard.cxx: In member function ‘virtual void cmakewizard::AskUser(const char*, cmCacheManager::CacheIterator&)’: .../Source/cmakewizard.cxx:31:35: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion] if(!fgets(buffer, sizeof(buffer)-1, stdin)) ^ .../Source/cmakewizard.cxx: In member function ‘virtual bool cmakewizard::AskAdvanced()’: .../Source/cmakewizard.cxx:70:35: warning: conversion to ‘int’ from ‘long unsigned int’ may alter its value [-Wconversion] if(!fgets(buffer, sizeof(buffer)-1, stdin)) The compiler is (partly) right here: sizeof() returns a size_t, which often is bigger as as int as it is unsigned long or something similar. Add an explicit cast to get rid of that warning here, the buffer has a size of 4KiB, so the value will fit into an int on all sane platforms.
* | | | Merge topic 'Memchecker-AIX-fix'Brad King2013-05-221-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 1a2a6ba Tests: fix build of dummy memtester on AIX
| * | | | Tests: fix build of dummy memtester on AIXRolf Eike Beer2013-05-211-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | Both <string> and <cmSystemTools.h> include <stdio.h> one or the other way. It looks like the CMake header adds a define that will cause another part of the system header to be used, resulting in conflicting definitions.
* | | | Merge topic 'ctest-parallel-env'Brad King2013-05-223-1/+14
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | db43502 CTest: Read CTEST_PARALLEL_LEVEL from environment
| * | | | CTest: Read CTEST_PARALLEL_LEVEL from environmentStephen Kelly2013-05-213-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | | If no explicit "-j <n>" option is given on the command line then read the parallel level from an environment variable.
* | | | | Merge topic 'string-MAKE_C_IDENTIFIER-subcommand'Brad King2013-05-223-1/+33
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 0ab50ae string: Add MAKE_C_IDENTIFIER subcommand
| * | | | | string: Add MAKE_C_IDENTIFIER subcommandStephen Kelly2013-05-213-1/+33
| | | | | |
* | | | | | Merge topic 'add-EXPORT_NAME-property'Brad King2013-05-2214-17/+122
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | b5d6f5d Add EXPORT_NAME property.
| * | | | | | Add EXPORT_NAME property.Stephen Kelly2013-05-1814-17/+122
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows for example, the buildsystem to use names like 'boost_any' instead of the overly generic 'any', and still be able to generate IMPORTED targets called 'boost::any'.
* | | | | | CMake Nightly Date StampKitware Robot2013-05-221-1/+1
| |_|/ / / |/| | | |
* | | | | Merge topic 'fix-include_directories-whitespace-handling'Brad King2013-05-213-2/+25
|\ \ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | 2f84dfe include_directories: Fix handling of empty or space-only entries
| * | | | include_directories: Fix handling of empty or space-only entriesStephen Kelly2013-05-213-2/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a regression introduced in commit 0d46e9a0 (Store includes from the same include_directories call together., 2013-01-20). Reported at http://thread.gmane.org/gmane.comp.programming.tools.cmake.user/46695/focus=7847
* | | | | Merge topic 'fix-genex-comma-parse'Brad King2013-05-213-5/+24
|\ \ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | 05bf972 Fix handling of commas in arbitrary content in genexes.
| * | | | Fix handling of commas in arbitrary content in genexes.Stephen Kelly2013-05-163-5/+24
| | |_|/ | |/| | | | | | | | | | | | | | As the comma is the parameter separator, it needs to be re-added when evaluating to reconstruct arbitrary content.
* | | | Merge topic 'unused-cleanup'Brad King2013-05-213-4/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 994dc70 Remove unused vector population. 44258cc Remove unused typedef. 0a4d7d8 Remove unused marker for a variable which is now used.
| * | | | Remove unused vector population.Stephen Kelly2013-05-201-2/+0
| | | | | | | | | | | | | | | | | | | | The call to GetEnabledLanguages does not have side-effects.
| * | | | Remove unused typedef.Stephen Kelly2013-05-171-1/+0
| | | | |
| * | | | Remove unused marker for a variable which is now used.Stephen Kelly2013-05-161-1/+0
| |/ / /
* | | | Merge topic 'vs-asm-listing-location'Brad King2013-05-212-4/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | 7e0c45e VS: Allow /Fa to set AssemblerListingLocation (#14153)
| * | | | VS: Allow /Fa to set AssemblerListingLocation (#14153)Brad King2013-05-202-4/+7
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Generate the default AssemblerListingLocation through the flag map so that it can be overridden by a user /Fa flag. Also teach the VS 7-9 generators to map /Fa to AssemblerListingLocation. While at it, fix the AssemblerListingLocation default value to have a trailing slash after the configuration name. This ensures it will be treated as a directory and not a file name.