Module: check_mk
Branch: master
Commit: fd283e35e8fba04b4c48b036c6ec082d16bd7c97
URL:
http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=fd283e35e8fba0…
Author: Lars Michelsen <lm(a)mathias-kettner.de>
Date: Wed Nov 10 22:41:56 2010 +0100
check validation: Fixed typo
---
doc/helpers/validate_checks | 22 +++++++++++++---------
1 files changed, 13 insertions(+), 9 deletions(-)
diff --git a/doc/helpers/validate_checks b/doc/helpers/validate_checks
index 91c5577..d6cd425 100755
--- a/doc/helpers/validate_checks
+++ b/doc/helpers/validate_checks
@@ -93,15 +93,15 @@ def grep_manpage(check, match, section = None):
if section is None:
return not re.search(match, manpage) is None
else:
- started = False
- for line in manpage.split("\n"):
+ sec_started = False
+ for line in [ l.strip() for l in manpage.split("\n") ]:
if line == '[%s]' % section:
# Block starts with this line
- started = True
- elif started and line == '' or line.startswith('['):
+ sec_started = True
+ elif sec_started == True and (line == '' or
line.startswith('[')):
# Reached nex section. Didn't find a match ... so return false
return False
- elif started:
+ elif sec_started == True:
if not re.search(match, line) is None:
return True
return False
@@ -166,10 +166,14 @@ def verify_global_vars(check):
if not var.islower():
invalid = True
- # Filter out known config vars
- # And check for correct prefix
- if not invalid and not grep_manpage(check, "^%s" % var,
'configuration') \
- and not var.startswith('%s_' % check_file):
+ # Documented configuration vars can be skipped
+ if not invalid and grep_manpage(check, "^%s" % var,
'configuration'):
+ continue
+ elif check == 'aironet_clients':
+ print 'found'
+
+ # Check unknown vars for correct prefix
+ if not var.startswith('%s_' % check_file):
invalid = True
if invalid: