You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2013/05/31 18:05:20 UTC
[Bug 55035] New: Deploy ROOT webapp with version through text
manager using Ant
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
Bug ID: 55035
Summary: Deploy ROOT webapp with version through text manager
using Ant
Product: Tomcat 7
Version: 7.0.40
Hardware: PC
OS: All
Status: NEW
Severity: major
Priority: P2
Component: Manager
Assignee: dev@tomcat.apache.org
Reporter: sergeytch@gmail.com
Currently Ant "deploy" task does not support separate parameter "version" and,
thanks to URLEncoder, there is no workaround for that.
But for apps with not empty context it's possible to define version using ##,
like setting path to "/someContext#someVersion".
For apps with empty context we receive error:
Case #1:
<deploy url="{tomcat.manager}/text" username="${username}",
password="${password}" path="/ROOT##12345" war="${pathtowar}" />
Result:
Failed to deploy application at context path /ROOT##12345
Case #1:
<deploy url="{tomcat.manager}/text" username="${username}",
password="${password}" path="/##12345" war="${pathtowar}" />
Result:
Failed to deploy application at context path /##12345
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
--- Comment #3 from Sergey Tcherednichenko <se...@gmail.com> ---
thanks for including it to trunk :) and sorry for the problem with patch :)
though, this is a fix for ant task property "version", but the issue with using
"/##someversion" path is not fixed by this patch. Should I create another
issue, or maybe just call such usage as deprecated?
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
Sergey Tcherednichenko <se...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Version|7.0.40 |7.0.37
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
--- Comment #4 from Mark Thomas <ma...@apache.org> ---
I'd say treat using ##version in a path as deprecated / unsupported.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
Mark Thomas <ma...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #2 from Mark Thomas <ma...@apache.org> ---
Thanks for the patch (it was inverted by the way). It has been applied to trunk
and 7.0.x and will be included in 7.0.41 onwards.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
Sergey Tcherednichenko <se...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |sergeytch@gmail.com
--- Comment #1 from Sergey Tcherednichenko <se...@gmail.com> ---
Created attachment 30353
--> https://issues.apache.org/bugzilla/attachment.cgi?id=30353&action=edit
Ant "deploy" task should also take "version" as a parameter, and can extend
AbstractCatalinaCommandTask
This patch makes DeployTask to extend AbstractCatalinaCommandTask, so path and
version parameters are set uniformly ("version" parameter was missing in the
deploy task, and usage like ${context}##{version} did not work fine for the
root context).
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
[Bug 55035] Deploy ROOT webapp with version through text manager
using Ant
Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=55035
Sergey Tcherednichenko <se...@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Version|7.0.37 |7.0.40
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org