[
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
|
Done. The id is now totally arbitrary. -A;
----- Albert L. Rossi <arossi@xxxxxxxxxxxxx> wrote:
> 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
> >
>
> _______________________________________________
> ptp-dev mailing list
> ptp-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/ptp-dev
>
