windows-only read patch (was: error in ./configure when building gcl 2.6.7 onUbuntu)

Greetings!  Any progress here?

Take care,

Camm Maguire <camm at enhanced.com> writes:

> Greetings!
> 
> "Bill Page" <bill.page1 at synthesis.anikast.ca> writes:
> 
> > Camm,
> > 
> > What are the symptoms of this change? What problem is it
> > intended to correct? How can I test it? Is it sufficient
> > if I can build Axiom on windows after this change?
> > 
> 
> The code is intended to fix read-char-no-hang, which had been broken
> for a long time.  This caused Mike Thomas some problem of which I do
> not know the details, hence his ifdef patch with the comment:
> 
> revision 1.14.4.1.2.2.2.4.6.1.6.1.4.3
> date: 2006-06-15 22:26:22 -0400;  author: mjthomas;  state: Exp;  lines: +2 -1
> Band aid fix to stop build failure on Windows.
> ----------------------------
> 
> Vadim found the discrepancy in testing maxima:
> 
> http://www.mail-archive.com/gcl-devel at gnu.org/msg01408.html
> 
> Ideally, I'd like to know that reverting the patch works for both of
> you.  I'd really be pleased if someone could try acl2 as well.
> 
> Alas, due to email chaos, I seem to be disconnected from Vadim.
> 
> Take care,
> 
> 
> > Regards,
> > Bill Page.
> > 
> > > -----Original Message-----
> > > From: Camm Maguire [mailto:camm at enhanced.com] 
> > > Sent: January 11, 2007 10:51 AM
> > > To: Bill Page
> > > Cc: gcl-devel at gnu.org; axiom-developer at nongnu.org
> > > Subject: Re: [Gcl-devel] Re: error in ./configure when 
> > > building gcl 2.6.7 onUbuntu
> > > 
> > > 
> > > Greetings, and thanks so much!
> > > 
> > > Could you please try reverting:
> > > 
> > > cvs -z9 -q diff -u -r 1.14.4.1.2.2.2.4.6.1.6.1.4.2 -r 
> > > 1.14.4.1.2.2.2.4.6.1.6.1.4.3 read.d
> > > Index: read.d
> > > ===================================================================
> > > RCS file: /cvsroot/gcl/gcl/o/read.d,v
> > > retrieving revision 1.14.4.1.2.2.2.4.6.1.6.1.4.2
> > > retrieving revision 1.14.4.1.2.2.2.4.6.1.6.1.4.3
> > > diff -u -r1.14.4.1.2.2.2.4.6.1.6.1.4.2 -r1.14.4.1.2.2.2.4.6.1.6.1.4.3
> > > --- read.d	7 Jun 2006 15:09:38 -0000	
> > > 1.14.4.1.2.2.2.4.6.1.6.1.4.2
> > > +++ read.d	16 Jun 2006 02:26:22 -0000	
> > > 1.14.4.1.2.2.2.4.6.1.6.1.4.3
> > > @@ -256,6 +256,7 @@
> > >  
> > >  	x = read_object(in);
> > >  	vs_push(x);
> > > +#ifndef _WIN32
> > >  	while (listen_stream(in)) {
> > >  	  object c=read_char(in);
> > >  	  if (cat(c)!=cat_whitespace) {
> > > @@ -263,7 +264,7 @@
> > >  	    break;
> > >  	  }
> > >  	}
> > > -
> > > +#endif
> > >  	if (sharp_eq_context_max > 0)
> > >  		x = vs_head = patch_sharp(x);
> > >  
> > > Take care,
> > > 
> > > 
> > > 
> > > "Bill Page" <bill.page1 at synthesis.anikast.ca> writes:
> > > 
> > > > Camm,
> > > > 
> > > > On January 10, 2007 11:12 AM you wrote:
> > > > > ... 
> > > > > export CVSROOT=:pserver:anonymous at cvs.sv.gnu.org:/sources/gcl
> > > > > cvs -z9 -q co -d gcl-2.6.8pre -r Version_2_6_8pre
> > > > > 
> > > > > This is way past due for release.  I am only waiting on a 
> > > volunteer
> > > > > to test a windows-only read patch -- alas our heroic windows
> > > > > volunteer has resigned.  If anyone here can help please let me
> > > > > know.
> > > > > ...
> > > > 
> > > > Both Gaby and I have successfully built Axiom on Windows from a
> > > > recent version of gcl-2.6.8pre. What is required in order to test
> > > > this "windows-only read patch"?
> > > > 
> > > > Regards,
> > > > Bill Page.
> > > > 
> > > > 
> > > > 
> > > > 
> > > > 
> > > 
> > > -- 
> > > Camm Maguire			     			
> > > camm at enhanced.com
> > > ==============================================================
> > > ============
> > > "The earth is but one country, and mankind its citizens."  -- 
> > >  Baha'u'llah
> > > 
> > > 
> > 
> > 
> > 
> > 
> > 
> 
> -- 
> Camm Maguire			     			camm at enhanced.com
> ==========================================================================
> "The earth is but one country, and mankind its citizens."  --  Baha'u'llah
> _______________________________________________
> Maxima mailing list
> Maxima at math.utexas.edu
> http://www.math.utexas.edu/mailman/listinfo/maxima
> 
> 
> 

-- 
Camm Maguire			     			camm at enhanced.com
==========================================================================
"The earth is but one country, and mankind its citizens."  --  Baha'u'llah