Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-08-08 Thread Andrew Gaul
Pushed to master as 80a0256cc7919b32612cfa585e7f4224f412ea02. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/438#issuecomment-51665774

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-19 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #1480](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1480/) UNSTABLE Looks like there's a problem with this pull request --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/438#issuecomment-49500652

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-19 Thread Andrew Phillips
jclouds-java-7-pull-requests #1480 UNSTABLE Spurious [test failure](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/org.apache.jclouds$jclouds-compute/1480/testReport/junit/org.jclouds.compute.util/ConcurrentOpenSocketFinderTest/testChecksSocketsConcurrently/). +1, looks good

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-18 Thread Andrew Phillips
@@ -20,9 +20,7 @@ import java.io.InputStream; import java.io.IOException; -import com.google.common.io.InputSupplier; - -public interface Payload extends InputSupplierInputStream, Closeable { +public interface Payload extends Closeable { I resubmitted this pull request and we can

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-18 Thread Andrew Gaul
@aledsage wrote: Is deleting Strings2.toString(InputSupplier) rather than deprecating it too aggressive for 1.8? Agreed and I added back this method in 0df905a870563236bb095772ae00a1975ec3aece. --- Reply to this email directly or view it on GitHub:

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-18 Thread Andrew Gaul
@@ -20,9 +20,7 @@ import java.io.InputStream; import java.io.IOException; -import com.google.common.io.InputSupplier; - -public interface Payload extends InputSupplierInputStream, Closeable { +public interface Payload extends Closeable { By that, do you mean merging it to master once

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-18 Thread BuildHive
[jclouds » jclouds #1390](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1390/) 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:

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-18 Thread CloudBees pull request builder plugin
[jclouds-pull-requests #1009](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1009/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/438#issuecomment-49500572

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-17 Thread Aled Sage
Relating more to the already-merged commit https://github.com/jclouds/jclouds/commit/fb60d76704745317186ad700e9b7ce5a569a614d, and as commented on in the JCLOUDS-622 issue ... Is deleting `Strings2.toString(InputSupplier)` rather than deprecating it too aggressive for 1.8? Though guava 18 will

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-16 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #1459](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1459/) ABORTED --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/438#issuecomment-49153335

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-16 Thread Jeremy Daggett
@@ -20,9 +20,7 @@ import java.io.InputStream; import java.io.IOException; -import com.google.common.io.InputSupplier; - -public interface Payload extends InputSupplierInputStream, Closeable { +public interface Payload extends Closeable { @andrewgaul It looks like you deprecated the

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-16 Thread Andrew Gaul
@@ -20,9 +20,7 @@ import java.io.InputStream; import java.io.IOException; -import com.google.common.io.InputSupplier; - -public interface Payload extends InputSupplierInputStream, Closeable { +public interface Payload extends Closeable { @jdaggett I removed the interface changes and

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-16 Thread CloudBees pull request builder plugin
[jclouds-java-7-pull-requests #1472](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1472/) SUCCESS This pull request looks good --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/438#issuecomment-49246057

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-16 Thread BuildHive
[jclouds » jclouds #1382](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1382/) 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:

Re: [jclouds] JCLOUDS-622: Remove last vestiges of InputSupplier (#438)

2014-07-15 Thread BuildHive
[jclouds » jclouds #1363](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1363/) 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: