commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ryan Lubke <Ryan.Lu...@Sun.COM>
Subject Re: [HttpClient][PATCH] Minor enhancement to build process
Date Fri, 30 Aug 2002 02:26:37 GMT
On Thu, 2002-08-29 at 09:04, Jeff Dever wrote:
> Ryan Lubke wrote:
> > Hi,
> >
> > I've added dependency checking for the JAR files specified in the
> > for the different compilation targets.
> > If the dependency isn't satisfied, the build process will fail
> > before any attempt at compilation is made (NOTE:  the dependency
> > checking targets are leveraged from work done on the Standard
> > Tag Library project).
> We are supposed to be moving to Maven for building.  build.xml is a maven
> output.  I'm not sure how this work would effect that.

I don't think that the build.xml is a maven output, at least not
according to CVS. 

> >
> >
> > If jsse.jar, jcert.jar,jnet.jar, or log4j.jar is not present,
> > the default compile target will not run.
> This is excessively strict.  The jsse jars are included in the jdk1.4
> distribution so this should not prevent compilation (did not check if this is
> the case or not).  log4j is not a requiremennt for httpclient, but
> commons-logging.jar is.

OK.  So the build process will now go through and try to detect if JDK
1.4 is being used.  If it is, then the JSSE JAR file dependency check
(which is now in its own target) will not be invoked, and the
compilation process works as expected. 

If JDK 1.4 is not being used, then the process will check for the JSSE
JAR files based on the user's  If found, all is well,
otherwise an error.

I've also removed the check for log4j.

Anyway, take or leave it.  It's up to you.  I kinda like it at this
point.  This definitely takes some of the guess work out of the build

> >
> >
> > I've also made a modification so that when the compile.test-webapp
> > target is run, it will create the WAR and EAR files.  Saves folks from
> > having to call the targets after the compilation is complete.
> I like that!
> --
> To unsubscribe, e-mail:   <>
> For additional commands, e-mail: <>

View raw message