Oops. That certainly should not have been there. Thanks for spotting that :)

On 23 Jun 2011, at 15:24, Jeremy Hughes wrote:

> On 16 May 2011 19:47,  <[email protected]> wrote:
>> Author: mahrwald
>> Date: Mon May 16 18:47:53 2011
>> New Revision: 1103835
>> 
>> URL: http://svn.apache.org/viewvc?rev=1103835&view=rev
>> Log:
>> ARIES-652: Add zips nested in file IDirectories + fix a couple of minor bugs
>> 
> [snip]
>> ==============================================================================
>> --- 
>> aries/trunk/application/application-deployment-management/src/main/java/org/apache/aries/application/deployment/management/impl/DeploymentManifestManagerImpl.java
>>  (original)
>> +++ 
>> aries/trunk/application/application-deployment-management/src/main/java/org/apache/aries/application/deployment/management/impl/DeploymentManifestManagerImpl.java
>>  Mon May 16 18:47:53 2011
>> @@ -180,7 +180,8 @@ public class DeploymentManifestManagerIm
>>    * @return
>>    * @throws ResolverException
>>    */
>> -    public DeployedBundles generateDeployedBundles(ApplicationMetadata 
>> appMetadata,
>> +    @Override
>> +       public DeployedBundles generateDeployedBundles(ApplicationMetadata 
>> appMetadata,
>>             Collection<ModelledResource> provideByValueBundles, 
>> Collection<Content> otherBundles)
>>             throws ResolverException {
>> 
>> @@ -662,6 +663,9 @@ public class DeploymentManifestManagerIm
>>       OutputStream out = new FileOutputStream(temp);
>>       IOUtils.copy(in, out);
>>       IOUtils.close(out);
>> +
>> +      System.out.println("XXX: "+temp.getAbsolutePath()+" - 
>> "+temp.exists());
>> +
> 
> Is this sysout still needed?
> 
>>       result.add(modelledResourceManager.getModelledResource(null, 
>> FileSystem.getFSRoot(temp)));
>>       // delete the temp file
>>       temp.delete();
>> 
> [snip]
> 
> Cheers,
> Jeremy

Reply via email to