Werner,

Fair point. I'll look at it later.

Andrew

On Tue, May 18, 2010 at 10:53:15AM +0200, Werner Smekal wrote:
> Hi Andrew,
> 
> wouldn't it be better, if we disable python if numpy is not found and
> print a message, that you need to set an option (WANT_NUMERIC or
> similar) so that cmake tests for numeric and enable python if available
> (and skip the numpy check)? In that case people who use Numeric will
> definitely see, that they are in trouble soon, a mere warning message I
> think is just not enough, since you look in the cmake output only if
> something doesn't get configured as you want to.
> 
> Regards,
> Werner
> 
> -------- Original Message --------
> Subject: [Plplot-cvs] SF.net SVN: plplot:[11004]
> trunk/cmake/modules/python.cmake
> Date: Tue, 18 May 2010 07:47:37 +0000
> From: andrewr...@users.sourceforge.net
> To: plplot-...@lists.sourceforge.net
> 
> Revision: 11004
>           http://plplot.svn.sourceforge.net/plplot/?rev=11004&view=rev
> Author:   andrewross
> Date:     2010-05-18 07:47:37 +0000 (Tue, 18 May 2010)
> 
> Log Message:
> -----------
> Add warning that Numeric support is deprecated and will be dropped in a
> future release of plplot.
> 
> Modified Paths:
> --------------
>     trunk/cmake/modules/python.cmake
> 
> Modified: trunk/cmake/modules/python.cmake
> ===================================================================
> --- trunk/cmake/modules/python.cmake  2010-05-17 21:10:22 UTC (rev 11003)
> +++ trunk/cmake/modules/python.cmake  2010-05-18 07:47:37 UTC (rev 11004)
> @@ -135,3 +135,7 @@
>      OUTPUT_STRIP_TRAILING_WHITESPACE
>      )
>  endif(ENABLE_python)
> +
> +if(ENABLE_python AND NOT HAVE_NUMPY)
> +  message(STATUS "WARNING: " "The Numeric extensions for python are
> deprecated. Support for Numeric will be dropped in a future plplot
> release. Please switch to numpy.")
> +endif(ENABLE_python AND NOT HAVE_NUMPY)
> 
> 
> This was sent by the SourceForge.net collaborative development platform,
> the world's largest Open Source development site.
> 
> ------------------------------------------------------------------------------
> 
> _______________________________________________
> Plplot-cvs mailing list
> plplot-...@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-cvs
> 
> ------------------------------------------------------------------------------
> 
> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel
> 

------------------------------------------------------------------------------

_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to