DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;
	h=from:to:subject:in-reply-to:references:user-agent:date:message-id
	:mime-version:content-type;
	b=XSt4MJsXudFWN7MLHTceh3apNwfHy9Pomw6Xso/ATve9MxOy/esKu4zwwqIuhdB7pb
	ReLfcZWebrQG0X2wYyGXWSg9l6PxUCjBm8HEsYM3C8btQcWt4PItEIf1wbrfV5jIx8Gy
	68ISJn/xcCtAjV4eAd8HNi+2RfUYBgQ4auJN0=
From: Felix Geller <fgeller@gmail.com>
To: Carl Worth <cworth@cworth.org>, notmuch@notmuchmail.org
Subject: Re: [PATCH] Change in increment_mtime for BSD compatibility of test
	suite
In-Reply-To: <87boypcdrw.fsf@yoom.home.cworth.org>
References: <m2sjs3q66v.fsf@gmail.com> <87boypcdrw.fsf@yoom.home.cworth.org>
User-Agent: Emacs/23.3.1 (x86_64-apple-darwin10.7.0)
Date: Mon, 30 May 2011 14:40:04 +0200
Message-ID: <m21uzgfjkr.fsf@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha1; protocol="application/pgp-signature"
Precedence: list

--=-=-=
Content-Transfer-Encoding: quoted-printable

Another try :)

[...]

    Support for platform-specific test configuration
     - Platform-specific functionality is stored in test-config-PLATFORM.sh=
 files
     - configure script creates a test/test-config.sh depending on the plat=
form
     - test-lib.sh loads test-config.sh file
     - Some platform-specific functionality included for gnu/bsd:
       - Work-around for touch -d on BSD
       - Variable to store sed command for extended expressions

diff --git a/configure b/configure
index bbf30cd..053a4a1 100755
=2D-- a/configure
+++ b/configure
@@ -16,6 +16,13 @@ if [ "$srcdir" !=3D "." ]; then
     # whole thing into the build directory.
     cp -a "$srcdir"/test/* test
=20
+    # Platform specific test configuration
+    if [ $platform =3D MACOSX ] ; then
+	cp test/test-config-bsd.sh test/test-config.sh
+    else
+	cp test/test-config-gnu.sh test/test-config.sh
+    fi
+
     # Emacs only likes to generate compiled files next to the .el files
     # by default so copy these as well (which is not ideal0.
     cp -a "$srcdir"/emacs/*.el emacs
diff --git a/test/basic b/test/basic
index 808c968..5fbedfe 100755
=2D-- a/test/basic
+++ b/test/basic
@@ -54,13 +54,14 @@ test_begin_subtest 'Ensure that all available tests wil=
l be run by notmuch-test'
 eval $(sed -n -e '/^TESTS=3D"$/,/^"$/p' notmuch-test ../notmuch-test)
 tests_in_suite=3D$(for i in $TESTS; do echo $i; done | sort)
 available=3D$(ls -1 ../ | \
=2D    sed -r -e "/^(aggregate-results.sh|Makefile|Makefile.local|notmuch-t=
est)/d" \
=2D	   -e "/^(README|test-lib.sh|test-lib.el|test-results|tmp.*|valgrind|co=
rpus*)/d" \
=2D	   -e "/^(emacs.expected-output|smtp-dummy|smtp-dummy.c|test-verbose)/d=
" \
=2D	   -e "/^(test.expected-output|.*~)/d" \
=2D	   -e "/^(gnupg-secret-key.asc)/d" \
=2D	   -e "/^(gnupg-secret-key.NOTE)/d" \
=2D	   | sort)
+    $SED_EXTENDED -e "/^(aggregate-results.sh|Makefile|Makefile.local|notm=
uch-test)/d" \
+    		  -e "/^(README|test-lib.sh|test-lib.el|test-results|tmp.*|valgrind|=
corpus*)/d" \
+    		  -e "/^(emacs.expected-output|smtp-dummy|smtp-dummy.c|test-verbose)=
/d" \
+	   	  -e "/^(test.expected-output|.*~)/d" \
+	   	  -e "/^(gnupg-secret-key.asc)/d" \
+	   	  -e "/^(gnupg-secret-key.NOTE)/d" \
+	   	  -e "/^(test-config*)/d" \
+	   	  | sort)
 test_expect_equal "$tests_in_suite" "$available"
=20
 EXPECTED=3D../test.expected-output
diff --git a/test/test-config-bsd.sh b/test/test-config-bsd.sh
new file mode 100644
index 0000000..e35d2fa
=2D-- /dev/null
+++ b/test/test-config-bsd.sh
@@ -0,0 +1,23 @@
+# This file contains helper functions and other values that are
+# require platform-specific functionality (e.g., differing between GNU
+# and BSD). The configure script is used to identify the appropriate
+# file for a given platform, and copies the file test-config-FOO.sh to
+# test-config.sh, where FOO is the respective platform (bsd or gnu).
+#
+# This file is BSD-specific.
+
+# Syntax for extended expressions
+SED_EXTENDED=3D"sed -E"
+
+# There is no touch -d on BSD, therefore we have to use a more tedious
+# version that uses date/stat to increment a date by a single second.
+increment_mtime_amount=3D0
+increment_mtime ()
+{
+    dir=3D"$1"
+=20=20=20=20
+    last_mod_date=3D`date -j -f %Y%m%d%H%M%S \`stat -f %Sm -t %Y%m%d%H%M%S=
 ${dir}\` +%s`
+    increment_mtime_amount=3D$((increment_mtime_amount + 1))
+    new_date=3D`date -j -r ${last_mod_date} -v+${increment_mtime_amount}S =
+%Y%m%d%H%M.%S`
+    touch -t ${new_date} ${dir}
+}
diff --git a/test/test-config-gnu.sh b/test/test-config-gnu.sh
new file mode 100644
index 0000000..596505a
=2D-- /dev/null
+++ b/test/test-config-gnu.sh
@@ -0,0 +1,20 @@
+# This file contains helper functions and other values that are
+# require platform-specific functionality (e.g., differing between GNU
+# and BSD). The configure script is used to identify the appropriate
+# file for a given platform, and copies the file test-config-FOO.sh to
+# test-config.sh, where FOO is the respective platform (bsd or gnu).
+#
+# This file is GNU-specific.
+
+# Syntax for extended expressions
+SED_EXTENDED=3D"sed -r"
+
+# Use touch to increment the modification date by a single second.
+increment_mtime_amount=3D0
+increment_mtime ()
+{
+    dir=3D"$1"
+
+    increment_mtime_amount=3D$((increment_mtime_amount + 1))
+    touch -d "+${increment_mtime_amount} seconds" "$dir"
+}
diff --git a/test/test-lib.sh b/test/test-lib.sh
index 922b1ef..1aa3a1c 100755
=2D-- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -23,6 +23,9 @@ if [ ${BASH_VERSINFO[0]} -lt 4 ]; then
     exit 1
 fi
=20
+# Load platform-specific values
+. ./test-config.sh
+
 # if --tee was passed, write the output not only to the terminal, but
 # additionally to the file test-results/$BASENAME.out, too.
 case "$GIT_TEST_TEE_STARTED, $* " in
@@ -214,14 +217,6 @@ remove_cr () {
 }
=20
 # Notmuch helper functions
=2Dincrement_mtime_amount=3D0
=2Dincrement_mtime ()
=2D{
=2D    dir=3D"$1"
=2D
=2D    increment_mtime_amount=3D$((increment_mtime_amount + 1))
=2D    touch -d "+${increment_mtime_amount} seconds" "$dir"
=2D}
=20
 # Generate a new message in the mail directory, with a unique message
 # ID and subject. The message is not added to the index.
@@ -514,7 +509,7 @@ NOTMUCH_NEW ()
=20
 notmuch_search_sanitize ()
 {
=2D    sed -r -e 's/("?thread"?: ?)("?)................("?)/\1\2XXX\3/'
+    $SED_EXTENDED 's/("?thread"?: ?)("?)................("?)/\1\2XXX\3/'
 }
=20
 NOTMUCH_SHOW_FILENAME_SQUELCH=3D's,filename:.*/mail,filename:/XXX/mail,'

--=-=-=
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG/MacGPG2 v2.0.17 (Darwin)

iEYEARECAAYFAk3jkCQACgkQMxgKfMb9y5amNACfU35yBvIF2S0VB8Tx9u0MWJqN
ij4AoLmczzcV6N67MnTcqKIp4dO8SAQt
=n/ID
-----END PGP SIGNATURE-----
--=-=-=--