Re: [jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-17 Thread Andrew Phillips
> Squashed, amended the commit message and pushed to master as ed2a6b47

Thanks, Ignasi!

-- 
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-karaf/pull/83#issuecomment-247806799

Re: [jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-17 Thread Ignasi Barrera
Squashed, amended the commit message and pushed to master as 
[ed2a6b47](http://git-wip-us.apache.org/repos/asf/jclouds-karaf/commit/ed2a6b47).

-- 
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-karaf/pull/83#issuecomment-247806439

Re: [jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-17 Thread Ignasi Barrera
Closed #83.

-- 
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-karaf/pull/83#event-793136018

Re: [jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-17 Thread Andrew Phillips
Oops :-( Sorry, sloppy testing on my part. Changes look good to me - thanks, 
@nacx!

When you merge, could you change the commit title to reference the issue 
number? Thanks!

-- 
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-karaf/pull/83#issuecomment-247788653

Re: [jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-16 Thread Ignasi Barrera
@nacx pushed 1 commit.

a36ede1  Fix log message


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/83/files/5eeb5cfbfacaf8c05e7057727638f3bf2297717b..a36ede13de7f7ea873827b985607ffb61425354d


[jclouds/jclouds-karaf] Fix OSGiScriptEngineManager package (#83)

2016-09-16 Thread Ignasi Barrera
Minor issue after moving the classes that caused the jclouds commands not to be 
installed.

/cc @demobox 
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-karaf/pull/83

-- Commit Summary --

  * Fix OSGiScriptEngineManager package

-- File Changes --

M commands/src/main/resources/OSGI-INF/blueprint/jclouds-commands.xml (2)

-- Patch Links --

https://github.com/jclouds/jclouds-karaf/pull/83.patch
https://github.com/jclouds/jclouds-karaf/pull/83.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-karaf/pull/83