You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Pierre De Rop (JIRA)" <ji...@apache.org> on 2016/09/07 14:14:21 UTC

[jira] [Commented] (FELIX-5238) ConfigurationDependency should provide the setRequired() method

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

Pierre De Rop commented on FELIX-5238:
--------------------------------------

is there somebody disagreeing with this improvement ?

if not, then I would like to commit the proposed patch for the next dm-r9 version. The patch is reasonably simple, and I think it's simplifying the way to deal with optional configurations ...


> ConfigurationDependency should provide the setRequired() method
> ---------------------------------------------------------------
>
>                 Key: FELIX-5238
>                 URL: https://issues.apache.org/jira/browse/FELIX-5238
>             Project: Felix
>          Issue Type: Improvement
>          Components: Dependency Manager
>    Affects Versions: dependencymanager-4.3.0
>            Reporter: Guillaume Nodet
>            Assignee: Pierre De Rop
>         Attachments: FELIX-5238.patch, OptionalConfigurationDependencyTest.java
>
>
> Even though the default required for configuration is {{true}}, there's no reason why it could not be {{false}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)