[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159430073 this problem, was solved when this PR was merged. The problem we were talking about was the build of f5 plugin project that is not building with Java 1.8.

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159429530 This build problem happens to me on Java 1.7. Isn't this just a syntax error? The annotation markers are missing. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159418662 Interesting, so it looks like a Java problem --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159402363 1.8.0_05 works, 1.8.0_25 doesn't --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your projec

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159386959 i have some lying around: ``` 1.8.0_25, x86_64: "Java SE 8" /Library/Java/JavaVirtualMachines/jdk1.8.0_25.jdk/Contents/Home 1.8.0_

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159379578 @DaanHoogland , I have just checked with @cristofolini, I also had the problem you described here. I did not see that before because I had compiled withou

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159268349 My bad @remibergsma I counted @ProjectMoon as lgtm --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159267770 Guys, why didn't we get 2x LGTM on this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1110 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159175335 retried but it is repeatable :') ``` $ mvn -version Apache Maven 3.3.3 (7994120775791599e205a5524ec3e0dfe41d4a06; 2015-04-22T13:57:37+02:00) Mav

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159109110 That is the plugin, here everything compiled without problems. ~$ mvn -version Apache Maven 3.0.5 Maven home: /usr/share/maven Java version:

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159108087 macosx 10.11.1: ``` $ java -version java version "1.8.0_66" Java(TM) SE Runtime Environment (build 1.8.0_66-b17) Java HotSpot(TM) 64-Bit Serv

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159047345 @DaanHoogland, I have just ran "mvn clean install -Dnoredist -P developer,systemvm" with java 8 and everything was ok. What version of java 8 are you usin

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159018652 ``` mvn -P developer,systemvm clean install -Dnoredist ``` ``` [INFO]

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159017809 the problem seems to be only in using java8. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: [GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread Daan Hoogland
Did anybody build 4.6 noredist? I can not get it or master built. On Mon, Nov 23, 2015 at 6:33 PM, DaanHoogland wrote: > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/1110#issuecomment-159005121 > > The error occurs in 4.6 as well,

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1110#issuecomment-159005121 The error occurs in 4.6 as well, so I think this PR should go through --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] cloudstack pull request: PR1102 regression fix; half the annotatio...

2015-11-23 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1110 PR1102 regression fix; half the annotation was removed I am having troubles getting this to work but it passes the point at which the master slowbuild is broken. You can merge this pull r