Opened 4 years ago
Last modified 4 years ago
#21140 assigned enhancement
Include transitive dependencies in josm-sources.jar
Reported by: | Don-vip | Owned by: | Don-vip |
---|---|---|---|
Priority: | normal | Milestone: | Longterm |
Component: | Core | Version: | |
Keywords: | Cc: | sebastic |
Description
problematic when using josm-sources.jar, because it lacks the sources of transitive dependencies (e.g. org.jetbrains.annotations
for ch.poole.openinghoursparser
).
Attachments (0)
Change History (11)
comment:1 by , 4 years ago
comment:2 by , 4 years ago
The code does, e.g.:
package ch.poole.openinghoursparser; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable;
Several of the other files use the same.
comment:3 by , 4 years ago
The dependency is also recorded in build.gradle
:
dependencies { compileOnly 'org.jetbrains:annotations:19.0.0' testCompileOnly 'org.jetbrains:annotations:19.0.0' testCompile 'junit:junit:4.12' }
https://github.com/simonpoole/OpeningHoursParser/blob/master/build.gradle#L192
Issue reported upstream: https://github.com/simonpoole/OpeningHoursParser/issues/67
comment:4 by , 4 years ago
This is a bit of a pain, but the real issue is that pom files don't have a scope that is equivalent to gradles 'compileOnly' and forcing users of the binary artifact to add a completely pointless dependency, doesn't really make sense.
I've add a comprise solution in 0.24.0 that will hopefully work, if it doesn't pls complain.
follow-up: 8 comment:7 by , 4 years ago
xmpcore for metadataextractor is another transitive dependency for which the sources need to be downloaded separately.
The recent update to 6.1.11 is also problematic because there is no source JAR for it.
comment:8 by , 4 years ago
Replying to sebastic:
The recent update to 6.1.11 is also problematic because there is no source JAR for it.
I've sent an e-mail to the maintainer at Adobe. Let's see.
comment:9 by , 4 years ago
Milestone: | 21.07 → 21.08 |
---|
comment:10 by , 4 years ago
No answer to my e-mail, I just created https://github.com/adobe/XMP-Toolkit-SDK/issues/41
comment:11 by , 4 years ago
Milestone: | 21.08 → Longterm |
---|
@sebastic when I look at https://repo1.maven.org/maven2/ch/poole/OpeningHoursParser/0.23.4/OpeningHoursParser-0.23.4.pom, openinghoursparser does not depend on
org.jetbrains.annotations
?