Grails

Transitive dependencies not being resolved

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.2-M4
  • Fix Version/s: 1.2-RC1
  • Component/s: None
  • Labels:
    None

Description

When using a local resolver mavenLocal() and/or pom resolver, any dependencies that are stored in the local maven repository ~/.m2 are resolved, but their transitive dependencies are not.

This is very crucial I think, especially when working in an environment when local modules are used. For example in many projects, you'll have a few components build using maven and they are installed locally during install phase. Maven resolves those correctly, along with their transitive dependencies. Grails/Ivy seems to not. At least the local file resolver and pom resolver seem to ignore transitive deps for any local packages.

I added the mavenLocal() support back a few releases ago and I'd look into this issue, but I'm not sure if I'll have time. I think this is an ivy issue. Seems like file resolver doesn't enforce transitive dependencies. I'm wondering if we should use a different resolver? Maybe dual? I can't seem to find ivy documentation about this, so wondering if someone knows a bit more about how these resolvers work.

Thanks.

Ilya

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ilya Sterin added a comment -

Great, I'll test and submit a patch. Seems like that could be it.

Thanks Graeme.

Ilya

Show
Ilya Sterin added a comment - Great, I'll test and submit a patch. Seems like that could be it. Thanks Graeme. Ilya
Hide
Permalink
Graeme Rocher added a comment -

Hi Ilya can you verify whether changing the pattern to:

localMavenResolver.addArtifactPattern(
                    "${repoPath}/[organisation]/[module]/[revision]/[artifact]-[revision](-[classifier]).pom")

Works for you?

Show
Graeme Rocher added a comment - Hi Ilya can you verify whether changing the pattern to: 
localMavenResolver.addArtifactPattern(
                    "${repoPath}/[organisation]/[module]/[revision]/[artifact]-[revision](-[classifier]).pom")
Works for you?
Hide
Permalink
Ilya Sterin added a comment -

Graeme, I tried this a few weeks ago with no luck. I haven't had the time to look into it. I tried it again, still no luck. It resolves the specified local dependency, but doesn't do anything to the transitive dependencies of that local dep.

I'll take a look. How much time do we have until you plan on 1.2 release? I'd really want this to get fixed before this. I'll try to figure this out today unless anything major happens at work.

Ilya

Show
Ilya Sterin added a comment - Graeme, I tried this a few weeks ago with no luck. I haven't had the time to look into it. I tried it again, still no luck. It resolves the specified local dependency, but doesn't do anything to the transitive dependencies of that local dep. I'll take a look. How much time do we have until you plan on 1.2 release? I'd really want this to get fixed before this. I'll try to figure this out today unless anything major happens at work. Ilya
Hide
Permalink
Ilya Sterin added a comment -

Ok, I figured it out. I wish Ivy API was actually documented. After trying some combinations, it appears that it has to be a combination of ivy and artifact pattern. I'm attaching the patch.

Show
Ilya Sterin added a comment - Ok, I figured it out. I wish Ivy API was actually documented. After trying some combinations, it appears that it has to be a combination of ivy and artifact pattern. I'm attaching the patch.
Hide
Permalink
Ilya Sterin added a comment -

Ok, here is the patch. It all works. You might have to clean your local ivy cache.

Show
Ilya Sterin added a comment - Ok, here is the patch. It all works. You might have to clean your local ivy cache.
Hide
Permalink
Graeme Rocher added a comment -

Thanks, i'll make sure it gets into 1.2 final

Show
Graeme Rocher added a comment - Thanks, i'll make sure it gets into 1.2 final
Hide
Permalink
Graeme Rocher added a comment -

Thanks for the patch

Show
Graeme Rocher added a comment - Thanks for the patch

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.1#660-r161644) | Report a problem
Powered by a free Atlassian JIRA open source license for Grails project. Try JIRA - bug tracking software for your team.