Re: svn commit: r829712 - /geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java

[David Jencks]

I'm confused as to how this is relevant to using references rather  
than copied bundles.  My goals here are to convert the build to use  
mvn urls, and to get the mvn urls to use bundle references.  I don't  
see how this change gets us any closer to this goal.

thanks
david jencks
On Oct 26, 2009, at 12:17 AM, xuhaih...@apache.org wrote:

Author: xuhaihong
Date: Mon Oct 26 07:17:05 2009
New Revision: 829712

URL: http://svn.apache.org/viewvc?rev=829712&view=rev
Log:
GERONIMO-4921 Use the reference file URL for bundle loading on the  
runtime

Modified:
   geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
java/org/apache/geronimo/system/configuration/DependencyManager.java

Modified: geronimo/server/trunk/framework/modules/geronimo-system/ 
src/main/java/org/apache/geronimo/system/configuration/ 
DependencyManager.java
URL: 
http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-system/src/main/java/org/apache/geronimo/system/configuration/DependencyManager.java?rev=829712&r1=829711&r2=829712&view=diff
= 
= 
= 
= 
= 
= 
= 
= 
======================================================================
--- geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
java/org/apache/geronimo/system/configuration/DependencyManager.java  
(original)
+++ geronimo/server/trunk/framework/modules/geronimo-system/src/main/ 
java/org/apache/geronimo/system/configuration/DependencyManager.java  
Mon Oct 26 07:17:05 2009
@@ -118,7 +118,7 @@
    }

    private String locateBundle(Artifact configurationId) throws  
NoSuchConfigException, IOException, InvalidConfigException {
-        if (System.getProperty("geronimo.build.car") == null) {
+        if  
(Boolean.valueOf(System.getProperty("geronimo.build.car"))) {
            return "mvn:" + configurationId.getGroupId() + "/" +  
configurationId.getArtifactId() + "/" + configurationId.getVersion()  
+ ("jar".equals(configurationId.getType())?  "": "/" +  
configurationId.getType());
        }
        for (Repository repo : repositories) {