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

>2011/11/2 Mark Thomas <ma...@apache.org>:
>> On 01/11/2011 22:39, Mark Thomas wrote:

>> My first stab at a patch for this is at [1]. Comments welcome.

>>http://people.apache.org/~markt/patches/2011-11-01-redeploy-trunk-v1.patch


>Quick review - several typos:
>
>mbeans-descriptors.xml:
>- typo "findRedployResources"
>- "Path to the resource, relative to docBase" - It is "either absolute
>or relative to docBase".
>
>docs: s/if is is updated/if it is updated/

I'll get those fixed for the next iteration.

>At first view I was hesitant wrt to new syntax in conf/context.xml
>(s/<WatchedResource>/<ReloadResource>/ but I am starting to like it.
>
>Need to apply and play with it a while.

Fair enough 

>I do not see though how it is related to another/main topic of
>http://markmail.org/message/5k4urwimvvmeqees
>that is: if context.xml startup fails somehow the webapp continues
>with the deploying the expanded dir.
>
>I observed it during startup time, not during redeployment:
>
>To reproduce, create conf/Catalina/localhost/host-manager.xml:

Ah. I think I misunderstood what you meant by this in the original message. I 
can see how this happens. Not sure on how to fix it yet.

Mark





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

Reply via email to