You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sling.apache.org by GitBox <gi...@apache.org> on 2021/11/24 13:43:55 UTC

[GitHub] [sling-org-apache-sling-testing-osgi-mock] kwin commented on pull request #14: SLING-10930 add failing test with multiple deactivate() calls on the

kwin commented on pull request #14:
URL: https://github.com/apache/sling-org-apache-sling-testing-osgi-mock/pull/14#issuecomment-977891648


   I am still not sure about this fix, as this may non-deliberately call a method on a simple service which is coincidentally also called "deactivate()", but this is not new behaviour. Therefore I would still rather not call deactivate at all (instead of calling it unconditionally on all services). A proper fix is as I said a dedicated component registry (decoupled from the service registry).


-- 
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-unsubscribe@sling.apache.org

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