[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [ptp-dev] Problems trying to create resource manager using XML framework
|
Great, I'll take care of it.
Al
----- Greg Watson <g.watson@xxxxxxxxxxxx> wrote:
> Al,
>
> I had a quick look and I don't think there is any need to change RMProviderContributor. I think you just need to modify JAXBRMConfigurationSelectionFactory so that it ignores the id altogether. The id in the org.eclipse.ptp.rm.jaxb.core.JAXBResourceManagerConfigurations extension point is not used by anything in RMProviderContributor. fRMJAXBResourceManagers map can just be reduced to Map<String, URL>.
>
> Greg
>
> On Apr 30, 2011, at 3:38 PM, Albert L. Rossi wrote:
>
> > Greg, Dave
> >
> > This is a follow-up to the discussion about the "id" attribute on the org.eclipse.ptp.rm.jaxb.core.JAXBResourceManagerConfigurations extension point. That attribute points to the _selection factory_ (org.eclipse.ptp.ui.wizards.RMConfigurationSelectionFactory).
> >
> > If you change it, you have to implement a new selection factory as well. But there is no need, because
> >
> > org.eclipse.ptp.rm.jaxb.ui.wizards.JAXBRMConfigurationSelectionFactory
> >
> > is written to handle all contributions to this extension point. It's id is "org.eclipse.ptp.rm.jaxb.JAXBServiceProvider"
> >
> > At this point I'm not sure what the correct thing to do is; the id may be redundant, or it could be useful if a special corner-case configuration needs to be created (but that sort of defeats the purpose of this undertaking); hence I would be inclined to remove it from the extension point. However, that will inevitably entail changes to
> >
> > org.eclipse.ptp.ui.wizards.RMProviderContributor,
> >
> > private static void getRMConfigurationSelectionFactories() [ll. 58ff.].
> >
> > This is basically your domain Greg. Let me know what to do (I know you are currently overloaded).
> >
> > For the moment, Dave, continue to set that id to "org.eclipse.ptp.rm.jaxb.JAXBServiceProvider" for all contributors to the extension point.
> >
> > Al
> > _______________________________________________
> > ptp-dev mailing list
> > ptp-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/ptp-dev
>
> _______________________________________________
> ptp-dev mailing list
> ptp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/ptp-dev
>