You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Andrew Phillips <no...@github.com> on 2014/02/07 15:31:27 UTC

[jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Looks like the workaround for Guava 1635 will be with us for a while on 1.7.x
You can merge this Pull Request by running:

  git pull https://github.com/jclouds/jclouds cleanup-5976159-1.7.x

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

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

-- Commit Summary --

  * Cleaning up checkstyle warnings caused by 5976159

-- File Changes --

    M core/src/main/java/org/jclouds/internal/BaseView.java (1)
    M core/src/main/java/org/jclouds/util/TypeToken2.java (4)

-- Patch Links --

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

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #830](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/830/) 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/pull/284#issuecomment-34780086

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
Committed to 1.7.x: [#1](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=4804edb7e873b9ce7161a38a69cf1706106c1d89), [#2](https://git-wip-us.apache.org/repos/asf?p=jclouds.git;a=commit;h=b1a2baee6227cd9cbad87189fb6b54747cbc8cdd)

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> jclouds-java-7-pull-requests #1054 UNSTABLE

Unrelated [test failure](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/org.apache.jclouds$jclouds-compute/1054/testReport/junit/org.jclouds.compute.callables/BlockUntilInitScriptStatusIsZeroThenReturnOutputTest/testloopUntilTrueOrThrowCancellationExceptionReturnsWhenPredicateIsTrueSecondTimeWhileNotCancelled/). And [Checkstyle](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1054/violations/) is indeed back down to 0!

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> 7u51 DEV@cloud job running

Successful:
```
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 46:42.311s
[INFO] Finished at: Fri Feb 07 10:42:21 EST 2014
[INFO] Final Memory: 108M/344M
[INFO] ------------------------------------------------------------------------
```

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #834](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/834/) 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/pull/284#issuecomment-34835412

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1060](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1060/) FAILURE
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/284#issuecomment-34828786

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> jclouds-pull-requests #588 SUCCESS

Weird. Maybe we don't need the repo thing after all. Let's try one more time...

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
Closed #284.

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1059](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1059/) FAILURE
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/284#issuecomment-34774795

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #821](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/821/) 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/pull/284#issuecomment-34446416

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
@nacx: Will wait for 1.7.1 to be pushed before merging...

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #832](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/832/) 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/pull/284#issuecomment-34830949

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #838](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/838/) 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/pull/284#issuecomment-34844689

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #597](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/597/) SUCCESS
This pull request looks good

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #582](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/582/) SUCCESS
This pull request looks good

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
Reopened #284.

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1054](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1054/) 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/284#issuecomment-34447632

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> Looks like it's looking in the wrong repo:

Waiting on https://github.com/jclouds/jclouds/pull/289

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #591](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/591/) SUCCESS
This pull request looks good

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
Reopened #284.

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
7u51 DEV@cloud job running: https://jclouds.ci.cloudbees.com/job/jclouds-java-7u51/35/console

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1068](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1068/) SUCCESS
This pull request looks good

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1063](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1063/) FAILURE
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/284#issuecomment-34831663

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> jclouds-java-7-pull-requests #1060 FAILURE

Still a problem with Checkstyle deps. Looks like it's looking in the wrong repo:
```
Caused by: org.apache.maven.plugin.PluginResolutionException: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:2.11 or one of its dependencies could not be resolved: Could not find artifact org.apache.jclouds:jclouds-resources:jar:1.7.2-SNAPSHOT in sonatype-nexus-snapshots
```

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
> jclouds-java-7-pull-requests #1059 FAILURE

Missing 1.7.2-SNAPSHOT deps. Let's try again later...
```
Could not find artifact org.apache.jclouds:jclouds-resources:jar:1.7.2-SNAPSHOT
```

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #587](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/587/) FAILURE
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/284#issuecomment-34774709

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #588](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/588/) SUCCESS
This pull request looks good

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

Re: [jclouds] Cleaning up checkstyle warnings caused by 5976159 (#284)

Posted by Andrew Phillips <no...@github.com>.
Closed #284.

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