<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=ISO-8859-15"
 http-equiv="Content-Type">
  <title></title>
</head>
<body bgcolor="#ffffff" text="#000000">
Max Rydahl Andersen wrote:
<blockquote cite="mid:op.uavf45hqw1tq5a@reddevil" type="cite">
  <blockquote type="cite">
    <pre wrap="">The following projects exist in our codebase.  They all use the Generic
Server adapter.

org.jboss.tools.jst.server.jboss
org.jboss.tools.jst.server.jetty
org.jboss.tools.jst.server.jrun
org.jboss.tools.jst.server.resin

At the very least, I recommend that the jboss adapter be removed from
our SVN.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
How many of these do we actually use in our build ?!
  </pre>
</blockquote>
They are excluded from builds since 1.0.0.Beta1.<br>
<blockquote cite="mid:op.uavf45hqw1tq5a@reddevil" type="cite">
  <pre wrap="">
  </pre>
  <blockquote type="cite">
    <pre wrap="">The only suggestion I'd have is to unify all of them into one plugin in
the "AS" module,
rather than keep them bundled in the "jst" module bundled with jst.web
and jst.jsp.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
Does anyone know if they are actually still relevant ? I recall them as being
"disabled" on purpose since they were outofdate...Denis/Alexey do you know ?
  </pre>
</blockquote>
They are not used in builds, but I'm not sure if somebody using it
right from svn.<br>
I'm sure that jboss and resin can be removed, because we have first in
AS component and WTP has Resin v3.0 adapter.<br>
<br>
Jetty and JRun are candidates for legacy :)<br>
<br>
Denis<br>
<br>
<blockquote cite="mid:op.uavf45hqw1tq5a@reddevil" type="cite">
  <pre wrap="">
  </pre>
  <blockquote type="cite">
    <pre wrap="">I also suggest that org.jboss.tools.jst.firstrun move to the "as" module
as well,
though I'm not quite as sure about this.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
You mean the "auto-configuration" code of AS, datasources etc. ? I think this 
needs cleaning up and some of it needs to be seperated out.

We might also want to introduce the concept of EAP "runtimes" instead of just having
individual configurations of it (datasource, as, seam etc.)

  </pre>
  <blockquote type="cite">
    <pre wrap="">Finally, if no one else has any problem with this, I also think moving the
"start server" / "stop server" tool button into the as.ui plugin...
though the
effects of that will need to be looked into since it's mixed in with the
org.jboss.tools.jst.web plugin.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
I don't see any big problems with this if you are ok of being "generic" AS module and not
just JBoss AS...but yes needs investigation on how much coupling it has to the rest.

-max
_______________________________________________
jbosstools-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:jbosstools-dev@lists.jboss.org">jbosstools-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/jbosstools-dev">https://lists.jboss.org/mailman/listinfo/jbosstools-dev</a>
  </pre>
</blockquote>
<br>
</body>
</html>