Modify

Opened 3 years ago

Last modified 3 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

From ticket:20768#comment:5

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 Don-vip, 3 years ago

@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?

comment:2 by anonymous, 3 years ago

The code does, e.g.:

package ch.poole.openinghoursparser;

import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;

https://github.com/simonpoole/OpeningHoursParser/blob/master/src/main/java/ch/poole/openinghoursparser/DateRange.java

Several of the other files use the same.

Last edited 3 years ago by Don-vip (previous) (diff)

comment:3 by sebastic, 3 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

Last edited 3 years ago by sebastic (previous) (diff)

comment:4 by SimonPoole, 3 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.

comment:5 by Don-vip, 3 years ago

In 18079/josm:

see #21140 - upgrade to openinghoursparser 0.24.0

comment:6 by Don-vip, 3 years ago

Thanks Simon! @sebastic are there some other missing dependencies?

comment:7 by sebastic, 3 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.

in reply to:  7 comment:8 by Don-vip, 3 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 Don-vip, 3 years ago

Milestone: 21.0721.08

comment:10 by Don-vip, 3 years ago

No answer to my e-mail, I just created https://github.com/adobe/XMP-Toolkit-SDK/issues/41

comment:11 by Don-vip, 3 years ago

Milestone: 21.08Longterm

Modify Ticket

Change Properties
Set your email in Preferences
Action
as assigned The owner will remain Don-vip.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from Don-vip to the specified user. Next status will be 'new'.
Next status will be 'needinfo'. The owner will be changed from Don-vip to Don-vip.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.