Project

General

Profile

Bug #3777

Move Tomcat libraries into metacat build

Added by Michael Daigle almost 11 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Normal
Category:
metacat
Target version:
Start date:
01/26/2009
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:
3777

Description

Right now, the Tomcat libraries (javax.http...) are referenced via the Tomcat install directory on the build machine. These libs should be included in metacat code for build purposes (not for distribution).


Related issues

Blocked by Metacat - Bug #3824: Normalize the app deploy dir in build.propertiesResolved02/17/2009

History

#1 Updated by Matt Jones almost 11 years ago

We used to include those libs back in the Tomcat 3 days, but took them out as
the servlet library jars changed in Tomcat 3, 4, 5, and 6. So now we use the
locally installed version of the jar that comes with Tomcat to build. We
should consider whether adding these jars to Metacat is really an advantage --
we have to have Tomcat, and we have to differentiate which version of Tomcat is
being used in order to find the right jar file for servlet support. Having
those be in Metacat may not gain us much, and we'd have to differentiate which
to use in any given build. I'd opt for improving the build logic to discover
the jar files in the locally installed Tomcat. And I agree with Duane -- it
would be nice if the default app.deploy.dir used the standard webapps location,
as that is where most people will want to put it. So, I vote for changing back
to:
app.deploy.dir=${build.tomcat.dir}/webapps
Then if we need a different location in our particular install we can change it
as needed when we build and install locally. But the default should be the
standard webapps location.

#2 Updated by Michael Daigle almost 11 years ago

Review comments on ticket 3824

#3 Updated by Michael Daigle almost 10 years ago

This is not practical. The server should reference the libraries of the native install.

#4 Updated by Redmine Admin over 6 years ago

Original Bugzilla ID was 3777

Also available in: Atom PDF