RE: [dsdp-tm-dev] Package org.eclipse.rse.core.subsystems.util

["Oberhuber, Martin" <Martin.Oberhuber@xxxxxxxxxxxxx>]

Hi Dave,

cool - thanks.

Actually, I do not think we'll need a bug
for this, since classes are just moving to
a different package. For users, this is a
simple "organize imports" operation so we
won't need specific docs for this. 

It will go together with Dave's and your
other refactorings.

Thanks!
--
Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm 

> -----Original Message-----
> From: dsdp-tm-dev-bounces@xxxxxxxxxxx 
> [mailto:dsdp-tm-dev-bounces@xxxxxxxxxxx] On Behalf Of David Dykstal
> Sent: Monday, April 02, 2007 5:36 PM
> To: Target Management developer discussions
> Subject: Re: [dsdp-tm-dev] Package 
> org.eclipse.rse.core.subsystems.util
> 
> +1 on renaming and moving org.eclipse.rse.core.subsystems.util
> +1 on moving org.eclipse.rse.ui.subsystems
> +1 on moving types from org.eclipse.rse.core.servicesubsystem to  
> org.eclipse.rse.core.subsystems
> 
> Moving SubSystem and ServiceSubSystem to the core plugin will be  
> tough and my guess is that they will need some additional  
> refactoring. This may be one bit we'll have to leave until after M6.
> 
> I'll get to SubSystemHelpers and SubSystemConfigurationProxy 
> yet today.
> 
> On Apr 2, 2007, at 9:50 AM, Oberhuber, Martin wrote:
> 
> > Hi Dave's,
> >
> > I've been checking rse.doc.isv for broken links
> > in order to see if we have any API violations.
> > So far, it looks pretty good. But what I've seen
> > along the way is this:
> >
> > org.eclipse.rse.ui/subsystems/org.eclipse.rse.core.subsystems.util
> >
> > I believe this package is the UI adaptor (glue)
> > for subsystems. So the package name should better be
> >
> > org.eclipse.rse.ui/UI/org.eclipse.rse.ui.subsystems
> >
> > Agree? If yes, I'd create a bug and we should refactor it
> > for M6. BTW, Package org.eclipse.rse.ui.subsystems already
> > exists below the subsystems/ source path, I think this
> > should go below the UI/ source path.
> >
> > There are others which we should put out of UI and into Core.
> > I believe you're pretty close to doing so, correct DaveD?
> >   SubSystemHelpers  - should work already
> >   SubSystemConfigurationProxy - only problem is the ImageDescriptor,
> >      should be possible to do this via Property similar to what
> >      we already did for the systemType when making it Core
> >
> > The others (SubSystem, ServiceSubSystem) seem to be
> > still pretty hard to do. But it is probably time to
> > move the classes from
> >   
> org.eclipse.rse.ui/subsystems/org.eclipse.rse.core.servicesubsystem
> > into
> >   org.eclipse.rse.ui/subsystems/org.eclipse.rse.core.subsystems
> > since the corresponding interfaces are already in
> > that package.
> >
> > Cheers,
> > --
> > Martin Oberhuber
> > Wind River Systems, Inc.
> > Target Management Project Lead, DSDP PMC Member
> > http://www.eclipse.org/dsdp/tm
> > _______________________________________________
> > dsdp-tm-dev mailing list
> > dsdp-tm-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/dsdp-tm-dev
> 
> _______________________________________________
> dsdp-tm-dev mailing list
> dsdp-tm-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-tm-dev
>