brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alex Heneveld <alex.henev...@cloudsoftcorp.com>
Subject Re: [PROPOSAL] Remove rest/rest-client from brooklyn-server
Date Fri, 09 Sep 2016 09:54:28 GMT

i lean towards this:

     brooklyn-client/
         cli/
             **/*.go
         java/
             src/main/java/**/*.java

or

     brooklyn-client/
         cli/
             **/*.go
         bindings/
             java/
                 src/main/java/**/*.java

but I could live with this:

     brooklyn-client-cli/
         **/*.go
     brooklyn-client-bindings/
         java/
            src/main/java/**/*.java

reason for preferring a single client project is to keep the top-level 
list of brooklyn sub-projects tighter (server and client has a nice 
symmetry ... and given how much larger server is than all the clients, 
it'd be odd and distracting to have multiple client projects)

--a


On 09/09/2016 10:17, Robert Moss wrote:
> +1, prefer separate repo, though.
>
> Robert
>
> On 9 September 2016 at 10:03, Andrea Turli <andrea.turli@cloudsoftcorp.com>
> wrote:
>
>> Hi,
>>
>> I'd like to move `rest/rest-client` out of `brooklyn-server`
>>
>> brooklyn-server has a dependency on resteasy which is used only by the
>> rest/rest-client module. Having resteasy and cxf (two jax-rs
>> implementations) in the classpath looks redundant to me.
>> Also the extra work needed to support osgi bundles for both. Notice
>> also jaxrs implementation are not really osgi friendly (ask
>> @googlielmo and @neykov for more details :P)
>>
>> So I think a diet for brooklyn-server is not a bad thing :)
>>
>> Said that, we are discussing with @geomacy
>> (https://github.com/apache/brooklyn-client/pull/25) if
>> `brooklyn-client` is the right new place for moving the rest java
>> client.
>>
>> We'd like to have more devs involved in this discussion, is it better
>> moving the java client to `brooklyn-client` (option1) or create a new
>> project (option2), say `brooklyn-java-client` or
>> `brooklyn-rest-clients` maybe? Else (option3)?
>>
>> Thanks,
>> Andrea
>>


Mime
View raw message