You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2010/03/17 14:13:02 UTC

svn commit: r924269 - /tomcat/tc5.5.x/trunk/STATUS.txt

Author: kkolinko
Date: Wed Mar 17 13:13:02 2010
New Revision: 924269

URL: http://svn.apache.org/viewvc?rev=924269&view=rev
Log:
update

Modified:
    tomcat/tc5.5.x/trunk/STATUS.txt

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=924269&r1=924268&r2=924269&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Wed Mar 17 13:13:02 2010
@@ -124,16 +124,20 @@ PATCHES PROPOSED TO BACKPORT:
   1)
     svn delete connectors/procrun
   2)
-    http://people.apache.org/~kkolinko/patches/2010-03-11_tc55_commons-daemon.patch
+    http://people.apache.org/~kkolinko/patches/2010-03-17_tc55_commons-daemon.patch
   +1: kkolinko
   -1: 
       mturk - Your patch contains a lot more then it claims. Proposed an
               alternative that does update only new commons-daemon dist binaries
               layout.
+      kkolinko - I updated the patch to do not delete "deploy.old" target.
+              Regarding "deploy.old": Note, that the commons-daemon files that were
+              copied in that target are already copied by "deploy-static", and that
+              I see no reason why one of the files was copied to ${tomcat.dist},
+              as that target updates ${tomcat.build} only.
 
 * Backport using Commons Daemon 1.0.2 from trunk
   Vote presumes removing daemon executables from our SVN.
   http://people.apache.org/~mturk/tomcat/patches/tomcat-5.5.x-daemon-1.0.2.patch
   +1: mturk
-  -1: 
-
+  -1: kkolinko: ${tomcat.dsit}



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


Re: svn commit: r924269 - /tomcat/tc5.5.x/trunk/STATUS.txt

Posted by Mladen Turk <mt...@apache.org>.
On 03/17/2010 02:13 PM, kkolinko@apache.org wrote:
> Author: kkolinko

> +      kkolinko - I updated the patch to do not delete "deploy.old" target.
> +              Regarding "deploy.old": Note, that the commons-daemon files that were
> +              copied in that target are already copied by "deploy-static", and that
> +              I see no reason why one of the files was copied to ${tomcat.dist},
> +              as that target updates ${tomcat.build} only.
>

Agreed.
Think another patch should be made to cleanup the build.xml
There are indeed huge amount of redundancy and leftovers from
who knows when.


Regards
-- 
^TM

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