You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@jclouds.apache.org by Ignasi Barrera <no...@github.com> on 2013/06/05 09:47:40 UTC

[jclouds-chef] JCLOUDS-107: Use Guice multibinding extensions (#4)

Remove the hardcoded copy of Guice multibindings and add the multibindings extension dependency instead.

cc @adriancole
You can merge this Pull Request by running:

  git pull https://github.com/nacx/jclouds-chef 107-multibindings

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds-chef/pull/4

-- Commit Summary --

  * JCLOUDS-107: Use Guice multibinding extensions

-- File Changes --

    M core/pom.xml (6)
    M core/src/main/java/org/jclouds/chef/util/ChefUtils.java (2)
    M core/src/main/java/org/jclouds/ohai/config/JMXOhaiModule.java (2)
    M core/src/main/java/org/jclouds/ohai/config/OhaiModule.java (2)
    D core/src/main/java/org/jclouds/ohai/config/multibindings/Element.java (37)
    D core/src/main/java/org/jclouds/ohai/config/multibindings/MapBinder.java (539)
    D core/src/main/java/org/jclouds/ohai/config/multibindings/Multibinder.java (409)
    D core/src/main/java/org/jclouds/ohai/config/multibindings/RealElement.java (63)
    D core/src/main/java/org/jclouds/ohai/config/multibindings/package-info.java (21)
    M core/src/test/java/org/jclouds/ohai/config/OhaiModuleTest.java (2)

-- Patch Links --

https://github.com/jclouds/jclouds-chef/pull/4.patch
https://github.com/jclouds/jclouds-chef/pull/4.diff

Re: [jclouds-chef] JCLOUDS-107: Use Guice multibinding extensions (#4)

Posted by Ignasi Barrera <no...@github.com>.

Merged

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/4#issuecomment-19053940

Re: [jclouds-chef] JCLOUDS-107: Use Guice multibinding extensions (#4)

Posted by BuildHive <no...@github.com>.

[jclouds » jclouds-chef #67](https://buildhive.cloudbees.com/job/jclouds/job/jclouds-chef/67/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/4#issuecomment-18960655

Re: [jclouds-chef] JCLOUDS-107: Use Guice multibinding extensions (#4)

Posted by Ignasi Barrera <no...@github.com>.

Closed #4.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-chef/pull/4