Transitive dependencies in a of a module in a multiproject should be exported

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 2.0
  • Fix Version/s: None
  • Labels:
    None

Description

Currently the maven-idea-plugin treats all dependencies the same in that it just adds all of them to each module

But what I would like is that if I have dependencies that are transative as a result of it depending upon another module in my multimodule project then I would like the module to be dependent only upon its upper module and this upper module to export all its dependencies.

This stops cluttering of alot of dependencies being put in for each module when they can all just be exported from the top level dependency.

  1. Text File
    MIDEA-80-maven-idea-plugin.patch
    11/Mar/07 11:52 PM
    10 kB
    Michael Mekaail
  2. Text File
    MIDEA-80-maven-idea-plugin.patch
    12/Dec/06 12:47 AM
    7 kB
    Michael Mekaail

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Mekaail added a comment -

Fixed
However there were 3 tests that failed but it was because the test data was returning a list of Strings for getDependencies when it should be a List of Dependencies. So I commented out those 3 tests until they get fixed.
I didn't know how to fix it sorry. I'm still a Java begginner

Show
Michael Mekaail added a comment - Fixed However there were 3 tests that failed but it was because the test data was returning a list of Strings for getDependencies when it should be a List of Dependencies. So I commented out those 3 tests until they get fixed. I didn't know how to fix it sorry. I'm still a Java begginner
Hide
Permalink
Michael Mekaail added a comment -

This is a working patch better than the previous one because it solves the problem and fixes a bug that was on the old one. In this patch I've also fixed the 3 tests that were failing and I've added an extra test to test my code.
Please can you look at this patch and give me feedback or commit it to the Maven Idea Plugin.
Thanks

Show
Michael Mekaail added a comment - This is a working patch better than the previous one because it solves the problem and fixes a bug that was on the old one. In this patch I've also fixed the 3 tests that were failing and I've added an extra test to test my code. Please can you look at this patch and give me feedback or commit it to the Maven Idea Plugin. Thanks
Hide
Permalink
Kohsuke Kawaguchi added a comment -

I tried the patch but I think it is incomplete. When module X depends on Y, this patch only excludes Y's direct dependencies from X, whereas the expected behavior is to exclude all the transitive dependencies of Y from X.

Show
Kohsuke Kawaguchi added a comment - I tried the patch but I think it is incomplete. When module X depends on Y, this patch only excludes Y's direct dependencies from X, whereas the expected behavior is to exclude all the transitive dependencies of Y from X.

People

  • Assignee:
    Unassigned
    Reporter:
    Michael Mekaail
Vote (1)
Watch (2)

Dates

  • Created:
    Updated: