RE: [xslt] Release of libxslt-1.0.31

[Jesse Pelton <jsp PKC com>]

Well, that's obviously not right! That's line 220 in my copy of the file,
which appears to match what's in CVS. Anyway, here's a little more context:

195	if (compiler == "msvc") {
196		vf.WriteLine("INCLUDE=$(INCLUDE);" + buildInclude);
197		vf.WriteLine("LIB=$(LIB);" + buildLib);
198	} else if (compiler == "mingw") {
199		vf.WriteLine("INCLUDE+=;" + buildInclude);
200		vf.WriteLine("LIB+=;" + buildLib);
201	}
202	} else if (compiler == "bcb") {
203		vf.WriteLine("INCLUDE=" + buildInclude);
204		vf.WriteLine("LIB=" + buildLib);
205	}
206	vf.Close();

Line 201 contains an extra close brace. It comes from the BCB patch applied
June 14. 

-----Original Message-----
From: Daniel Veillard [mailto:veillard@redhat.com] 
Sent: Monday, July 07, 2003 10:52 AM
To: xslt@gnome.org
Subject: Re: [xslt] Release of libxslt-1.0.31


On Mon, Jul 07, 2003 at 10:02:32AM -0400, Jesse Pelton wrote:
> Line 201 of win32\configure.js should be deleted. Otherwise, you get 
> the following message:
> 
> <path>\libxml2-2.5.8\win32\configure.js(202, 4) Microsoft JScript 
> compilation error: Syntax error

  To me line 201 is
                    if (s.search(/\@VERSION\@/) != -1) {

I really don't see how removing it would not break the script !

Daniel

-- 
Daniel Veillard      | Red Hat Network https://rhn.redhat.com/
veillard@redhat.com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
_______________________________________________
xslt mailing list, project page http://xmlsoft.org/XSLT/ xslt@gnome.org
http://mail.gnome.org/mailman/listinfo/xslt