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)