diff options
author | Holger Schurig <schurig@mn-solutions.de> | 2004-09-29 09:35:37 +0000 |
---|---|---|
committer | Holger Schurig <schurig@mn-solutions.de> | 2004-09-29 09:35:37 +0000 |
commit | 4bdb5b693911bfd72f19b809a0d770d523cd9cf1 (patch) | |
tree | c83177db7b5128b97a9426286bf95dead2f8b589 /classes/oelint.oeclass | |
parent | e72dfc0dfba0ec60a5039680ebc55dae43937be5 (diff) |
added more tests
BKrev: 415a81e9YfgUPHbJ9oIk7b_9nclVhw
Diffstat (limited to 'classes/oelint.oeclass')
-rw-r--r-- | classes/oelint.oeclass | 138 |
1 files changed, 138 insertions, 0 deletions
diff --git a/classes/oelint.oeclass b/classes/oelint.oeclass index e69de29bb2..d100fc6e4a 100644 --- a/classes/oelint.oeclass +++ b/classes/oelint.oeclass @@ -0,0 +1,138 @@ +addtask lint before do_fetch +do_lint[nostamp] = 1 +python do_lint() { + def testVar(var, explain=None): + try: + s = d[var] + return s["content"] + except KeyError: + oe.error("%s is not set" % var) + if explain: oe.note(explain) + return None + + + ############################## + # Test that DESCRIPTION exists + # + testVar("DESCRIPTION") + + + ############################## + # Test for valid LICENSE + # + s = testVar("LICENSE") + if s=="unknown": + oe.error("LICENSE is not set") + + + ############################## + # Test for valid MAINTAINER + # + s = testVar("MAINTAINER") + if s and s.find("@") == -1: + oe.error("You forgot to put an e-mail address into MAINTAINER") + + + ############################## + # Test for valid SECTION + # + # if Correct section: True section name is valid + # False section name is invalid, no suggestion + # string section name is invalid, better name suggested + # + valid_sections = { + # Current Section Correct section + "applications" : True, + "Applications" : "applications", + "apps" : True, + "audio" : True, + "base" : True, + "console/games" : True, + "console/net" : "network", + "console/network" : True, + "console/utils" : True, + "devel" : True, + "developing" : "devel", + "devel/python" : True, + "dvb" : "multimedia", + "fonts" : True, + "games" : True, + "games/libs" : True, + "gnome/base" : True, + "gnome/libs" : True, + "gpe" : True, + "gpe/libs" : True, + "gui" : False, + "libc" : True, + "libs" : True, + "libs/net" : True, + "multimedia" : True, + "net" : "network", + "NET" : "network", + "network" : True, + "opie/applets" : True, + "opie/applications" : True, + "opie/base" : True, + "opie/codecs" : True, + "opie/decorations" : True, + "opie/fontfactories" : True, + "opie/fonts" : True, + "opie/games" : True, + "opie/help" : True, + "opie/inputmethods" : True, + "opie/libs" : True, + "opie/multimedia" : True, + "opie/pim" : True, + "opie/setting" : True, + "opie/settings" : True, + "opie/Shell" : False, + "opie/styles" : True, + "opie/today" : True, + "scientific" : True, + "tools" : "utils", + "utils" : True, + "x11" : True, + "x11/libs" : True, + "x11/wm" : True, + } + s = testVar("SECTION") + if s: + try: + newsect = valid_sections[s] + if newsect == False: + oe.note("SECTION '%s' is not recommended" % s) + elif newsect != True: + oe.note("SECTION '%s' is not recommended, better use '%s'" % (s, newsect)) + except: + oe.note("SECTION '%s' is not recommended" % s) + + if not s.islower(): + oe.error("SECTION should only use lower case") + + + + + ############################## + # Test for valid PRIORITY + # + valid_priorities = { + "standed" : True, + "required" : True, + "optional" : True, + "extra" : True, + } + s = testVar("PRIORITY") + if s: + try: + newprio = valid_priorities[s] + if newprio == False: + oe.note("PRIORITY '%s' is not recommended" % s) + elif newprio != True: + oe.note("PRIORITY '%s' is not recommended, better use '%s'" % (s, newprio)) + except: + oe.note("PRIORITY '%s' is not recommended" % s) + + if not s.islower(): + oe.error("PRIORITY should only use lower case") + +} |