Return-Path: X-Original-To: apmail-openjpa-dev-archive@www.apache.org Delivered-To: apmail-openjpa-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 22650108F8 for ; Wed, 9 Jul 2014 15:30:06 +0000 (UTC) Received: (qmail 8007 invoked by uid 500); 9 Jul 2014 15:30:06 -0000 Delivered-To: apmail-openjpa-dev-archive@openjpa.apache.org Received: (qmail 7956 invoked by uid 500); 9 Jul 2014 15:30:06 -0000 Mailing-List: contact dev-help@openjpa.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@openjpa.apache.org Delivered-To: mailing list dev@openjpa.apache.org Received: (qmail 7944 invoked by uid 99); 9 Jul 2014 15:30:05 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 09 Jul 2014 15:30:05 +0000 Date: Wed, 9 Jul 2014 15:30:05 +0000 (UTC) From: "Romain Manni-Bucau (JIRA)" To: dev@openjpa.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (OPENJPA-2514) ASM dependency revisited MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/OPENJPA-2514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14056343#comment-14056343 ] Romain Manni-Bucau commented on OPENJPA-2514: --------------------------------------------- Did you try java 8 feature in entities (stream + lambda)? > ASM dependency revisited > ------------------------ > > Key: OPENJPA-2514 > URL: https://issues.apache.org/jira/browse/OPENJPA-2514 > Project: OpenJPA > Issue Type: Bug > Components: kernel > Affects Versions: 2.2.2, 2.3.0, 2.4.0 > Reporter: Rick Curtis > Assignee: Rick Curtis > > OPENJPA-2171 introduced a dependency on xbean-asm-4 and with this JIRA I'd like to revisit that issue. > On the mailing list the TomEE folks have pointed that they need a java 6 release with xbean asm5(in support of runtime java8) and Kevin has even reverted a number of changes to trunk to rollback some of the java7(compile) support changes. I put together a patch that is similar to Romain's patch from OPENJPA-2171 that will selectively load whichever ASM library is available for a given environment. The attached patch will also upgrade the packaged asm dependency in openjpa-all to xbean-asm-5, but that could be changed depending on the environment. > While I understand, and appreciate the need to insulate OpenJPA from ASM(breaking) changes, I don't like the fact that to use OpenJPA a user is also required to use a special version of ASM. There are scenarios where a user might already have their own 'blessed' version of ASM in use, and they don't want to have yet another copy. Allowing for selecting loading of a given ASM library should be able to satisfy both use cases. > If this seems like a reasonable way forward, I could commit this change to trunk and 2.3.x. This would mean that the TomEE folks could cut another release from the 2.3.x branch, rather than creating yet another branch for limited benefit. > Thoughts, comments? -- This message was sent by Atlassian JIRA (v6.2#6252)