Added a test for conformance of python scripts

Makefile

@@ -59,6 +59,7 @@ dist:
 		$(PACKAGE)-$(VERSION)/src/engines/Makefile \
 		$(PACKAGE)-$(VERSION)/src/engines/project.conf \
 		$(PACKAGE)-$(VERSION)/tests/Makefile \
+		$(PACKAGE)-$(VERSION)/tests/pylint.sh \
 		$(PACKAGE)-$(VERSION)/tests/python.sh \
 		$(PACKAGE)-$(VERSION)/tests/tests.sh \
 		$(PACKAGE)-$(VERSION)/tests/project.conf \

tests/Makefile

@@ -1,4 +1,4 @@
-TARGETS	= $(OBJDIR)tests.log
+TARGETS	= $(OBJDIR)pylint.log $(OBJDIR)tests.log
 OBJDIR	=
 PREFIX	= /usr/local
 DESTDIR	=
@@ -10,11 +10,15 @@ INSTALL	= install
 
 all: $(TARGETS)
 
+$(OBJDIR)pylint.log: pylint.sh
+	./pylint.sh -P "$(PREFIX)" -- "$(OBJDIR)pylint.log"
+
 $(OBJDIR)tests.log: python.sh tests.sh
 	./tests.sh -P "$(PREFIX)" -- "$(OBJDIR)tests.log"
 
 clean:
-	$(RM) -- $(tests.log_OBJS)
+	$(RM) -- $(pylint.log_OBJS) $(tests.log_OBJS)
+	./pylint.sh -c -P "$(PREFIX)" -- "pylint.log"
 	./tests.sh -c -P "$(PREFIX)" -- "tests.log"
 
 distclean: clean

tests/project.conf

@@ -1,5 +1,10 @@
-targets=tests.log
-dist=Makefile,python.sh,tests.sh
+targets=pylint.log,tests.log
+dist=Makefile,pylint.sh,python.sh,tests.sh
+
+[pylint.log]
+type=script
+script=./pylint.sh
+depends=pylint.sh
 
 [tests.log]
 type=script

tests/pylint.sh

@@ -0,0 +1,101 @@
+#!/bin/sh
+#$Id$
+#Copyright (c) 2014 Pierre Pronchery <khorben@defora.org>
+#
+#Redistribution and use in source and binary forms, with or without
+#modification, are permitted provided that the following conditions are met:
+#
+# * Redistributions of source code must retain the above copyright notice, this
+#   list of conditions and the following disclaimer.
+# * Redistributions in binary form must reproduce the above copyright notice,
+#   this list of conditions and the following disclaimer in the documentation
+#   and/or other materials provided with the distribution.
+#
+#THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+#AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+#IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+#DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+#FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+#DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+#SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+#CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+#OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+#OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+
+
+#variables
+PROGNAME="pylint.sh"
+#executables
+DEBUG="_debug"
+FIND="find"
+PYLINT="pep8"
+
+
+#functions
+#pylint
+_pylint()
+{
+	filename="$1"
+
+	#XXX ignores errors
+	$DEBUG $PYLINT -- "$filename" 2>&1 | while read line; do
+		echo "$line" 1>&2
+		echo "$line"
+	done
+}
+
+
+#debug
+_debug()
+{
+	echo "$@" 1>&2
+	"$@"
+	res=$?
+	#ignore errors when the command is not available
+	[ $res -eq 127 ]					&& return 0
+	return $res
+}
+
+
+#usage
+_usage()
+{
+	echo "Usage: $PROGNAME [-c] target" 1>&2
+	return 1
+}
+
+
+#main
+clean=0
+while getopts "cP:" name; do
+	case "$name" in
+		c)
+			clean=1
+			;;
+		P)
+			#XXX ignored for compatibility
+			;;
+		?)
+			_usage
+			exit $?
+			;;
+	esac
+done
+shift $((OPTIND - 1))
+if [ $# -ne 1 ]; then
+	_usage
+	exit $?
+fi
+target="$1"
+
+#clean
+[ $clean -ne 0 ] && return 0
+
+ret=0
+(date
+echo
+$FIND "../doc" "../src" "../tests" "../tools" -name '*.py' | while read filename; do
+	_pylint "$filename"
+done) > "$target"
+exit $ret