Modify

Opened 20 months ago

Last modified 19 months 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 Changed 20 months ago by Don-vip

@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 Changed 20 months ago by anonymous

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 20 months ago by Don-vip (previous) (diff)

comment:3 Changed 20 months ago by sebastic

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 20 months ago by sebastic (previous) (diff)

comment:4 Changed 20 months ago by SimonPoole

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 Changed 20 months ago by Don-vip

In 18079/josm:

see #21140 - upgrade to openinghoursparser 0.24.0

comment:6 Changed 20 months ago by Don-vip

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

comment:7 Changed 20 months ago by sebastic

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 in reply to:  7 Changed 20 months ago by Don-vip

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 Changed 20 months ago by Don-vip

Milestone: 21.0721.08

comment:10 Changed 20 months ago by Don-vip

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

comment:11 Changed 19 months ago by Don-vip

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.
to The owner will be changed from Don-vip to the specified user.
The owner will change to Don-vip
as duplicate The resolution will be set to duplicate.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.