Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-26 Thread Andrew Phillips
> Yeah, let's merge it!

;-) Merge to 
[master](https://git-wip-us.apache.org/repos/asf?p=jclouds-cli.git;a=commit;h=cf1d92f1cac00bb4df54c15bd91b09fbae68)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256498369

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-26 Thread Andrew Phillips
Closed #31.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#event-837828686

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-26 Thread Ignasi Barrera
Yeah, let's merge it!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256392542

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-26 Thread Andrew Phillips
> I've also successfully tested basic operations with Azure ARM

Good to merge and perhaps call out testing of the CLI as a task for verifiers 
for the 2.0.0. release?

Thanks for trying this out, @nacx!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256386363

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-26 Thread Ignasi Barrera
I've also successfully tested basic operations with Azure ARM

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256365123

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-25 Thread Andrew Gaul
I successfully tested a few operations with aws-s3.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256178245

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-25 Thread Andrew Phillips
> Which is the status of this PR?

I'd still like to merge it; I think we were hoping someone else would get a 
chance to verify that this doesn't break anything...

Thanks for the ping, @nacx!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256085392

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-10-25 Thread Ignasi Barrera
@demobox quick reminder :) Which is the status of this PR?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31#issuecomment-256039710

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-08 Thread Andrew Phillips
> @@ -137,13 +137,13 @@
>  
>  false
>  
> -
> +

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-07 Thread Ignasi Barrera
> @@ -137,13 +137,13 @@
>  
>  false
>  
> -
> +

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-06 Thread Andrew Phillips
> \ No newline at end of file
> +featuresBoot=jclouds-chef,jclouds-aws-ec2,jclouds-aws-s3

`config:list` and other commands still seem to be present

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31/files/0061346d112da919d20fe52c51973c6f46ef6ec2..bc003de96a72d889203a0b043b03a7161f93d7cb#r77762920

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-06 Thread Andrew Phillips
> @@ -39,11 +39,12 @@ 
> org/apache/aries/blueprint/org.apache.aries.blueprint.api/${aries.blueprint.api.
>  
> org/apache/aries/blueprint/org.apache.aries.blueprint.core/${aries.blueprint.core.version}/org.apache.aries.blueprint.core-${aries.blueprint.core.version}.jar=20
>  
> org/apache/aries/blueprint/org.apache.aries.blueprint.cm/${aries.blueprint.cm.version}/org.apache.aries.blueprint.cm-${aries.blueprint.cm.version}.jar=20
>  
> -org/apache/karaf/shell/org.apache.karaf.shell.console/${karaf.version}/org.apache.karaf.shell.console-${karaf.version}.jar=29
> +org/apache/karaf/shell/org.apache.karaf.shell.console/${karaf.version}/org.apache.karaf.shell.console-${karaf.version}.jar=25

See 
https://github.com/apache/karaf/blob/karaf-2.3.11/assemblies/apache-karaf/src/main/filtered-resources/minimal/startup.properties
 for the motivation for these changes

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31/files/0061346d112da919d20fe52c51973c6f46ef6ec2#r77762880

Re: [jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-06 Thread Andrew Phillips
> @@ -137,13 +137,13 @@
>  
>  false
>  
> -
> +

[jclouds/jclouds-cli] Misc cleanup (#31)

2016-09-06 Thread Andrew Phillips

You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-cli/pull/31

-- Commit Summary --

  * Align the startup.properties file with Karaf 2.3.11
  * Remove a non-existent feature from the list of features to boot on startup
  * Tentatively remove Guava from the system deps for the distribution

-- File Changes --

M assembly/src/main/assembly/unix.xml (4)
M assembly/src/main/assembly/win.xml (4)
M assembly/src/main/filtered-resources/etc/org.apache.karaf.features.cfg (2)
M assembly/src/main/filtered-resources/etc/startup.properties (7)
M project/pom.xml (2)

-- Patch Links --

https://github.com/jclouds/jclouds-cli/pull/31.patch
https://github.com/jclouds/jclouds-cli/pull/31.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-cli/pull/31