Maven 1.x Eclipse Plugin
  1. Maven 1.x Eclipse Plugin
  2. MPECLIPSE-105

Ability to add projects of local group to build path

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 1.9
    • Fix Version/s: None
    • Labels:
      None
    • Number of attachments :
      1

      Description

      We are using eclipse and maven for our internal project.

      Our project is splitted up into multiple semi-independent projects (modules).

      So, using repository and maven eclipse plugin we were able to work. However, there is still one important feature of Eclipse that is not available anymore after we swtiched to maven: sources of different projects got decoupled and you can not search, for instance, for references to some class or method.

      This could be only fixed by adding referencing projects' source code from other projects.

      So, I decided to write a patch for this.

      The point is: during generation of .classpath plugin behaves as usually, with an exception of cases, where groupId is the same as maven.eclipse.localgroupid. For this case there will be an additional classpathentry with reference to project.

      Hope, my patch explains more then this text (it is a little bit mixed .

      1. PATCH
        0.5 kB
        Rodion Zhitomirsky

        Activity

        Hide
        Rodion Zhitomirsky added a comment -

        Here patch in plain text:

        — plugin.jelly-1.9 2005-10-21 20:13:02.000000000 +0200
        +++ plugin.jelly-with-eclipse-project-feature 2005-10-21 20:06:59.000000000 +0200
        @@ -48,6 +48,10 @@
        <classpathentry kind="var" path="MAVEN_REPO/$

        {relativePath}

        " />
        </j:otherwise>
        </j:choose>
        + <j:set var="localGroupId" value="$

        {maven.eclipse.localgroupid}

        "/>
        + <j:if test='$

        {groupId == localGroupId}

        '>
        + <classpathentry combineaccessrules="false" kind="src" path="/$

        {artifactId}

        "/>
        + </j:if>
        </define:tag>
        </define:taglib>

        Show
        Rodion Zhitomirsky added a comment - Here patch in plain text: — plugin.jelly-1.9 2005-10-21 20:13:02.000000000 +0200 +++ plugin.jelly-with-eclipse-project-feature 2005-10-21 20:06:59.000000000 +0200 @@ -48,6 +48,10 @@ <classpathentry kind="var" path="MAVEN_REPO/$ {relativePath} " /> </j:otherwise> </j:choose> + <j:set var="localGroupId" value="$ {maven.eclipse.localgroupid} "/> + <j:if test='$ {groupId == localGroupId} '> + <classpathentry combineaccessrules="false" kind="src" path="/$ {artifactId} "/> + </j:if> </define:tag> </define:taglib>
        Hide
        fabrizio giustina added a comment -

        why the eclipse.dependency in project.xml can't be used in this situation?
        Just add:
        <properties>
        <eclipse.dependency>true</eclipse.dependency>
        </properties>
        to the dependencies you want to link as project references in eclipse and you should achieve the same result.
        Is this correct (and so we can close the issue) or I'm missing something?

        Show
        fabrizio giustina added a comment - why the eclipse.dependency in project.xml can't be used in this situation? Just add: <properties> <eclipse.dependency>true</eclipse.dependency> </properties> to the dependencies you want to link as project references in eclipse and you should achieve the same result. Is this correct (and so we can close the issue) or I'm missing something?
        Hide
        Michael Osipov added a comment -

        Please refer to https://cwiki.apache.org/confluence/display/MAVEN/The+Great+JIRA+Cleanup+of+2014 if you're wondering why this issue was closed out.

        Show
        Michael Osipov added a comment - Please refer to https://cwiki.apache.org/confluence/display/MAVEN/The+Great+JIRA+Cleanup+of+2014 if you're wondering why this issue was closed out.

          People

          • Assignee:
            Unassigned
            Reporter:
            Rodion Zhitomirsky
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: