[ 
https://issues.apache.org/jira/browse/FELIX-5723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16215485#comment-16215485
 ] 

Brandan Jeter commented on FELIX-5723:
--------------------------------------

As an interesting side note, we had fixed this awhile back with this solution 
by implementing our own ConfigurationListener: 
https://github.com/codice/ddf/commit/e2bc11abdc00ae21089dcfb49d67e26d1299c0dd#diff-74d5172cba120da06c8629e456e88eb8R50

We didn't contribute it back here because we weren't sure if that would be an 
accepted solution to the problem. Next time, we'll offer the solution anyways 
and risk getting shot down! Thanks again.

> ConfigInstaller is not deleting files in the etc directory
> ----------------------------------------------------------
>
>                 Key: FELIX-5723
>                 URL: https://issues.apache.org/jira/browse/FELIX-5723
>             Project: Felix
>          Issue Type: Bug
>    Affects Versions: fileinstall-3.6.0, fileinstall-3.6.2
>            Reporter: Brandan Jeter
>            Assignee: Guillaume Nodet
>             Fix For: fileinstall-3.6.4
>
>
> The ConfigInstaller takes the pid from the ConfigurationEvent and tries to 
> get the configuration from ConfigAdmin. This will not return the config 
> because the config has been deleted. See 
> https://github.com/apache/felix/blob/trunk/fileinstall/src/main/java/org/apache/felix/fileinstall/internal/ConfigInstaller.java#L182
> The ConfigurationEvent must provide the felix.fileinstall property, so that 
> the ConfigInstaller can use that to delete the corresponding file in etc.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to