i think leave it `brooklyn/software/winrm` in the old hierarchy, and in the new hierarchy `brooklyn-server/software/winrm` alongside `brooklyn/software/base`
`locations/*` is intended for implementations of `Location` --a On 9 December 2015 at 22:36, John McCabe <[email protected]> wrote: > Does locations/winrm sound reasonable? (rather than reverting back to > /core) > > On Wed, Dec 9, 2015 at 5:19 PM, Aled Sage <[email protected]> wrote: > > > +1 > > > > Let's move winrm to brooklyn-server. > > > > > > > > On 09/12/2015 16:56, Hadrian Zbarcea wrote: > > > >> That should be (or at least become) a soft dependency, not a hard one. > >> That aside, the winrm piece probably belongs in server. > >> > >> Hadrian > >> > >> On 12/09/2015 11:27 AM, John McCabe wrote: > >> > >>> Alex, all, (fyi Hadrian/Ciprian), > >>> > >>> > >>> * fix pom files on result of `rearrange-incubator.sh` script so it > builds > >>>> > >>>>> (make this a diff / git cherry-pick we can just apply once all PRs > are > >>>>>> merged?) > >>>>>> > >>>>>> John also said to me that he would take a look at this problem. He > was > >>>>> temporarily prevented from doing so by a bug in the split script > which > >>>>> broke TEMP-brooklyn-server, but that dodgy repo should now be fixed. > >>>>> > >>>>> John - any update? > >>>> > >>>> > >>>>> > >>>>>> Just after having a chat with Richard about this, and am in the > >>> process of > >>> working through it. > >>> > >>> Have hit one problem so far, a circular dependency between > >>> brooklyn-server > >>> and brooklyn-library from some of the OSGI-ification changes. > >>> > >>> The 'Brooklyn Jclouds Location Targets' module in brooklyn-server has a > >>> dependency on the 'brooklyn-software-winrm' artifact from > >>> brooklyn-library, > >>> but brooklyn-library itself depends on brooklyn-server. > >>> > >>> Any suggestions on what we should do with it? There's mention of > >>> additional > >>> cleanup required later in the commit (see pull#957 [1] (JIRA > BROOKLYN-182 > >>> [2])) > >>> > >>> /John > >>> > >>> [1] https://github.com/apache/incubator-brooklyn/pull/957 > >>> [2] https://issues.apache.org/jira/browse/BROOKLYN-182 > >>> > >>> > > >
