Details
-
Type:
Improvement
-
Status:
Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.1-beta-2
-
Fix Version/s: 1.1-rc-1
-
Component/s: groovy-jdk
-
Labels:None
-
Number of attachments :
Description
I'm using the groovy-all-1.1-beta-2.jar in a Maven project that is also using a recent version of Castor (1.1.2.1). Unfortunately, the groovy-all pom.xml includes a runtime dependency to castor-0.9.9.jar. 0.9.9 is several years old and should be updated to a more recent version of Castor.
Attachments
Activity
I'll try to provide a patch in the next day or two.
Shouldn't the dependency be a compile-time dependency instead?
It doesn't necessarily matter whether the dependency is declared as runtime or compile (as long as the unit tests pass 
). In the end, either one will result in the jar be included at runtime when building an assembly.
The current workaround is to exclude the old castor jar in Maven2 projects that use both groovy-all and a more recent castor. Like so:
<dependency> <groupId>groovy</groupId> <artifactId>groovy-all</artifactId> <version>1.1-beta-2</version> <scope>runtime</scope> <exclusions> <exclusion> <artifactId>castor</artifactId> <groupId>castor</groupId> </exclusion> </exclusions> </dependency>
Shouldn't the dependency be a compile-time dependency instead?It doesn't necessarily matter whether the dependency is declared as runtime or compile (as long as the unit tests pass
). In the end, either one will result in the jar be included at runtime when building an assembly.
The current workaround is to exclude the old castor jar in Maven2 projects that use both groovy-all and a more recent castor. Like so:
<dependency> <groupId>groovy</groupId> <artifactId>groovy-all</artifactId> <version>1.1-beta-2</version> <scope>runtime</scope> <exclusions> <exclusion> <artifactId>castor</artifactId> <groupId>castor</groupId> </exclusion> </exclusions> </dependency>
Could you provide a patch with changes to the pom that would suite your needs?
Shouldn't the dependency be a compile-time dependency instead?