On 2016-07-21 14:25-0600 Orion Poplawski wrote:

> Re:
>
> commit 893625ca34ed22f4d8941f9902aa71bf30eaf8fb
> Author: Alan W. Irwin <air...@users.sourceforge.net>
> Date:   Sat Oct 24 13:30:10 2015 -0700
>
>    Build system: Disable OCAML_HAS_CAIRO
>
>    I have taken this step because substantial maintenance will be
>    required before this component of PLplot will configure, build, or run
>    again because of backwards incompatibilities that have been introduced
>    in ocaml support of cairo. For example, the package name "cairo2" no
>    longer exists any more and should probably be replaced by the package
>    name cairo, but that is just the start of the backwards
>    incompatibilities.
>
> diff --git a/cmake/modules/ocaml.cmake b/cmake/modules/ocaml.cmake
> index 0403abd..a106ed2 100644
> --- a/cmake/modules/ocaml.cmake
> +++ b/cmake/modules/ocaml.cmake
> @@ -2,7 +2,7 @@
> #
> # Copyright (C) 2008 Andrew Ross
> # Copyright (C) 2009 Hezekiah M. Carty
> -# Copyright (C) 2009-2014 Alan W. Irwin
> +# Copyright (C) 2009-2015 Alan W. Irwin
> #
> # This file is part of PLplot.
> #
> @@ -196,6 +196,9 @@ if(ENABLE_ocaml)
>   if(OCAMLFIND)
>     if(PLD_extcairo)
>       option(OCAML_HAS_CAIRO "OCaml has the cairo package" ON)
> +      # Disable since substantial maintenance is required before this 
> component
> +      # of PLplot will configure, build, and/or run.
> +      set(OCAML_HAS_CAIRO OFF CACHE BOOL "OCaml has the cairo package" FORCE)
>       if(OCAML_HAS_CAIRO)
>        set(text_cairo "module C = Cairo")
>        file(WRITE ${CMAKE_BINARY_DIR}/test_cairo.ml ${text_cairo})
>
>
> It would be nice if the message was changed to something like "plplot ocaml
> cairo support currently disabled", so avoid having people like me spend time
> figuring out why it's not finding the cairo package.  Thanks.

Hi Orion:

I don't want to clutter up the usual stdout from cmake with warning
messages concerning development issues that ordinary users can do nothing
about. So this is a perfect case where the CMake message option AUTHOR_WARNING
should be used.  The result is the messages go to stderr and can be
turned off by ordinary users if they use the CMake -Wno-dev option.

So I have used message(AUTHOR_WARNING ....) for this case (commit
56f415a) and plan to do that from now on when other components of
PLplot need to be permanently disabled.  I hope this method will
completely satisfy your future packaging needs.

Alan
__________________________
Alan W. Irwin

Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).

Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.sf.net); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________

Linux-powered Science
__________________________

------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity planning
reports.http://sdm.link/zohodev2dev
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to