diff options
author | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-04-28 13:54:33 +0700 |
---|---|---|
committer | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-04-28 13:54:33 +0700 |
commit | 3f127347ceeb042f6624c9ce956cc31555bc376d (patch) | |
tree | d9c66f89753913e10226a318ea6236990ce7b41c /ajdoc | |
parent | 48e1b14cb46f389ed7cb78312d8c8ab61f1fba05 (diff) | |
download | aspectj-3f127347ceeb042f6624c9ce956cc31555bc376d.tar.gz aspectj-3f127347ceeb042f6624c9ce956cc31555bc376d.zip |
Fix missing dependencies in module 'run-all-junit-tests'
Some runtime dependencies are reported as unused in Maven Dependency
Plugin goal 'dependency:analyze', but actually they are needed. I
noticed by chance when running RunTheseBeforeYouCommitTests in IntelliJ
IDEA for the first time after a while and dependency modules could not
find classes.
Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
Diffstat (limited to 'ajdoc')
-rw-r--r-- | ajdoc/pom.xml | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/ajdoc/pom.xml b/ajdoc/pom.xml index 22252b570..787f27600 100644 --- a/ajdoc/pom.xml +++ b/ajdoc/pom.xml @@ -18,7 +18,7 @@ <artifactId>maven-dependency-plugin</artifactId> <configuration> <usedDependencies> - <!-- The tests need this during runtime, even though no direct usage is in our classes --> + <!-- The tests need these during runtime, even though no direct usage is in our classes --> <usedDependency>com.github.olivergondza:maven-jdk-tools-wrapper</usedDependency> </usedDependencies> </configuration> @@ -27,6 +27,7 @@ </build> <dependencies> + <dependency> <groupId>org.aspectj</groupId> <artifactId>util</artifactId> @@ -48,12 +49,6 @@ <version>${project.version}</version> </dependency> <dependency> - <!-- enables easy dependency on tools.jar --> - <groupId>com.github.olivergondza</groupId> - <artifactId>maven-jdk-tools-wrapper</artifactId> - <version>0.1</version> - </dependency> - <dependency> <groupId>org.aspectj</groupId> <artifactId>asm-renamed</artifactId> </dependency> @@ -64,5 +59,16 @@ <scope>test</scope> </dependency> + <!-- + The tests need these during runtime, even though no direct usage is in our classes. + See also 'usedDependencies' in maven-dependency-plugin configuration. + --> + <dependency> + <!-- enables easy dependency on tools.jar --> + <groupId>com.github.olivergondza</groupId> + <artifactId>maven-jdk-tools-wrapper</artifactId> + <version>0.1</version> + </dependency> + </dependencies> </project> |