From 02d182799edf1fc95907bdb3455bfe55569ecc5b Mon Sep 17 00:00:00 2001
From: Samuel Thibault <samuel.thibault@ens-lyon.org>
Date: Thu, 27 Nov 2014 16:14:51 +0100
Subject: g-i: Use different pixel threshold for d-i and desktop

d-i doesn't have any clockish thing, and its text interface doesn't
change much while progressing, thus needing a small threshold

The LXDE desktop, on the other hand, has by default a CPU meter and
clock, which needs a bigger threshold.
---
 bin/g-i-installation.sh | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/bin/g-i-installation.sh b/bin/g-i-installation.sh
index 8ffd78b5..96e0ee5a 100644
--- a/bin/g-i-installation.sh
+++ b/bin/g-i-installation.sh
@@ -1026,6 +1026,11 @@ monitor_system() {
 	else
 		TIMEOUT=$3
 	fi
+	if [ -z "$4" ] ; then
+		PIXELDIFF=100
+	else
+		PIXELDIFF=$4
+	fi
 	cd $RESULTS
 	sleep 4	# chosen by fair dice roll
 	hourlimit=16 # hours
@@ -1108,18 +1113,18 @@ monitor_system() {
 			let OLD=NR-$TIMEOUT
 			PRINTF_OLD=$(printf "%06d" $OLD)
 			# test if this screenshot is basically the same as the one $TIMEOUT screenshots ago
-			# 400 pixels difference between to images is tolerated, to ignore updating clocks
+			# $PIXELDIFF pixels difference between to images is tolerated, to ignore updating clocks
 			PIXEL=$(compare -metric AE snapshot_${PRINTF_NR}.png snapshot_${PRINTF_OLD}.png /dev/null 2>&1 || true )
 			# usually this returns an integer, but not always....
 			if [[ "$PIXEL" =~ ^[0-9]+$ ]] ; then
 				echo "$PIXEL pixel difference between snapshot_${PRINTF_NR}.png and snapshot_${PRINTF_OLD}.png"
-				if [ $PIXEL -lt 400 ] ; then
+				if [ $PIXEL -lt $PIXELDIFF ] ; then
 				    SAME=Y
 				    for INTER in $(seq $OLD 10 $NR); do
 					PRINTF_INTER=$(printf "%06d" $INTER)
 					PIXEL=$(compare -metric AE snapshot_${PRINTF_NR}.png snapshot_${PRINTF_INTER}.png /dev/null 2>&1 || true )
 					if [[ "$PIXEL" =~ ^[0-9]+$ ]] ; then
-						if [ $PIXEL -ge 400 ] ; then
+						if [ $PIXEL -ge $PIXELDIFF ] ; then
 							echo "but $PIXEL difference between snapshot_${PRINTF_NR}.png and snapshot_${PRINTF_INTER}.png"
 							SAME=N
 							break
@@ -1320,9 +1325,9 @@ case $NAME in
 						;;
 	*_presentation)	 			monitor_system presentation 10
 						;;
-	debian-edu_*combi-server)		monitor_system install wait4match 3000
+	debian-edu_*combi-server)		monitor_system install wait4match 3000 100
 						;;
-	debian-edu_*wheezy*standalone*)		monitor_system install wait4match 1200
+	debian-edu_*wheezy*standalone*)		monitor_system install wait4match 1200 100
 						;;
 	*)					monitor_system install wait4match
 						;;
@@ -1354,7 +1359,7 @@ case $NAME in
 				*)				let START_TRIGGER=NR+200
 								;;
 			esac
-			monitor_system post_install $START_TRIGGER
+			monitor_system post_install $START_TRIGGER 600 1000
 esac
 cleanup_all
 
-- 
cgit v1.2.3-70-g09d2