Fix committed.

Thanks,
Krishantha.

On Fri, Oct 12, 2012 at 11:54 PM, Krishantha Samaraweera <
[email protected]> wrote:

> Fixed the issue. I will commit the changes after running product
> integration tests on Windows and Linux.
>
> Thanks,
> Krishantha.
>
> On Thu, Oct 11, 2012 at 10:21 AM, Krishantha Samaraweera <
> [email protected]> wrote:
>
>> Hi Samisa,
>>
>> On Thu, Oct 11, 2012 at 7:30 AM, Samisa Abeysinghe <[email protected]>wrote:
>>
>>> Did you commit?
>>
>>
>> I do not think this is the correct fix. Because the file instrument
>> throws error on windows when running the tests, and the same command works
>> fine when executed though command line. Need to figure out whether any
>> process locking the jar under instrumentation. So it might take some more
>> time to find the correct fix for this.
>>
>> Thanks,
>> Krishantha.
>>
>>
>>>
>>>
>>> On Wed, Oct 10, 2012 at 9:57 PM, Krishantha Samaraweera <
>>> [email protected]> wrote:
>>>
>>>> Command line jar instrumentation works on Windows, this issue caused
>>>> when doing the same thing programmatically.  Emma overwrite the same jar at
>>>> plugins directory after instrumenting its classes.
>>>>
>>>> *$JAVA_HOME/jre/bin/java -cp emma.jar emma instr -m o**verwrite -cp < 
>>>> jarlist.txt*
>>>>
>>>> The issue is solved after using copy/fullcopy option instead of
>>>> overwrite. i have gone though the Emma code and found that when overwrite
>>>> option is true, it simply rename the file as below.
>>>>
>>>> if (! Files.renameFile (m_tempArchiveFile, m_origArchiveFile, true)) //
>>>> overwrite the original archive
>>>>
>>>> Thanks,
>>>> Krishantha.
>>>>
>>>> On Wed, Oct 10, 2012 at 12:49 PM, Krishantha Samaraweera <
>>>> [email protected]> wrote:
>>>>
>>>>> It seems like this is windows specific issue. I will check this.
>>>>>
>>>>> Thanks,
>>>>> Krishantha.
>>>>>
>>>>> On Wed, Oct 10, 2012 at 8:06 AM, Samisa Abeysinghe <[email protected]>wrote:
>>>>>
>>>>>> Running TestSuite
>>>>>> Configuring TestNG with:
>>>>>> org.apache.maven.surefire.testng.conf.TestNGMapConfigurator@54fc9944
>>>>>> [2012-10-10 08:05:21,246]  INFO
>>>>>> {org.wso2.carbon.integration.framework.utils.TestUtil} -  Copying
>>>>>> E:\svn\c4\kernel\distribution\integration\4.0.2\tests\target\security-verifier\SecVerifier.aa
>>>>>> r =>
>>>>>> E:\svn\c4\kernel\distribution\integration\4.0.2\tests\target\carbontmp1349836519090\wso2carbon-core-4.0.2\repository\deployment\server\axis2services\SecVerifier.aar
>>>>>> [EMMA v2.1, build 5320 (stable)]
>>>>>> emmarun: could not rename temporary file
>>>>>> [E:\svn\c4\kernel\distribution\integration\4.0.2\tests\target\carbontmp1349836519090\wso2carbon-core-4.0.2\repository\components\plugins\org.wso2.carb
>>>>>> on.addressing_4.0.0emma1387182540788778040.et] to
>>>>>> [E:\svn\c4\kernel\distribution\integration\4.0.2\tests\target\carbontmp1349836519090\wso2carbon-core-4.0.2\repository\components\plugins\org.
>>>>>> wso2.carbon.addressing_4.0.0.jar]: make sure the original file is not
>>>>>> locked and can be deleted
>>>>>> Tests run: 29, Failures: 1, Errors: 0, Skipped: 28, Time elapsed:
>>>>>> 3.486 sec <<< FAILURE!
>>>>>>
>>>>>>  Thanks,
>>>>>> Samisa...
>>>>>>
>>>>>> Samisa Abeysinghe
>>>>>> VP Engineering
>>>>>> WSO2 Inc.
>>>>>> http://wso2.com
>>>>>> http://wso2.org
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> Dev mailing list
>>>>>> [email protected]
>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>>>
>>>>>>
>>>>>
>>>>  Thanks,
>>> Samisa...
>>>
>>> Samisa Abeysinghe
>>> VP Engineering
>>> WSO2 Inc.
>>> http://wso2.com
>>> http://wso2.org
>>>
>>>
>>>
>>
>
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to