> -----Original Message----- > From: Craig R. McClanahan [mailto:[EMAIL PROTECTED]] > Sent: Wednesday, October 02, 2002 8:12 AM > To: Struts Developers List > Subject: Re: cvs commit: jakarta-struts/contrib/struts-el build.xml > > On 1 Oct 2002, David M. Karr wrote: > > > >> Also, if Struts-EL is being built by default, what > will happen if someone tries > > >> to build it with a 2.2 servlet.jar? > > > > Craig> Good point ... it should probably be made > conditional based on the > > Craig> presence of Servlet 2.3 (and the JSTL jars, for > that matter). > > > > Correct me if I'm wrong, but doesn't this mean that it > can't be provided in the > > nightly build, unless we do both a Servlet 2.2 and Servlet > 2.3 build? > > The nightly builds are created against Servlet 2.3 already. If you > download and use the nightly build in a Servlet 2.2 > environment, you can't > use the struts-el stuff, but it doesn't change anything else that is > created -- so I don't see a need to create two different versions. > > > I've tested a change to the top-level build.xml to only > build it if "jstl.jar" > > is set (and I put my "jstl.jar" setting in my top-level > build.properties file), > > but I didn't try to do anything with a Servlet 2.2 build. > > > > I've got my announcement note written, and ready to send, > but I have a feeling > > I may have to delay/change it, based on this. > > If you want to go ahead and check in the conditional change to the > "dist.contrib" target, that would be great.
I'll do that, but I won't be able to commit it until tonight. I don't have my Jakarta CVS environment set up here at work (and I hesitate trying to manage more than one CVSROOT on one box). On the nightly build, I noticed that the build did eventually complete, including the struts-el.jar, so I assume the issue with the Catalina ant tasks was worked around? I committed the change to my build.xml to remove those soon after I saw the build failure this morning. If this morning's nightly build (I'll just have to get used to that not sounding right :) ) is a go, I'll send in my announcement note in a few minutes. -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
