[ http://issues.apache.org/jira/browse/JS2-223?page=comments#action_61528 ]
     
Ate Douma commented on JS2-223:
-------------------------------

Scott,

I've looked at this issue and how this was handled before I committed the new 
deployment implementation.
I'm a bit puzzled why we would need this feature. As far as I can see, the only 
effect this feature had (in the end) was a rename of the originally deployed 
war to a name as specified as portlet-app id.
In all cases, the portlet application name and identifier are set equal so I 
also don't understand why we have
these two fields instead of one.

I find this a bit confusing feature, certainly for the end users. 
Why wouldn't you just use a war filename equal to the portlet app id?

The only situation I can imagine you might want/need to supply a different name 
is with local (jetspeed- prefixed)
portlet applications. But then we could provide other means to that end with 
for example a different deploy folder,
different extension or whatever.
Being able to use the filename of the supplied war as identifier makes it much 
easier (and lighter) to validate/check if a redeploy is in order, instead of 
having to extract the portlet.xml first to be sure what pa is
actually deployed.

I see no real problems for reimplementing this feature again but I wonder if we 
really need it.
I don't mind doing it if I know why...
 

> New deployment not using portlet app id for web app name
> --------------------------------------------------------
>
>          Key: JS2-223
>          URL: http://issues.apache.org/jira/browse/JS2-223
>      Project: Jetspeed 2
>         Type: Bug
>   Components: Deployment
>     Versions: 2.0-M2
>     Reporter: Scott T Weaver
>     Assignee: Ate Douma

>
> Deployment used to use the <portlet-app id> attribute to override the war 
> name when deploying portlet apps.  This is no longer the case.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


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

Reply via email to