The disabling of plmeta is mentioned in the release notes for 5.8.0 in 
OLD-README.release, but I agree this is a bit arcane. It also doesn't
mention the need for PL_DEPRECATED.

We could automatically set PL_DEPRECATED on if plmeta is selected. 

We should also add some cmake warnings if people enable unsupported 
devices. I've just checked in a patch to do this.

I also found and fixed another minor cmake bug which came to light when 
plmeta was enabled.

Andrew

On Wed, Jan 04, 2012 at 09:31:55AM +0100, Arjen Markus wrote:
> Hi Andrew, Walter,
> 
> I just checked why Walter got the original error regarding plSetOpt
> in building the plrender utility that led to us including the
> option PL_DEPRECATED and from there on to the other build errors
> that Andrew solved yesterday.
> 
> In the CMake GUI you can turn on the "meta" device driver. This
> driver is not actually supported because of all manner of build issues,
> for which no volunteer has come forward yet. The code is still there
> and you can turn it on.
> 
> However, doing so causes CMake to include building the plrender
> utility and that relies on a few deprecated functions. So you need
> to define PL_DEPRECATED to get beyond this. Turning that flag on
> revealed some issues in the C++ bindings that Andrew has now solved.
> 
> I just checked: PLplot is bult correctly now, even with this flag
> in place.
> 
> Perhaps we should add a warning about the "meta" device?
> 
> Regards,
> 
> Arjen
> 
> On 2012-01-03 11:40, Arjen Markus wrote:
> > Hi Andrew,
> > 
> > On Tue, 3 Jan 2012 10:25:17 +0000
> >   Andrew Ross <andrewr...@users.sourceforge.net> wrote:
> >> Good spot! I can reproduce the problem on Linux as well. 
> >> Looking at the 
> >> output of objdump it turns out the problem is the 
> >> backward compatibility
> >> macros for Alloc2dGrid, Free2dGrid and MinMax2dGrid in 
> >> include/plplot.h.
> >> Since the C++ class uses the plplot function names 
> >> without the pl
> >> prefix (it's not needed as the functions are in the 
> >> namespace of the
> >> class anyway), these macros mess up the C++ functions 
> >> for these 
> >> functions. I've committed a fix which just comments out 
> >> these macros. 
> >> They are only for backwards compatibility and these old 
> >> versions haven't
> >> been used in the examples for a long time. If anyone 
> >> shouts, I'll try 
> >> a more complicated fix to check whether plplot.h is 
> >> being included from
> >> plstream.h, but I don't think this is very robust. Given 
> >> no-one has 
> >> complained (or even noticed) this bug for months I 
> >> suspect it is not
> >> a problem.
> >>
> >> Andrew
> >>
> > 
> > Thanks for fixing this.
> > 
> > I will have to check how it arose (something to do with 
> > plrender and some old driver). I will test your fix on my
> > system and see whether all the issues are gone.
> > 
> > Regards,
> > 
> > Arjen
> > 
> >  
> > 
> > DISCLAIMER: This message is intended exclusively for the addressee(s) and 
> > may contain confidential and privileged information. If you are not the 
> > intended recipient please notify the sender immediately and destroy this 
> > message. Unauthorized use, disclosure or copying of this message is 
> > strictly prohibited.
> > The foundation 'Stichting Deltares', which has its seat at Delft, The 
> > Netherlands, Commercial Registration Number 41146461, is not liable in any 
> > way whatsoever for consequences and/or damages resulting from the improper, 
> > incomplete and untimely dispatch, receipt and/or content of this e-mail.
> > 
> > 
> > 
> > 
> > 
> > ------------------------------------------------------------------------------
> > Write once. Port to many.
> > Get the SDK and tools to simplify cross-platform app development. Create 
> > new or port existing apps to sell to consumers worldwide. Explore the 
> > Intel AppUpSM program developer opportunity. appdeveloper.intel.com/join
> > http://p.sf.net/sfu/intel-appdev
> > _______________________________________________
> > Plplot-devel mailing list
> > Plplot-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/plplot-devel
> > 
> 
>  
> 
> DISCLAIMER: This message is intended exclusively for the addressee(s) and may 
> contain confidential and privileged information. If you are not the intended 
> recipient please notify the sender immediately and destroy this message. 
> Unauthorized use, disclosure or copying of this message is strictly 
> prohibited.
> The foundation 'Stichting Deltares', which has its seat at Delft, The 
> Netherlands, Commercial Registration Number 41146461, is not liable in any 
> way whatsoever for consequences and/or damages resulting from the improper, 
> incomplete and untimely dispatch, receipt and/or content of this e-mail.
> 
> 
> 
> 
> 
> ------------------------------------------------------------------------------
> Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
> infrastructure or vast IT resources to deliver seamless, secure access to
> virtual desktops. With this all-in-one solution, easily deploy virtual 
> desktops for less than the cost of PCs and save 60% on VDI infrastructure 
> costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel
> 

------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to