Author: rfeng
Date: Wed Oct 15 10:58:08 2008
New Revision: 704971

URL: http://svn.apache.org/viewvc?rev=704971&view=rev
Log:
Adjust the phase to make sure pde-classpath comes after assemble-bundle

Removed:
    
tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/tools/sca/tuscany/bundle/plugin/
Modified:
    
tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/sca/tools/bundle/plugin/ThirdPartyBundleClasspathGeneratorMojo.java

Modified: 
tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/sca/tools/bundle/plugin/ThirdPartyBundleClasspathGeneratorMojo.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/sca/tools/bundle/plugin/ThirdPartyBundleClasspathGeneratorMojo.java?rev=704971&r1=704970&r2=704971&view=diff
==============================================================================
--- 
tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/sca/tools/bundle/plugin/ThirdPartyBundleClasspathGeneratorMojo.java
 (original)
+++ 
tuscany/branches/sca-equinox/tools/maven/maven-bundle-plugin/src/main/java/org/apache/tuscany/sca/tools/bundle/plugin/ThirdPartyBundleClasspathGeneratorMojo.java
 Wed Oct 15 10:58:08 2008
@@ -18,37 +18,25 @@
  */
 package org.apache.tuscany.sca.tools.bundle.plugin;
 
-import static org.apache.tuscany.sca.tools.bundle.plugin.BundleUtil.write;
-
 import java.io.BufferedReader;
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.FileOutputStream;
-import java.io.IOException;
 import java.io.InputStreamReader;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
 import java.io.StringWriter;
 import java.io.Writer;
-import java.util.HashSet;
-import java.util.Set;
-import java.util.jar.Manifest;
-
-import org.apache.maven.artifact.Artifact;
-import org.apache.maven.artifact.resolver.ArtifactNotFoundException;
-import org.apache.maven.artifact.resolver.ArtifactResolutionException;
-import org.apache.maven.artifact.versioning.VersionRange;
+
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.logging.Log;
 import org.apache.maven.project.MavenProject;
-import org.apache.maven.shared.dependency.tree.DependencyTree;
-import org.apache.maven.shared.dependency.tree.DependencyTreeBuilderException;
 
 /**
  * @version $Rev$ $Date$
  * @goal generate-pde-classpath
- * @phase process-resources
+ * @phase process-classes
  * @requiresDependencyResolution test
  * @description Adjust third party bundle classpath
  */
@@ -86,19 +74,20 @@
             BufferedReader reader = new BufferedReader(new 
InputStreamReader(new FileInputStream(classpath)));
             StringWriter buffer = new StringWriter();
             PrintWriter printer = new PrintWriter(buffer);
-            boolean generatedLib = false;
             for (;;) {
                 String line = reader.readLine();
                 if (line == null) {
                     break;
                 }
-                if (line.contains("kind=\"var\"")) {
-                    if (!generatedLib) {
-                        generateLibClasspathEntries(printer);
-                        generatedLib = true;
-                    }
+                if(line.contains("kind=\"lib\"") || 
line.contains("kind=\"var\"")) {
+                    // Skip kind="lib", kind="var"
                     continue;
                 }
+                if (line.contains("</classpath>")) {
+                    log.info("Adding lib classpath entries ...");
+                    // Generate the lib classpath entries before the 
</classpath>
+                    generateLibClasspathEntries(printer);
+                }
                 printer.println(line);
             }
             reader.close();


Reply via email to