On Fri, Jul 18, 2008 at 11:46:19AM +0000, Sjoerd Mullender wrote:

Update of /cvsroot/monetdb/MonetDB4/src/mapi/Tests In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv19788

Modified Files: All Log Message: Test can only be run if compiled with HAVE_PYTHON (else Mapi.py does not get installed).

This reminds me of a "minor"(?) flaw in Mtest.py: Mapi.py gets installed in case clients was configured with python avialable, but Mtest.py set "HAVE_PYTHON" if MonetDB was configured with python avialable. Hence, in case clients and MonetDB are not configured equally, the conditional does not reflect the actual status. The same hold for all static/compile-time conditionals currently supported by Mtest.py.

To solve this problem, we could consider deriving all conditionals only at runtime as Mtest.py already does, e.g., for all MonetDB/Java related conditional.

... once we will have some spare time ...

Stefan

U All Index: All =================================================================== RCS file: /cvsroot/monetdb/MonetDB4/src/mapi/Tests/All,v retrieving revision 1.2 retrieving revision 1.3 diff -u -d -r1.2 -r1.3 --- All 12 Jan 2007 17:19:23 -0000 1.2 +++ All 18 Jul 2008 11:46:14 -0000 1.3 @@ -1,3 +1,3 @@ monet_BATs -manyMapiClients +HAVE_PYTHON?manyMapiClients utf_8_C

---

This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Monetdb-checkins mailing list Monetdb-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-checkins