o.a.s.util.GenericDataSource is in the core and extends the legacy GenericDataSource so legacy needs to be built first.
It is deprecated since 1.1. Can we get rid of all legacy / DataSource stuff for 1.2? Steve > -----Original Message----- > From: David Graham [mailto:[EMAIL PROTECTED] > Sent: August 9, 2003 4:03 PM > To: Struts Developers List > Subject: Building contrib packages WAS Re: cvs commit: > jakarta-struts/contrib/tag-doc build.xml > > > While we're somewhat on this topic I'd like to get some clarification on > the build process. Why do struts-el and struts-legacy get built with the > standard Struts build file? Struts-EL has *never* built on my machine and > I have just taken it on faith the rest of Struts built properly. Also, > after struts-legacy was added I've had to run my builds twice. The first > one always fails and the second succeeds (well, not really succeeds > because struts-el fails). > > In general, I don't see a reason for contrib packages to be built with the > rest of Struts. Am I missing something? > > David > > --- [EMAIL PROTECTED] wrote: > > craigmcc 2003/08/09 12:29:30 > > > > Modified: . build.xml > > contrib/struts-el build.xml > > > > contrib/struts-el/src/share/org/apache/strutsel/taglib/html > > ELFormTag.java ELHtmlTag.java > > ELJavascriptValidatorTag.java > > contrib/struts-faces build.xml > > contrib/struts-legacy build.xml > > contrib/tag-doc build.xml > > Log: > > Correct loading order of properties files to go from most local > > (current > > directory) to most global (${user.home}/build.properties). Since Ant > > follows a "first definition wins" strategy, this makes the most sense > > for our convention of allowing local overrides of global values. It > > also > > means that I can now do an "ant dist" in the top level directory with > > only > > one thing in my build.properties file (jdk.version=1.4), so this > > should fix > > the nightly builds as well (verifying that is my next step). > > > > Also fixed some compile errors in struts-el -- I don't know how that > > code > > could have compiled for anyone. Could someone more familiar with that > > library make sure I did the changes correctly? > > > > Revision Changes Path > > 1.118 +5 -2 jakarta-struts/build.xml > > > > Index: build.xml > > =================================================================== > > RCS file: /home/cvs/jakarta-struts/build.xml,v > > retrieving revision 1.117 > > retrieving revision 1.118 > > diff -u -r1.117 -r1.118 > > --- build.xml 8 Aug 2003 06:00:55 -0000 1.117 > > +++ build.xml 9 Aug 2003 19:29:30 -0000 1.118 > > @@ -113,9 +113,9 @@ > > --> > > > > <!-- Load local and user build preferences --> > > - <property file="${user.home}/build.properties"/> > > - <property file="../build.properties"/> > > <property file="build.properties"/> > > + <property file="../build.properties"/> > > + <property file="${user.home}/build.properties"/> > > > > <!-- Default values for unspecified properties --> > > <property name="catalina.home" > > value="../jakarta-tomcat-4.0/build"/> > > @@ -204,6 +204,9 @@ > > > > <!-- Web directory --> > > <property name="web.dir" value="web"/> > > + > > + <!-- Pointer to struts-legacy.jar --> > > + <property name="struts-legacy.jar" > > value="${basedir}/contrib/struts-legacy/dist/struts-legacy.jar"/> > > > > <!-- Compilation Classpath --> > > <path id="compile.classpath"> > > > > > > > > 1.17 +4 -4 jakarta-struts/contrib/struts-el/build.xml > > > > Index: build.xml > > =================================================================== > > RCS file: /home/cvs/jakarta-struts/contrib/struts-el/build.xml,v > > retrieving revision 1.16 > > retrieving revision 1.17 > > diff -u -r1.16 -r1.17 > > --- build.xml 13 Jun 2003 03:22:14 -0000 1.16 > > +++ build.xml 9 Aug 2003 19:29:30 -0000 1.17 > > @@ -51,10 +51,10 @@ > > --> > > > > <!-- Load local and user build preferences --> > > - <property file="${user.home}/build.properties"/> > > - <property file="../../../build.properties"/> > > - <property file="../../build.properties"/> > > <property file="build.properties"/> > > + <property file="../../build.properties"/> > > + <property file="../../../build.properties"/> > > + <property file="${user.home}/build.properties"/> > > > > <!-- Default values for unspecified properties --> > > <property name="jdbc20ext.jar" value="../jdbc2_0-stdext.jar"/> > > @@ -219,7 +219,7 @@ > > --> > > <copy todir="${build.home}/library"> > > <fileset dir="${jstl.home}/tld" includes="*.tld"/> > > - <fileset dir="${struts.home}/dist/lib" includes="*.tld"/> > > + <fileset dir="../../dist/lib" includes="*.tld"/> > > </copy> > > </target> > > > > > > > > > > 1.9 +7 -7 > > > jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/tag > lib/html/ELFormTag.java > > > > Index: ELFormTag.java > > =================================================================== > > RCS file: > > > /home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/st > rutsel/taglib/html/ELFormTag.java,v > > retrieving revision 1.8 > > retrieving revision 1.9 > > diff -u -r1.8 -r1.9 > > --- ELFormTag.java 26 Jul 2003 05:48:02 -0000 1.8 > > +++ ELFormTag.java 9 Aug 2003 19:29:30 -0000 1.9 > > @@ -386,9 +386,9 @@ > > this, pageContext)) != > > null) > > setScope(string); > > > > - if ((bool = EvalHelper.evalBoolean("scriptLanguage", > > getScriptLanguageExpr(), > > - this, pageContext)) != > > null) > > - setScriptLanguage(bool.booleanValue()); > > + if ((string = EvalHelper.evalString("scriptLanguage", > > getScriptLanguageExpr(), > > + this, pageContext)) != > > null) > > + setScriptLanguageExpr(string); > > > > if ((string = EvalHelper.evalString("style", getStyleExpr(), > > this, pageContext)) != > > null) > > > > > > > > 1.8 +7 -7 > > > jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/tag > lib/html/ELHtmlTag.java > > > > Index: ELHtmlTag.java > > =================================================================== > > RCS file: > > > /home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/st > rutsel/taglib/html/ELHtmlTag.java,v > > retrieving revision 1.7 > > retrieving revision 1.8 > > diff -u -r1.7 -r1.8 > > --- ELHtmlTag.java 26 Jul 2003 05:48:03 -0000 1.7 > > +++ ELHtmlTag.java 9 Aug 2003 19:29:30 -0000 1.8 > > @@ -158,9 +158,9 @@ > > Boolean bool = null; > > String string = null; > > > > - if ((bool = EvalHelper.evalBoolean("lang", getLangExpr(), > > - this, pageContext)) != > > null) > > - setLang(bool.booleanValue()); > > + if ((string = EvalHelper.evalString("lang", getLangExpr(), > > + this, pageContext)) != > > null) > > + setLangExpr(string); > > > > if ((bool = EvalHelper.evalBoolean("locale", getLocaleExpr(), > > this, pageContext)) != > > null) > > > > > > > > 1.9 +7 -7 > > > jakarta-struts/contrib/struts-el/src/share/org/apache/strutsel/tag > lib/html/ELJavascriptValidatorTag.java > > > > Index: ELJavascriptValidatorTag.java > > =================================================================== > > RCS file: > > > /home/cvs/jakarta-struts/contrib/struts-el/src/share/org/apache/st > rutsel/taglib/html/ELJavascriptValidatorTag.java,v > > retrieving revision 1.8 > > retrieving revision 1.9 > > diff -u -r1.8 -r1.9 > > --- ELJavascriptValidatorTag.java 26 Jul 2003 05:48:03 -0000 1.8 > > +++ ELJavascriptValidatorTag.java 9 Aug 2003 19:29:30 -0000 1.9 > > @@ -276,9 +276,9 @@ > > this, pageContext)) != > > null) > > setPage(integer.intValue()); > > > > - if ((bool = EvalHelper.evalBoolean("scriptLanguage", > > getScriptLanguageExpr(), > > - this, pageContext)) != > > null) > > - setScriptLanguage(bool.booleanValue()); > > + if ((string = EvalHelper.evalString("scriptLanguage", > > getScriptLanguageExpr(), > > + this, pageContext)) != > > null) > > + setScriptLanguageExpr(string); > > > > if ((string = EvalHelper.evalString("src", getSrcExpr(), > > this, pageContext)) != > > null) > > > > > > > === message truncated === > > > __________________________________ > Do you Yahoo!? > Yahoo! SiteBuilder - Free, easy-to-use web site design software > http://sitebuilder.yahoo.com > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]