proxy-plus icon indicating copy to clipboard operation
proxy-plus copied to clipboard

Upgrade to latest (newer) asm library - support newer jdk's

Open ieugen opened this issue 3 years ago • 2 comments

Hello,

ASM has eveloved, the current release is from 2013. New versions support newer JDK releases. See https://asm.ow2.io/versions.html .

I did upgrade locally and it builds and the tests pass - but did not try any code against other jvm's .

Perhaps an automated build to test against multiple JVM might be a good option.

(⎈ |libris-k2-admin:prod)ieugen@daos-495:~/.../clojure/proxy-plus$ lein test
Retrieving org/ow2/asm/asm/9.2/asm-9.2.pom from central
Retrieving org/ow2/asm/asm-commons/9.2/asm-commons-9.2.pom from central
Retrieving org/ow2/asm/asm-tree/9.2/asm-tree-9.2.pom from central
Retrieving org/ow2/asm/asm-analysis/9.2/asm-analysis-9.2.pom from central
Retrieving org/ow2/asm/asm-util/9.2/asm-util-9.2.pom from central
Retrieving org/ow2/asm/asm-commons/9.2/asm-commons-9.2.jar from central
Retrieving org/ow2/asm/asm-tree/9.2/asm-tree-9.2.jar from central
Retrieving org/ow2/asm/asm-util/9.2/asm-util-9.2.jar from central
Retrieving org/ow2/asm/asm/9.2/asm-9.2.jar from central
Retrieving org/ow2/asm/asm-analysis/9.2/asm-analysis-9.2.jar from central

lein test com.rpl.proxy-plus-test

Ran 13 tests containing 52 assertions.
0 failures, 0 errors.

ieugen avatar Dec 27 '21 20:12 ieugen

Even if the new byte code features are not useful for proxy+, the lib should be really updated, because it's a runtime dependency.

alexanderkiel avatar May 24 '22 13:05 alexanderkiel

Works for me with ASM 9.2.

alexanderkiel avatar May 24 '22 13:05 alexanderkiel