PengZheng opened a new pull request, #554:
URL: https://github.com/apache/celix/pull/554

   The behavior of the `uninstall()` method of Bundle interface is specified as 
follows: 
   
   > To whatever extent possible, the Framework must remove any resources 
related to the bundle. This method must always uninstall the bundle from the 
persistent storage of the Framework.
   
   Clearly, the current implementation, which is introduced by #476, does not 
touch persistent storage.
   As seen in `TEST_F(CxxBundleArchiveTestSuite, BundleArchiveReusedTest)`, 
after uninstall, the bundle archive is intact.
   The previous implementation does set the bundle's persistent state to 
UNINSTALLED, but does nothing more than that.
   Therefore, both are incorrect.
   
   This PR tries to implement the correct behavior without affecting 
`framework_shutdown`. 
   If this behavior change is acceptable, I'll add more tests.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to