ok, this should take care of the problem, sorry for the delay

http://svn.apache.org/viewvc?rev=607800&view=rev

Filip

Mark Thomas wrote:
Filip Hanik - Dev Lists wrote:
Mark Thomas wrote:
[EMAIL PROTECTED] wrote:
Author: fhanik
Date: Mon Dec 17 12:55:56 2007
New Revision: 605000

URL: http://svn.apache.org/viewvc?rev=605000&view=rev
Log:
port new annotation logic from previous trunk
-1.
give it some time, before you veto it, not a release branch, no need to
spill the eggnog yet :)

It wasn't meant to be a particularly strong veto but it is a veto. I am not
asking for an immediate revert (since I can revert it locally), I am asking
that the code gets fixed in the short to medium term.

I will look into it

Great - that is all I was looking for.

An unrelated minor point is org.apache.InstanceManager should be moved,
probably to org.apache.catalina.InstanceManager, as we don't have control
over the org.apache package.
no-can-do, this was discussed previously, I suggested org.apache.tomcat
and that was turned down, see archives

I remember the discussion. If you don't want to move it as part of your
patch I'll wait until these changes are complete and then propose a patch
to move any classes still in org.apache and we'll see what votes the patch
gets.

Mark

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to