Re: [Monetdb-checkins] MonetDB4/src/mapi/Tests All, , 1.2, 1.3

View: New views
1 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: [Monetdb-checkins] MonetDB4/src/mapi/Tests All, , 1.2, 1.3

by Stefan Manegold :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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@...
> https://lists.sourceforge.net/lists/listinfo/monetdb-checkins
>

--
| Dr. Stefan Manegold | mailto:Stefan.Manegold@... |
| CWI,  P.O.Box 94079 | http://www.cwi.nl/~manegold/  |
| 1090 GB Amsterdam   | Tel.: +31 (20) 592-4212       |
| The Netherlands     | Fax : +31 (20) 592-4312       |

-------------------------------------------------------------------------
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-developers mailing list
Monetdb-developers@...
https://lists.sourceforge.net/lists/listinfo/monetdb-developers