svn commit: r1609953 - /incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java

[brett]

Author: brett
Date: Sat Jul 12 15:51:52 2014
New Revision: 1609953

URL: http://svn.apache.org/r1609953
Log:
correct issue with multiple configuration files

vendorRequirement is mutated and subsequent attempts to match it will fail

Modified:
    
incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java

Modified: 
incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java
URL: 
http://svn.apache.org/viewvc/incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java?rev=1609953&r1=1609952&r2=1609953&view=diff
==============================================================================
--- 
incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java
 (original)
+++ 
incubator/npanday/trunk/components/dotnet-packaging/src/main/java/npanday/packaging/ConfigFileHandler.java
 Sat Jul 12 15:51:52 2014
@@ -187,8 +187,13 @@ public class ConfigFileHandler
 
             final File targetConfigFile = new File( targetFolder, 
configFilePath );
 
+            // clone vendorRequirement as it gets mutated, which isn't good 
for loop
+            VendorRequirement vr = new 
VendorRequirement(vendorRequirement.getVendor(),
+                    vendorRequirement.getVendorVersion(), 
vendorRequirement.getFrameworkVersion());
+
+            // TODO: should pass in the wohle list and amend the file to 
minimise the number of msbuild invocations
             handleConfigFile(
-                vendorRequirement, configFile, transformationHint, 
targetConfigFile
+                vr, configFile, transformationHint, targetConfigFile
             );
         }
     }