summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Tests: Eliminate unnecessary files and variables.David Cole2011-01-2817-169/+0
| | | | | | | | | | | | | | | | | | | | | 1: The CTestTestFailedSubmits tests do not need to run the CTEST_MEMCHECK command. It is covered by the CTestTest2 test. If more coverage *is* required, choose a test that runs by itself once to add it to... This one gets used several times in a loop to test submission failure via the various submission protocols. With memchecking on, each test here takes 1000 seconds or more on a "fast" build... 2: Other ctest tests do not need any of the CTEST_MEMORYCHECK_* variables defined. They don't even call CTEST_MEMCHECK, or -D NightlyMemCheck, or anything. So: remove the unnecessary set calls. 3: CTestTest3 has gone intentionally unused for months because it did not add any coverage and caused no ends of sporadic randomish failures when network connectivity was spotty. Remove its directory. Never add it again.
* KWSys Nightly Date StampKWSys Robot2011-01-281-1/+1
|
* Merge topic 'dev/strict-mode'Brad King2011-01-2727-18/+621
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dd2f814 Merge branch 'dev/add_test-working-directory' into dev/strict-mode 949d32c Unwatch manual variables upon removal in cmake-gui 3939032 Unwatch manual variables upon removal in ccmake 8354413 Add method to unwatch a manual variable 8ed3c85 Give a better message for unused variables 729db48 Fix ArgumentExpansion test expected results 89c2544 Checking for a definition is a usage 5625dee Don't output to stderr in the GUI ad25a96 Merge branch 'ImprovedDotSupport2' into dev/strict-mode c128abe Merge branch 'AddCMAKE_CURRENT_LIST_DIR' into dev/strict-mode 9bcaff0 Merge branch 'cmake-guiRememberAdvancedCheckbox' into dev/strict-mode 544d0c3 Fix expected output for WarnUninitialized test 4e3bea4 Update expected messages to new format 8e8c9e4 Don't check at destruction for usage 668e005 Use cmake::IssueMessage for warnings 88cd4c1 Use 'CMake Warning' versus 'warning' for CDash 3c3b98d Initialize the class before setting warn flags cf8b15a Ignore files under the CMakeFiles directory fd50f06 Don't check for unused vars at configure time 447a04c Don't warn during configure when doing everything b97ee21 Check for unused variables at the end of generate c18c977 When checking for variables, specify a reason 3f1121f Use a long int since Line is a long as well 2507f93 Change the failure case string to 'Unexpected' fe390a2 Add 'ArgumentExpansion' test 8dbb209 Wrong boolean value for CLI warnings d4ee998 Hard-code the --no-warn-unused-cli flag a267b99 Fix line lengths 82ed104 Flag that the directories have been set 5aa535b Add argument to arg parsing to not set directories 367e5c3 Revert "Revert "When calling CMake, set the args and the cache"" ab5d4e4 Revert "When calling CMake, set the args and the cache" 9b90040 When calling CMake, set the args and the cache fe56002 Fix long lines for KWStyle 5d30cfc Set a watch on variables added through the gui 33c63b1 Add a method to put a watch for variables 535253f Initialize the warning variables earlier cbb286c Fix the path detection to work for top-level 62be1f7 Initialize the usage stack earlier c6e7fab Factor out the checks for unused variables 5e41ba8 When using the API, check for Add vs. Remove dee1976 Fix typo in VariableUnusedViaUnset test f231ce5 Remove old false positive avoidance code a117e02 Revert "Add test for unused warnings at the end of scope" 2c82f2b Exempt CMAKE(CURRENT|PARENT)_LIST_FILE from usage 6d7d449 Ignore CLI warnings for ABI determination 7740a73 Only return local keys that are defined bef3aee Use the API so that warnings can be tracked 05cb0f4 Check for unused variables in the dtor 91c4c99 Add test for unused warnings at the end of scope ca90f67 Fix detection of unused variables when setting f7438ca Add test for unused detection via setting it 995cfb0 Don't warn if the variable wasn't defined aefc91d Add test for usage checks via unset a8e97f8 Remove VarRemoved code since it's been superceded 59463ef Rework CheckVariableForUnused usage f117423 Fix line lengths to be no more than 78 e49a935 Improve unused warning logic e01e40c Mark ARGC, ARGV*, and ARGN as used a17aff7 Ignore CMAKE_MATCH_* variables for usage 02a114d Add method to allow variables to be marked as used a0b0d23 CMAKE_DO_TRY_COMPILE is no longer used ae3eff3 Fix the path used for ignoring system warnings 056b441 Fix missing case for usage of a variable 980e048 Factor out checks for unused variables 83acb0a Remove now unused variables 3801463 Use built-ins for readability and maintainability 8b52015 Push the initialize and unused states when copying 439877f Be consistent with single and double quotes 4cf1706 Add documentation for check-system-vars b74777f Fix the spelling of the flag for warn-unused-vars b948120 Change logic of flag to turn off cli unused checks f047a17 Add test for uninitialized variables 75bda38 Add tests for unused command line variables 300fc15 Fix detection of system files d784e6a Run the unused variables check on the final pass 9efc057 VariableWatch is not available when bootstrapping 2e78224 Add a missing comma to the warning message 7499700 Add a flag to warn about system files fff9f6d Rename flags again and use variablewatch for cli 786e269 Add warn-unused to the Qt interface 636e6c4 Default to marking things as used 4ff0340 Rename find-unused to warn-unused d7999e9 Rename strict-mode to warn-uninitialized e141bc9 Detect unused variables d3e8eb5 Add flags to detect unused variables f332e14 Complete strict-mode checks for uninitialized vars 52f9637 Add method to get the local scope variables f794d58 Make --strict-mode option, and integrate with cmake-gui 48b5b85 Add a warning when variables are used uninitialized. cd626ea For macros make sure the FilePath points to a valid pointer in the args.
| * Merge branch 'dev/add_test-working-directory' into dev/strict-modeBrad King2011-01-27157-1419/+2846
| |\ | | | | | | | | | | | | Conflicts: Tests/CMakeLists.txt
| * | Unwatch manual variables upon removal in cmake-guiBen Boeckel2011-01-131-0/+2
| | |
| * | Unwatch manual variables upon removal in ccmakeBen Boeckel2011-01-131-0/+1
| | |
| * | Add method to unwatch a manual variableBen Boeckel2011-01-132-0/+9
| | |
| * | Give a better message for unused variablesBen Boeckel2011-01-131-1/+1
| | |
| * | Fix ArgumentExpansion test expected resultsBrad King2011-01-113-12/+31
| | | | | | | | | | | | | | | | | | | | | | | | Teach the ArgumentExpansion test to expect flattened lists as has always been the case in the CMake language. Now that the test should pass enable the failure regex even when CMAKE_STRICT is not on. Replace the reference to the old ArgumentExpansion test behavior in the workaround comment in cmMakefile::TryCompile with a full inline explanation.
| * | Checking for a definition is a usageBen Boeckel2011-01-031-0/+1
| | |
| * | Don't output to stderr in the GUIBen Boeckel2011-01-031-1/+0
| | |
| * | Merge branch 'ImprovedDotSupport2' into dev/strict-modeBrad King2010-12-1740-649/+1210
| |\ \ | | | | | | | | | | | | | | | | Conflicts: Source/cmake.cxx
| * \ \ Merge branch 'AddCMAKE_CURRENT_LIST_DIR' into dev/strict-modeBrad King2010-12-1795-164/+213
| |\ \ \ | | | | | | | | | | | | | | | | | | | | Conflicts: Source/cmMakefile.cxx
| * \ \ \ Merge branch 'cmake-guiRememberAdvancedCheckbox' into dev/strict-modeBrad King2010-12-17143-1743/+3161
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: Source/QtDialog/CMakeSetupDialog.cxx
| * | | | | Fix expected output for WarnUninitialized testBen Boeckel2010-12-071-1/+1
| | | | | |
| * | | | | Update expected messages to new formatBen Boeckel2010-12-071-4/+4
| | | | | |
| * | | | | Don't check at destruction for usageBen Boeckel2010-12-071-3/+0
| | | | | |
| * | | | | Use cmake::IssueMessage for warningsBen Boeckel2010-12-072-9/+18
| | | | | |
| * | | | | Use 'CMake Warning' versus 'warning' for CDashBen Boeckel2010-12-074-10/+10
| | | | | |
| * | | | | Initialize the class before setting warn flagsBen Boeckel2010-12-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since Initialize sets variables that we don't want to warn about, don't leak the original class' settings.
| * | | | | Ignore files under the CMakeFiles directoryBen Boeckel2010-12-031-2/+4
| | | | | |
| * | | | | Don't check for unused vars at configure timeBen Boeckel2010-12-021-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | The generate step should catch all of them.
| * | | | | Don't warn during configure when doing everythingBen Boeckel2010-11-151-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This prevents warnings from being generated after configure *and* after generation if both are going to be run anyways.
| * | | | | Check for unused variables at the end of generateBen Boeckel2010-11-151-0/+2
| | | | | |
| * | | | | When checking for variables, specify a reasonBen Boeckel2010-11-153-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | Allow reasons to begiven for checking for unused variables.
| * | | | | Use a long int since Line is a long as wellBen Boeckel2010-10-081-1/+1
| | | | | |
| * | | | | Change the failure case string to 'Unexpected'Ben Boeckel2010-10-082-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | VS6 detects the 'Error' string and fails itself even though we don't actually care about it unless CMAKE_STRICT is on.
| * | | | | Add 'ArgumentExpansion' testBen Boeckel2010-10-072-0/+74
| | | | | |
| * | | | | Wrong boolean value for CLI warningsBen Boeckel2010-10-071-1/+1
| | | | | |
| * | | | | Hard-code the --no-warn-unused-cli flagBen Boeckel2010-10-071-1/+5
| | | | | |
| * | | | | Fix line lengthsBen Boeckel2010-10-062-2/+4
| | | | | |
| * | | | | Flag that the directories have been setBen Boeckel2010-10-051-1/+1
| | | | | |
| * | | | | Add argument to arg parsing to not set directoriesBen Boeckel2010-10-052-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Argument parsing sets the source/build directories, but they may have been (meaningfully) set before hand. Let's not overwrite them.
| * | | | | Revert "Revert "When calling CMake, set the args and the cache""Ben Boeckel2010-10-051-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ab5d4e43d9a9ddde24a242092a0d5e5f9a6cbd01.
| * | | | | Revert "When calling CMake, set the args and the cache"Ben Boeckel2010-10-011-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9b90040edba1f89395295c27cd9919b0d5efd30b.
| * | | | | When calling CMake, set the args and the cacheBen Boeckel2010-10-011-0/+1
| | | | | |
| * | | | | Fix long lines for KWStyleBen Boeckel2010-09-232-4/+5
| | | | | |
| * | | | | Set a watch on variables added through the guiBen Boeckel2010-09-221-0/+2
| | | | | |
| * | | | | Add a method to put a watch for variablesBen Boeckel2010-09-222-4/+10
| | | | | |
| * | | | | Initialize the warning variables earlierBen Boeckel2010-09-221-2/+4
| | | | | |
| * | | | | Fix the path detection to work for top-levelBen Boeckel2010-09-221-5/+18
| | | | | |
| * | | | | Initialize the usage stack earlierBen Boeckel2010-09-221-14/+2
| | | | | |
| * | | | | Factor out the checks for unused variablesBen Boeckel2010-09-222-9/+19
| | | | | |
| * | | | | When using the API, check for Add vs. RemoveBen Boeckel2010-09-171-1/+8
| | | | | |
| * | | | | Fix typo in VariableUnusedViaUnset testBen Boeckel2010-09-171-1/+1
| | | | | |
| * | | | | Remove old false positive avoidance codeBen Boeckel2010-09-171-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | From email explaining existence in the first place: This is from before when the used checks throwing false positives about unused due to changing the definition without checking whether it *had* a value to begin with and me not realizing they were false positives. I was thinking that it was warning from ARGC et. al. not being used since the previous macro or function call and the new value warning about overwriting the old value.
| * | | | | Revert "Add test for unused warnings at the end of scope"Ben Boeckel2010-09-172-17/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 91c4c9921c40361df860c1384a46ed77c6118c23. The test doesn't work yet and should not have been put on this branch yet.
| * | | | | Exempt CMAKE(CURRENT|PARENT)_LIST_FILE from usageBen Boeckel2010-09-161-0/+6
| | | | | |
| * | | | | Ignore CLI warnings for ABI determinationBen Boeckel2010-09-161-0/+4
| | | | | |
| * | | | | Only return local keys that are definedBen Boeckel2010-09-161-1/+4
| | | | | |