Tomcat 5.5 startup and shutdown.. again

[Tomas Verbaitis]

On Fri, Oct 21, 2005 at 12:15:48AM +0200, Herve Quiroz wrote:
> Anyway, there seem to be a problem with your patch (although I have not
> tested it yet).
> 
> > --- tomcat.sh.in.orig   Wed Sep 14 15:30:23 2005
> > +++ tomcat.sh.in        Thu Oct 20 20:18:32 2005
> > -procname="java"
> > +procname="%%JAVA_HOME%%/bin/java"
> [...]
> > --- Makefile.orig       Thu Sep 29 18:08:23 2005
> > +++ Makefile    Thu Oct 20 20:35:40 2005
> > @@ -64,7 +64,8 @@
> > +               JAVA_HOME=${JAVA_HOME}
> Here you define ${procname} to be hardcoded from a JAVA_HOME value in
> the port. This JAVA_HOME is set at install time but the rcNG script
> allows to override JAVA_HOME at run-time:
> 
>   jakarta_tomcat%%TOMCAT_VERSION%%_java_home
 
Defining ${procname} to $JAVA_HOME_AT_INSTALL_TIME/bin/java is needed so
that rcNG script finds Tomcat instance launched by tomcat55ctl.
Therefore it suffers from the same problem as daemonctl.c where
%%JAVA_HOME%% is also set at port install time.

> What will happen if the JAVA_HOME from port install stage and the one
> set in rc.conf are different?

In this case we get the initial situation again -- rcNG script deals
with one instance of tomcat, tomcat55ctl with another. 

I still think my proposed change still should be applied as it at least
gets the default situation right, when both tomcat55ctl and rcNG script
refer to the same JVM. The "administrative" approach to this problem
would be only to put a warning near the place where
jakarta_tomcat%%TOMCAT_VERSION%%_java_home is defined. "Technically" i
aggree that it's better to get rid of tomcat55ctl as a program, no
matter how convenient and usual it was.

-- 
Tomas "Verbaitis" Verbaitis ** http://megalogika.lt
* šita gudri beždžionė moka dirbti su kompiuteriu *
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-java/attachments/20051021/0946e14c/attachment.bin