From 58d24d76c4850025cafa68918db99e8e6b3b0582 Mon Sep 17 00:00:00 2001 From: Nikolay Krasko Date: Thu, 19 May 2016 21:21:06 +0300 Subject: [PATCH] Check there's no direct usages of asm (org.objectweb.asm) library in project Such usages should be replaced with asm from idea (org.jetbrains.org.objectweb.asm) --- .../org/jetbrains/kotlin/code/CodeConformanceTest.kt | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt b/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt index 11bd324686f..b110346a04f 100644 --- a/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt +++ b/compiler/tests/org/jetbrains/kotlin/code/CodeConformanceTest.kt @@ -30,6 +30,7 @@ class CodeConformanceTest : TestCase() { "android.tests.dependencies", "core/reflection.jvm/src/kotlin/reflect/jvm/internal/pcollections", "libraries/tools/kotlin-reflect/target/copied-sources", + "libraries/tools/binary-compatibility-validator/src/main/kotlin/org.jetbrains.kotlin.tools", "dependencies", "js/js.translator/qunit/qunit.js", "libraries/tools/kotlin-js-tests/src/test/web/qunit.js", @@ -96,7 +97,14 @@ class CodeConformanceTest : TestCase() { { source -> "kotlin.reflect.jvm.internal.impl" in source } - ) + ), + TestData( + "%d source files contain references to package org.objectweb.asm.\n" + + "Package org.jetbrains.org.objectweb.asm should be used instead to avoid troubles with different asm versions in classpath. " + + "Please consider changing the package in these files:\n%s", + { source -> + " org.objectweb.asm" in source + }) ) for (sourceFile in FileUtil.findFilesByMask(SOURCES_FILE_PATTERN, File("."))) {