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

Posted to notifications@jclouds.apache.org by Adrian Cole <no...@github.com> on 2014/10/05 23:49:39 UTC

[jclouds] Replaced incorrect use of ConfiguresRestClient and deleted old rest client modules. (#550)

Orphaned files and find/replace errors.

Merging this allows us to do a follow-up removal of the `@ConfiguresRestClient` annotation.
You can merge this Pull Request by running:

  git pull https://github.com/adriancole/jclouds adrian.whoops-bad-annotations

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

  https://github.com/jclouds/jclouds/pull/550

-- Commit Summary --

  * JCLOUDS-40 Replaced incorrect use of @ConfiguresRestClient and deleted old rest client modules.

-- File Changes --

    M apis/elasticstack/src/main/java/org/jclouds/elasticstack/config/ElasticStackHttpApiModule.java (4)
    M apis/openstack-nova/src/main/java/org/jclouds/openstack/nova/v2_0/config/NovaHttpApiModule.java (4)
    D apis/sts/src/main/java/org/jclouds/aws/config/AWSRestClientModule.java (93)
    D apis/sts/src/main/java/org/jclouds/aws/config/FormSigningRestClientModule.java (64)
    D apis/sts/src/main/java/org/jclouds/aws/config/WithZonesFormSigningRestClientModule.java (45)
    M providers/glesys/src/main/java/org/jclouds/glesys/config/GleSYSHttpApiModule.java (4)
    M providers/gogrid/src/main/java/org/jclouds/gogrid/config/GoGridHttpApiModule.java (4)
    M providers/gogrid/src/test/java/org/jclouds/gogrid/features/BaseGoGridApiTest.java (4)
    M providers/gogrid/src/test/java/org/jclouds/gogrid/features/BaseGoGridHttpApiExpectTest.java (4)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/550.patch
https://github.com/jclouds/jclouds/pull/550.diff

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

Re: [jclouds] Replaced incorrect use of ConfiguresRestClient and deleted old rest client modules. (#550)

Posted by CloudBees pull request builder plugin <no...@github.com>.

[jclouds-pull-requests-java-6 #141](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/141/) SUCCESS
This pull request looks good

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

Re: [jclouds] Replaced incorrect use of ConfiguresRestClient and deleted old rest client modules. (#550)

Posted by Adrian Cole <no...@github.com>.

cherry-picked into 1.8.x and master

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

Re: [jclouds] Replaced incorrect use of ConfiguresRestClient and deleted old rest client modules. (#550)

Posted by Andrew Phillips <no...@github.com>.

Pending PR builders, +1 - looks good to me. Thanks, @adriancole!

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

Re: [jclouds] Replaced incorrect use of ConfiguresRestClient and deleted old rest client modules. (#550)

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

[jclouds » jclouds #1716](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1716/) FAILURE
Looks like there's a problem with this pull request
[(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/pull/550#issuecomment-57972497