Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error - PATCH working

2016-12-19 Thread Pedro Vicente
Hi Phil It seems you got it working I applied the patch, to 2 previous master commits that compile. results are: make VERBOSE=1 test_wxPLplotDemo 14:26:35: Debug: wxPLplotwindow::wxPLplotwindow 14:26:35: Debug: frame->Create 14:26:35: Debug: Plot() Yielding 14:26:35: Debug: Plot() Yielding 14:2

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
On 19 December 2016 at 17:47, Pedro Vicente wrote: > Hi Phil > >> Pedro can you please check if this works on your systems? > > > ok, I'll try it when the patch is pushed to the master . > > Alan > > I assume you are going to push the patch? > I get some compiling errors on the current master, ple

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Pedro Vicente
Hi Phil > Pedro can you please check if this works on your systems? ok, I'll try it when the patch is pushed to the master . Alan I assume you are going to push the patch? I get some compiling errors on the current master, please see my last post a new idea: what about if we just override th

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
No worries :-) Attached is a patch which I hope should fix things. It is the simplest possible fix as far as I can tell. It does have a while loop with no counter, but unless there is some rather fatal bug in wxWidgets it should never turn into an infinite loop. Alan if you prefer I will add a cou

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 11:31- Phil Rosenberg wrote: > I'm just generating a test fix. I will send a patch round once it's > done. If it works then I'll tidy it up. However you appear to have > bumped up the minimum CMake version so I need to reinstall CMake - > turns out the latest Windows version req

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Phil Rosenberg
I'm just generating a test fix. I will send a patch round once it's done. If it works then I'll tidy it up. However you appear to have bumped up the minimum CMake version so I need to reinstall CMake - turns out the latest Windows version requires a manual uninstall first. I'm sure you're not deli

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 00:30-0800 Alan W. Irwin wrote: > Anyhow, good luck with implementing this readiness check anyway you > like, and I look forward to your commit to that effect which can go > right on top of master tip (which should simplify your life). Then we can ask > Pedro to drop his (now hopeful

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-19 Thread Alan W. Irwin
On 2016-12-19 07:56- p.d.rosenb...@gmail.com wrote: > Hi Alan, Pedro > Yes Alan, that is what I intended. Sorry, that doesn't compile. I was on my > way to bed when I sent the email and was doing it from memory. > > I will try to find a moment today to fix this. > > I'm not sure what your one

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-18 Thread p.d.rosenberg
Phil Rosenberg; Pedro Vicente; PLplot development list Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error On 2016-12-18 23:27- Phil Rosenberg wrote: > The error is in the wxplplotdemo code, not in the wxplplotwindow. In > the demo we must simply wait for the OnCreate mes

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-18 Thread Alan W. Irwin
On 2016-12-18 23:27- Phil Rosenberg wrote: > The error is in the wxplplotdemo code, not in the wxplplotwindow. In > the demo we must simply wait for the OnCreate message to arrive and be > processed before we grab the stream in Plot(). We may be able to do > that with > while(!m_created) > w

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-18 Thread Pedro Vicente
ot;Phil Rosenberg" To: "Pedro Vicente" Cc: "Alan W. Irwin" ; "PLplot development list" Sent: Sunday, December 18, 2016 6:27 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > Hi Pedro > Ah, I had assumed that the OnCreate event was not b

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-18 Thread Phil Rosenberg
Hi Pedro Ah, I had assumed that the OnCreate event was not being triggered at all. I now understand what is happening. Or at least I think I do ;-) Actually now you have pointed out the issue I am surprised the code works on any platform. The error is in the wxplplotdemo code, not in the wxplplotw

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-16 Thread Pedro Vicente
Hi Phil, Alan I tried evtCreateExample.cpp and I get the good expected result. This just means that for this simple example the OnCreate event is triggered. To note that on my bad results of the PLplot demo, the OnCreate event is *also* triggered. the problem is that is triggered after the plot

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-16 01:05- Phil Rosenberg wrote: > Hmm - well another theory down in smoke. > > Attached is an absolute minimum example of the use of wxEVT_CREATE. On > Windows I get the expected behaviour of a popup dialog appearing > before the frame saying "OnCreate called." > > Could you try it

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Pedro Vicente
e, -Pedro - Original Message - From: "Alan W. Irwin" To: "Pedro Vicente" Cc: "Phil Rosenberg" ; "PLplot development list" Sent: Thursday, December 15, 2016 8:34 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > On 2016-12-15 19:56-0

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-15 19:56-0500 Pedro Vicente wrote: > Alan > > All my linux failures were on 3 different "real" machines (CentOS with > personal wxwidgets 3.1 build, 2 ubuntus 14.04 16.4 from packages) > > the 4th linux failure was on a debian I installed on VirtualBox. Hi Pedro: Sorry. As Phil said

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-16 00:26- Phil Rosenberg wrote: > Unfortunately I don't think I have enough space to install virtualbox. > I presume I need a few 10s of GB for an Ubuntu install? > > I have also just tried on the Bash on Ubuntu on Windows (still trying > to decide if I like this or Cygwin best. Any

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Phil Rosenberg
> > Cc: "PLplot development list" > Sent: Thursday, December 15, 2016 7:05 PM > Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > > >> On 2016-12-15 17:19-0500 Pedro Vicente wrote: >> >>> Hi Phil >>> >>> >>>&

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Pedro Vicente
From: "Phil Rosenberg" To: "Alan W. Irwin" Cc: "Pedro Vicente" ; "PLplot development list" Sent: Thursday, December 15, 2016 7:26 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > Unfortunately I don't think I have enough spac

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Pedro Vicente
From: "Alan W. Irwin" To: "Pedro Vicente" ; "Phil Rosenberg" Cc: "PLplot development list" Sent: Thursday, December 15, 2016 7:05 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > On 2016-12-15 17:19-0500 Pedro Vicente wrote:

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Phil Rosenberg
Unfortunately I don't think I have enough space to install virtualbox. I presume I need a few 10s of GB for an Ubuntu install? I have also just tried on the Bash on Ubuntu on Windows (still trying to decide if I like this or Cygwin best. Anyway...). Again everything works fine on this (I'm using X

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-15 17:19-0500 Pedro Vicente wrote: > Hi Phil > > >>> I can't help feeling that there is something that we are missing here. >>> Given both I and Pedro have tested on Ubuntu 14.04 there must be >>> something different between the two systems. > > it seems so. > the only way to be sure is

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Pedro Vicente
- Original Message - From: "Phil Rosenberg" To: "Pedro Vicente" Cc: "Alan W. Irwin" ; "PLplot development list" Sent: Thursday, December 15, 2016 5:06 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > Hi Pedro, Alan > I

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Phil Rosenberg
Hi Pedro, Alan I'm still unable to reproduce the problem on my Ubuntu 14.04 system with wxWidgets 3.0. Very strange. I have tried a totally fresh build directory and static and dynamic builds. I had forgotten to test on my work CentOS PC and although I know it is powered on I can't ssh into it. I'

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Pedro Vicente
Hi Alan I installed the latest from the debian site, here https://www.debian.org/distrib/ it's not surprising that I got the same results in ubuntu because I used the same package sudo apt-get install libwxgtk3.0-dev On 2016-12-15 04:32, Alan W. Irwin wrote: > On 2016-12-15 01:11-0500 Pedro

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-15 01:31-0500 Pedro Vicente wrote: > Hi Alan > >> I think you will agree with me that smells like a workaround rather >> than a fundamental fix. > > yes, I agree. > >>> Eventually, we had to change our >>> minimum acceptable wxwidgets version from 2.8 to 3.0 because of one >>> commit wh

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-15 Thread Alan W. Irwin
On 2016-12-15 01:11-0500 Pedro Vicente wrote: > Hi Alan > > I just installed the latest debian on VirtualBox, and I get the same errors. > the results are attached, same thing that I did for ubuntu. Hi Pedro: Just for the record, what version of Debian? Currently Jessie = stable (a largely froz

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
I'll try this tomorrow > > -Pedro > > > > ----- Original Message - > From: "Pedro Vicente" > To: "Alan W. Irwin" ; "Phil Rosenberg" > > Cc: "PLplot development list" > Sent: Thursday, December 15, 2016 1:31 AM >

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
Sent: Thursday, December 15, 2016 1:57 AM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > Hi Alan, Phil > > I forgot about a very simple fix that does need a custom function call > (therefore a "real" fix, or better a non intrusive fix for users) > &

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
makes it that it's only called 1 time I'll try this tomorrow -Pedro - Original Message - From: "Pedro Vicente" To: "Alan W. Irwin" ; "Phil Rosenberg" Cc: "PLplot development list" Sent: Thursday, December 15, 2016 1:31 AM Subject: R

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
win" To: "Pedro Vicente" ; "Phil Rosenberg" Cc: "PLplot development list" Sent: Thursday, December 15, 2016 12:35 AM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error > On 2016-12-14 22:30-0500 Pedro Vicente wrote: > >> Hi Alan >&g

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
r 14, 2016 7:16 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error On 2016-12-14 18:00-0500 Pedro Vicente wrote: Hi Alan What happens if you try a Ubuntu platform with a system version of wxwidgets version = 3.0.x? Same question for a Ubuntu platform with wxwidgets version = 3.1.x?

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Alan W. Irwin
On 2016-12-14 22:30-0500 Pedro Vicente wrote: > Hi Alan > > see my replies between your questions > >> This certainly qualifies as a release-critical wxwidgets bug. And you >> have certainly supplied enough information that Phil should find this >> to be straightforward to reproduce. > > easy to

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
OnCreate ) ); -Pedro - Original Message - From: "Alan W. Irwin" To: "Phil Rosenberg" ; "Pedro Vicente" Cc: "PLplot development list" Sent: Wednesday, December 14, 2016 7:16 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build erro

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error -- ubuntu build

2016-12-14 Thread Alan W. Irwin
On 2016-12-14 18:23-0500 Pedro Vicente wrote: > Hi Alan > > Results for > > lsb_release -a > Description: Ubuntu 16.04.1 LTS > Release: 16.04 > Codename: xenial > > sudo apt-get install libwxgtk3.0-dev > > cmake .. -G "Unix Makefiles" > -DCMAKE_INSTALL_PREFIX:PATH=/home/pvn/plplot-install -DCMAK

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Alan W. Irwin
On 2016-12-14 18:00-0500 Pedro Vicente wrote: Hi Alan What happens if you try a Ubuntu platform with a system version of wxwidgets version = 3.0.x? Same question for a Ubuntu platform with wxwidgets version = 3.1.x? I was actually just going to do that just now, so here it goes lsb_releas

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error -- ubuntu build

2016-12-14 Thread Pedro Vicente
ro Vicente" To: "Alan W. Irwin" Cc: "PLplot development list" Sent: Wednesday, December 14, 2016 6:00 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error Hi Alan What happens if you try a Ubuntu platform with a system version of wxwidgets versio

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
-Pedro - Original Message - From: "Alan W. Irwin" To: "Pedro Vicente" Cc: "PLplot development list" Sent: Wednesday, December 14, 2016 5:39 PM Subject: Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error On 2016-12-14 15:30-0500 Pedro Vicente wrote:

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Alan W. Irwin
On 2016-12-14 15:30-0500 Pedro Vicente wrote: > Hi Alan > > >>> Can you confirm that is the case for >>> unmodified PLplot from the tip of the git master branch? > > Yes, it is the unmodified PLplot from the tip of the git master branch cloned > today. > I only added this line > assert(pls) > in

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
Hi Alan Can you confirm that is the case for unmodified PLplot from the tip of the git master branch? Yes, it is the unmodified PLplot from the tip of the git master branch cloned today. I only added this line assert(pls) in wxPLplotDemo.cpp, which you can see in the output here on Debian

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Alan W. Irwin
On 2016-12-14 11:47-0500 Pedro Vicente wrote: > Hi Phil > > I am using the attached qt project file on my CentOS 6.8 build, adapted to > build > examples/c++/wxPLplotDemo.cpp > > to use, place the file in > examples/c++ > > just change these lines for your own case > > dir_inc_plplot = /data/home

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
Hi Phil I am using the attached qt project file on my CentOS 6.8 build, adapted to build examples/c++/wxPLplotDemo.cpp to use, place the file in examples/c++ just change these lines for your own case dir_inc_plplot = /data/home002/pvicente/plplot-plplot-install/include/plplot dir_lib_plpl

Re: [Plplot-devel] wxPLplotDemo.cpp -- linux build error

2016-12-14 Thread Pedro Vicente
Hi Phil Were you able to reproduce the error I get on my linux builds of wxPLplotDemo? I just did a git clone of the last plplot and I still get the error see my last email from today, there are instructions on how to debug with Qt project here https://github.com/pedro-vicente/plplot-wxwidge