https://issues.apache.org/bugzilla/show_bug.cgi?id=53340

Konstantin Kolinko <knst.koli...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |WONTFIX

--- Comment #1 from Konstantin Kolinko <knst.koli...@gmail.com> ---
Having more that 1 file to handle the debug-level feature of "catalina run" is
ridiculous. The patch makes it 3 files. ;(


There could be run.bat, like there is startup.bat. But even that seems too much
for a debug feature.

I like you idea of using "_RECURSE_" variable from your comment on bug 53115,
but it solves only small part of complexity.

I do not see a way to get rid of the temporary "Y" file. I tried several
variants of piping "echo Y | " into the recursively called batch file, but all
they do not work.  It gets rid of one "Terminate this batch?" confirmation, but
second one appears.


If there ever are noticeable concerns with the "^C" handling code in
catalina.bat, I think it all could be removed, reverting to the simple
catalina.bat of Tomcat 6.  The current code is a bit complicated, but I cannot
call it broken and worth of fixing.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to