You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by mc...@apache.org on 2008/01/29 12:51:14 UTC

svn commit: r616273 - /felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java

Author: mcculls
Date: Tue Jan 29 03:51:13 2008
New Revision: 616273

URL: http://svn.apache.org/viewvc?rev=616273&view=rev
Log:
FELIX-472: extract fields

Modified:
    felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java

Modified: felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java
URL: http://svn.apache.org/viewvc/felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java?rev=616273&r1=616272&r2=616273&view=diff
==============================================================================
--- felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java (original)
+++ felix/trunk/bundleplugin/src/main/java/org/apache/felix/bundleplugin/DependencyEmbedder.java Tue Jan 29 03:51:13 2008
@@ -47,6 +47,10 @@
     public static final String EMBED_STRIP_VERSION = "Embed-StripVersion";
     public static final String EMBED_TRANSITIVE = "Embed-Transitive";
 
+    private String m_embedDirectory;
+    private String m_embedStripGroup;
+    private String m_embedStripVersion;
+
     /**
      * Dependency artifacts.
      */
@@ -80,6 +84,10 @@
         String embedDependencyHeader = properties.getProperty( EMBED_DEPENDENCY );
         if ( null != embedDependencyHeader && embedDependencyHeader.length() > 0 )
         {
+            m_embedDirectory = properties.getProperty( EMBED_DIRECTORY );
+            m_embedStripGroup = properties.getProperty( EMBED_STRIP_GROUP, "true" );
+            m_embedStripVersion = properties.getProperty( EMBED_STRIP_VERSION );
+
             Map embedInstructions = OSGiHeader.parseHeader( embedDependencyHeader );
             processEmbedInstructions( embedInstructions );
 
@@ -281,7 +289,6 @@
     private void embedDependency( Properties properties, Artifact dependency )
     {
         File sourceFile = dependency.getFile();
-
         if ( null != sourceFile && sourceFile.exists() )
         {
             String bundleClassPath = properties.getProperty( Analyzer.BUNDLE_CLASSPATH );
@@ -305,22 +312,19 @@
                 includeResource += ",";
             }
 
-            String embedDirectory = properties.getProperty( EMBED_DIRECTORY );
-            String embedStripGroup = properties.getProperty( EMBED_STRIP_GROUP, "true" );
-            String embedStripVersion = properties.getProperty( EMBED_STRIP_VERSION );
-
+            String embedDirectory = m_embedDirectory;
             if ( "".equals( embedDirectory ) || ".".equals( embedDirectory ) )
             {
                 embedDirectory = null;
             }
 
-            if ( false == Boolean.valueOf( embedStripGroup ).booleanValue() )
+            if ( false == Boolean.valueOf( m_embedStripGroup ).booleanValue() )
             {
                 embedDirectory = new File( embedDirectory, dependency.getGroupId() ).getPath();
             }
 
             File targetFile;
-            if ( Boolean.valueOf( embedStripVersion ).booleanValue() )
+            if ( Boolean.valueOf( m_embedStripVersion ).booleanValue() )
             {
                 String extension = dependency.getArtifactHandler().getExtension();
                 if ( extension != null )
@@ -357,7 +361,6 @@
     private void inlineDependency( Properties properties, Artifact dependency )
     {
         File sourceFile = dependency.getFile();
-
         if ( null != sourceFile && sourceFile.exists() )
         {
             String includeResource = properties.getProperty( Analyzer.INCLUDE_RESOURCE );