gnuplot-beta

For gnuplot 4.2 rc3

[Petr M. <mi...@ph...>]

I wish rc3 appears soon. There, I propose the following changes:

1. Apply [ gnuplot-Bugs-1612995 ] Flag 'interactive' not reset to FALSE
   after '-' plot script

2. I propose to set
	--with-readline=gnu
   to be the default, instead of the built-in readline.
   People (except for some distributors) always use this option,
   and setting this to default will save us time to answer those
   permanent emails "tabulator does not work in the new version".

3. www.gnuplot.info is not mirroring gnuplot.sourceforge.net, for
   a long time, nobody responded until now. Is still Clark Gaylord the
   persons with the access there?

---
PM

Re: For gnuplot 4.2 rc3

[Ethan A M. <merritt@u.washington.edu>]

On Saturday 20 January 2007 11:54, Petr Mikulik wrote:
> I wish rc3 appears soon. There, I propose the following changes:
> 
> 1. Apply [ gnuplot-Bugs-1612995 ] Flag 'interactive' not reset to FALSE
>    after '-' plot script

As per the comments attached to the bug report, I don't think that
patch is the correct fix for the problem.  The problem is not the
interactive flag; it is a failure to distinguish between a typed
"load" command  and an additional input file.

 
> 2. I propose to set
> 	--with-readline=gnu
>    to be the default, instead of the built-in readline.
>    People (except for some distributors) always use this option,
>    and setting this to default will save us time to answer those
>    permanent emails "tabulator does not work in the new version".

Fine with me.  Do you have a patch?

I was just about to prepare the -rc3 tarball, but I'll wait for your
readline configuration patch if you have it ready to go.

	Ethan

-- 
Ethan A Merritt
Biomolecular Structure Center
University of Washington, Seattle 98195-7742

Re: For gnuplot 4.2 rc3

[Petr M. <mi...@ph...>]

> > I wish rc3 appears soon. There, I propose the following changes:
> > 
> > 1. Apply [ gnuplot-Bugs-1612995 ] Flag 'interactive' not reset to FALSE
> >    after '-' plot script
> 
> As per the comments attached to the bug report, I don't think that
> patch is the correct fix for the problem.  The problem is not the
> interactive flag; it is a failure to distinguish between a typed
> "load" command  and an additional input file.

If the input file does not exist, or there is an error in it, gnuplot should 
exit. Instead, it loops forever. So there is a bug... (but I don't know a 
better fix).


> > 2. I propose to set
> > 	--with-readline=gnu
> >    to be the default, instead of the built-in readline.
> >    People (except for some distributors) always use this option,
> >    and setting this to default will save us time to answer those
> >    permanent emails "tabulator does not work in the new version".
> 
> Fine with me.  Do you have a patch?

It is enclosed. I think it is OK; I will commit it unless some other 
comments.

---
PM

Re: For gnuplot 4.2 rc3

[Petr M. <mi...@ph...>]

> I was just about to prepare the -rc3 tarball, but I'll wait for your
> readline configuration patch if you have it ready to go.

It's done. Please make rc3.

---
PM

Re: For gnuplot 4.2 rc3

[Petr M. <mi...@ph...>]

> > > > 2. I propose to set
> > > > 	--with-readline=gnu
> > > >    to be the default, instead of the built-in readline.
> > > >    People (except for some distributors) always use this option,
> > > >    and setting this to default will save us time to answer those
> > > >    permanent emails "tabulator does not work in the new version".
> > > 
> > > Fine with me.  Do you have a patch?
> 
> Maybe have the built-in readline be selected by
>   --with-readline=builtin
> rather than
>   --with-readline
> 
> This comment is confusing:
> 
> Readline library: builtin minimal readline (use --with-readline to override)

Yes, it as. I've changed it:

$ ./configure --help
...
  --with-readline=builtin  use the built-in minimal readline
  --with-readline=gnu      use the GNU readline library (default; 4.2 or 
				newer required)
  --with-readline=DIR      specify the location of GNU readline
...


$ ./configure --with-readline
$ ./configure --with-readline=gnu
...
Readline library: GNU readline library with  -lncurses
...


$ ./configure --with-readline=builtin
...
Readline library: builtin minimal (use --with-readline=gnu for GNU 
readline)
...


---
PM