[jbosstools-dev] Weird code in Seam runtime management...

[Max Rydahl Andersen]

Denis Golovin wrote:
> Max,
> 
> how fast you forgot about org.jboss.tools.jst component :)

Sure - but our seam component is not in org.jboss.tools.jst and it ain't 
  a standard as "JEE standard tools" were for.

/max

> denis.
> 
> Max Rydahl Andersen wrote:
>> Third question:
>>
>> Why is the seam facet id "jst.seam" ?
>>
>> "jst" is the name domain of Eclipse WTP's jst modules; what does our 
>> seam facet have to do with that ?
>>
>> /max
>>
>>> Another question:
>>>
>>> Why do we need to have our own copy of application.xml when seam-gen 
>>> has one ?
>>>
>>> We need to get seam-gen fixed if this is broken somehow.
>>>
>>> /max
>>>
>>>> Hi Denis,
>>>>
>>>> I stepped over some code I don't understand why is needed in Seam 
>>>> runtime management.
>>>>
>>>> SeamRuntimeListConverter1 (weird class name, why an 1 at the end ?), 
>>>> seem to do some really weird conversion between maps and strings on 
>>>> the form
>>>>
>>>> "name|some Name|version|1123123|homeDir|somedir|default|true"
>>>>
>>>> ...why are we storing such weird strings in preferences ?
>>>> (Is that normal for eclipse preferences ? That looks very 
>>>> unmaintainable)
>>>>
>>>> Why isn't the info about what runtime is default a single named 
>>>> preference instead of maintained on each runtime ?
>>>>
>>>> And why are these classes in a package called 
>>>> org.seam.core.project.facet ? These things are and should be 
>>>> independent of WTP facets (and SeamRuntimeListConverter1 sound like 
>>>> something that should be inside the internal package)
>>>>
>>>> /max
>>>> _______________________________________________
>>>> jbosstools-dev mailing list
>>>> jbosstools-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/jbosstools-dev
>>>
>>> _______________________________________________
>>> jbosstools-dev mailing list
>>> jbosstools-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/jbosstools-dev
>>
>