Hi,
did bit of analysis:

The jjb patches which introduces verify jobs in Boron for jdk8 only had 
side-effect that they also change merge jobs to use jdk8.

For example this patch: 
https://github.com/opendaylight/releng-builder/commit/a0698da4a47a9ff477dd060cf6bc721c8792d95a
changes jjb template for YANGTools to build using jdk8 for Boron.

Even if no code does use Java 8 features set, when our custom maven plugins 
(karaf-plugin, yang-maven-plugin) are deployed into Nexus, Maven captures 
classpath of plugin
and breaks build if classpath is changed.
During YANGTools merge, Maven captured JDK8 classpath (which contains 
additional classes, even if we do not use them)
for YANG Maven Plugin and dissabled execution with following error on JDK7:

[ERROR] Failed to execute goal 
org.opendaylight.yangtools:yang-maven-plugin:0.9.0-SNAPSHOT:generate-sources 
(default) on project model-parent: Execution default of goal 
org.opendaylight.yangtools:yang-maven-plugin:0.9.0-SNAPSHOT:generate-sources 
failed: A required class was missing while executing 
org.opendaylight.yangtools:yang-maven-plugin:0.9.0-SNAPSHOT:generate-sources: 
java/util/function/Consumer
[ERROR] -----------------------------------------------------
[ERROR] realm =    
plugin>org.opendaylight.yangtools:yang-maven-plugin:0.9.0-SNAPSHOT


From: [email protected] 
[mailto:[email protected]] On Behalf Of Thanh Ha
Sent: Wednesday, January 20, 2016 5:58 AM
To: Abhijit Kumbhare <[email protected]>
Cc: openflowplugin-dev <[email protected]>; 
[email protected]
Subject: Re: [openflowplugin-dev] validate-autorelease jobs broken by 
openflowplugin

Looks like the revert didn't fix the compile issue. Upon closer look seems like 
the compile issue is only affecting JDK7. Did someone merge a change that only 
works in JDK8? Maybe an upstream project?

Thanh

On 19 January 2016 at 22:23, Abhijit Kumbhare 
<[email protected]<mailto:[email protected]>> wrote:
Being reverted by https://git.opendaylight.org/gerrit/#/c/33061/ .

On Wed, Jan 20, 2016 at 7:21 AM, Thanh Ha 
<[email protected]<mailto:[email protected]>> wrote:
Hi openflowplugin-devs,

I merged this patch on your behalf because openflowplugin's currently breaking 
all projects ability to validate their code:

    https://git.opendaylight.org/gerrit/33051

The issue was caused by this patch being merged:

    https://git.opendaylight.org/gerrit/#/c/31538/3

That patch was contributed before branch cutting activities last week and was 
not rebased so one of the pom parents were pointing to the Beryllium version. I 
think this patch should likely be reverted though because openflowplugin's 
merge job is currently failing so seems like something is broken by this patch 
that wasn't caught but I'll leave that decision to the project.

Regards,

Thanh

_______________________________________________
openflowplugin-dev mailing list
[email protected]<mailto:[email protected]>
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev


_______________________________________________
openflowplugin-dev mailing list
[email protected]
https://lists.opendaylight.org/mailman/listinfo/openflowplugin-dev

Reply via email to