added more tests

BKrev: 415a81e9YfgUPHbJ9oIk7b_9nclVhw

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")
+	
+}