"System" dependencies path non correctly added to eclipse buildpath

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    SVN snapshot a few days old

Description

Eclipse plugin handles <systemPath> as if it was a path into the repository, transforming a dependency like this one:

<dependency>
...
<scope>system</scope>
<systemPath>${basebir}/lib/myjdbcdriver.jar</systemPath>
</dependency>

into

M2_REPO/<pom.xml location dir>/lib/myjdbcdriver.jar

Issue Links

depends upon

Bug - A problem which impairs or prevents the functions of the product. MNG-1379 Wrong path for artifacts with system scope

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
fabrizio giustina added a comment -

testcase added to svn, can be reproduced in svn trunk.

Show
fabrizio giustina added a comment - testcase added to svn, can be reproduced in svn trunk.
Hide
Permalink
fabrizio giustina added a comment -

fixed in svn

Show
fabrizio giustina added a comment - fixed in svn
Hide
Permalink
Brill Pappin added a comment -

So what version do I use to get the fix?
it's not being downloaded by m2...

Show
Brill Pappin added a comment - So what version do I use to get the fix? it's not being downloaded by m2...
Hide
Permalink
Giorgio Gallo added a comment -

Sorry to bother Fabrizio..

I updated my m2 to the last svn snapshot this morning, but the eclipse plugin still can't handle system dependencies...

Show
Giorgio Gallo added a comment - Sorry to bother Fabrizio.. I updated my m2 to the last svn snapshot this morning, but the eclipse plugin still can't handle system dependencies...
Hide
Permalink
fabrizio giustina added a comment -

> I updated my m2 to the last svn snapshot this morning, but the eclipse plugin still can't handle system dependencies...
sounds strange, one of the tests run before the plugin is installed just tests for system dependencies. Did you upgrade m2 or the eclipse plugin? You need to install the eclipse plugin from the version in svn to get this fixed (the bug in maven core is still open, a workaround has been added directly to the plugin)

Show
fabrizio giustina added a comment - > I updated my m2 to the last svn snapshot this morning, but the eclipse plugin still can't handle system dependencies... sounds strange, one of the tests run before the plugin is installed just tests for system dependencies. Did you upgrade m2 or the eclipse plugin? You need to install the eclipse plugin from the version in svn to get this fixed (the bug in maven core is still open, a workaround has been added directly to the plugin)
Hide
Permalink
Giorgio Gallo added a comment -

I updated my svn snapshot and run m2-bootstrap-all, I also deleted the eclipse plugin from my local repository and re-run m2-bootstrap-all...

You saying I should have mvn install from the plugin project dir? Doesn't it all get built and deployed to my local repo running m2-bootstrap-all?

TIA

Show
Giorgio Gallo added a comment - I updated my svn snapshot and run m2-bootstrap-all, I also deleted the eclipse plugin from my local repository and re-run m2-bootstrap-all... You saying I should have mvn install from the plugin project dir? Doesn't it all get built and deployed to my local repo running m2-bootstrap-all? TIA
Hide
Permalink
fabrizio giustina added a comment -

this is definitively fixed in the current svn version of the eclipse plugin. Just try to update the maven-eclipse-plugin dir and install it, it will work.

Show
fabrizio giustina added a comment - this is definitively fixed in the current svn version of the eclipse plugin. Just try to update the maven-eclipse-plugin dir and install it, it will work.
Hide
Permalink
Brill Pappin added a comment -

What do you mean update the maven-eclipse-plugin?
What should end users do to get this fix in place?

Show
Brill Pappin added a comment - What do you mean update the maven-eclipse-plugin? What should end users do to get this fix in place?
Hide
Permalink
Giorgio Gallo added a comment -

Hi Brill, I just realized this plugin's surces are not checked out when you checkout maven from svn...
I guess it's the same problem you are having, so...
You should:

svn checkout https://svn.apache.org/repos/asf/maven/plugins/trunk/maven-eclipse-plugin/ maven-eclipse-plugin
cd maven-eclipse-plugin
mvn install

Hope this helps!

Show
Giorgio Gallo added a comment - Hi Brill, I just realized this plugin's surces are not checked out when you checkout maven from svn... I guess it's the same problem you are having, so... You should: svn checkout https://svn.apache.org/repos/asf/maven/plugins/trunk/maven-eclipse-plugin/ maven-eclipse-plugin cd maven-eclipse-plugin mvn install Hope this helps!

People

  • Assignee:
    fabrizio giustina
    Reporter:
    Giorgio Gallo
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: