Author: fgiust
Date: Wed Jul 26 07:50:58 2006
New Revision: 425748

URL: http://svn.apache.org/viewvc?rev=425748&view=rev
Log:
fix phisical path of linked jars and add missing linkedResources tag

Modified:
    
maven/plugins/trunk/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/writers/EclipseProjectWriter.java
    
maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/project-22/project

Modified: 
maven/plugins/trunk/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/writers/EclipseProjectWriter.java
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/writers/EclipseProjectWriter.java?rev=425748&r1=425747&r2=425748&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/writers/EclipseProjectWriter.java
 (original)
+++ 
maven/plugins/trunk/maven-eclipse-plugin/src/main/java/org/apache/maven/plugin/eclipse/writers/EclipseProjectWriter.java
 Wed Jul 26 07:50:58 2006
@@ -33,6 +33,7 @@
 import org.apache.maven.plugin.ide.IdeDependency;
 import org.apache.maven.plugin.ide.IdeUtils;
 import org.codehaus.plexus.util.IOUtil;
+import org.codehaus.plexus.util.StringUtils;
 import org.codehaus.plexus.util.xml.PrettyPrintXMLWriter;
 import org.codehaus.plexus.util.xml.XMLWriter;
 import org.codehaus.plexus.util.xml.Xpp3Dom;
@@ -222,39 +223,47 @@
 
         writer.endElement(); // natures
 
-        if ( !config.getProjectBaseDir().equals( 
config.getEclipseProjectDirectory() ) )
+        boolean addLinks = !config.getProjectBaseDir().equals( 
config.getEclipseProjectDirectory() );
+
+        if ( addLinks || ( config.isPde() && config.getDeps().length > 0 ) )
         {
             writer.startElement( "linkedResources" ); //$NON-NLS-1$
 
-            addFileLink( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config.getProject()
-                .getFile() );
+            if ( addLinks )
+            {
 
-            addSourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
-                .getProject().getCompileSourceRoots() );
-            addResourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
-                .getProject().getBuild().getResources() );
+                addFileLink( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
+                    .getProject().getFile() );
 
-            addSourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
-                .getProject().getTestCompileSourceRoots() );
-            addResourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
-                .getProject().getBuild().getTestResources() );
+                addSourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
+                    .getProject().getCompileSourceRoots() );
+                addResourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
+                    .getProject().getBuild().getResources() );
 
-            writer.endElement(); // linedResources
-        }
+                addSourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
+                    .getProject().getTestCompileSourceRoots() );
+                addResourceLinks( writer, config.getProjectBaseDir(), 
config.getEclipseProjectDirectory(), config
+                    .getProject().getBuild().getTestResources() );
 
-        if ( config.isPde() )
-        {
-            for ( int j = 0; j < config.getDeps().length; j++ )
-            {
-                IdeDependency dep = config.getDeps()[j];
+            }
 
-                if ( dep.isAddedToClasspath() && !dep.isProvided() && 
!dep.isReferencedProject()
-                    && !dep.isTestDependency() )
+            if ( config.isPde() )
+            {
+                for ( int j = 0; j < config.getDeps().length; j++ )
                 {
-                    String name = dep.getFile().getName();
-                    addLink( writer, name, name, LINK_TYPE_FILE );
+                    IdeDependency dep = config.getDeps()[j];
+
+                    if ( dep.isAddedToClasspath() && !dep.isProvided() && 
!dep.isReferencedProject()
+                        && !dep.isTestDependency() )
+                    {
+                        String name = dep.getFile().getName();
+                        addLink( writer, name, StringUtils.replace( 
IdeUtils.getCanonicalPath( dep.getFile() ), "\\",
+                                                                    "/" ), 
LINK_TYPE_FILE );
+                    }
                 }
             }
+
+            writer.endElement(); // linkedResources
         }
 
         writer.endElement(); // projectDescription

Modified: 
maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/project-22/project
URL: 
http://svn.apache.org/viewvc/maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/project-22/project?rev=425748&r1=425747&r2=425748&view=diff
==============================================================================
--- 
maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/project-22/project
 (original)
+++ 
maven/plugins/trunk/maven-eclipse-plugin/src/test/resources/projects/project-22/project
 Wed Jul 26 07:50:58 2006
@@ -20,9 +20,11 @@
     <nature>org.eclipse.jdt.core.javanature</nature>
     <nature>org.eclipse.pde.PluginNature</nature>
   </natures>
-  <link>
-    <name>junit-3.8.1.jar</name>
-    <type>1</type>
-    <location>junit-3.8.1.jar</location>
-  </link>
+  <linkedResources>
+    <link>
+      <name>junit-3.8.1.jar</name>
+      <type>1</type>
+      <location>${M2_REPO}/junit/junit/3.8.1/junit-3.8.1.jar</location>
+    </link>
+  </linkedResources>
 </projectDescription>


Reply via email to