[
https://issues.apache.org/jira/browse/SLING-8484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16861988#comment-16861988
]
Karl Pauls commented on SLING-8484:
-----------------------------------
The important thing for me is that it should be optional as I'm not a big fan
of the idea in general. On the one hand, something not resolving might still be
what I want from a feature and on the other hand, just because something
resolves it doesn't has to mean it is working (i.e., i fear this is a slippery
slope for introducing a poor man's system ready). Furthermore, I think this
will come with a potential performance penalty and changed behaviour as you
have to resolve everything ahead of time.
In other words, I'd prefer to not have such a feature at all and rather have
good tooling that quickly can tell you if something will resolve (IIRC, there
was some work on having the analyser work as a launcher - that would be
something where you can quickly check if things look good and then start. Might
need some work to make it good enough but I'd prefer it).
That said, I'm not feeling too strongly about it - hence, if you want to add it
I would prefer it to not be the default as I think it will be confusing but
ultimately, I'll yield if the consensus is to make it the default as long as it
can be disabled.
> Fail the launcher if not all bundles could be launched
> ------------------------------------------------------
>
> Key: SLING-8484
> URL: https://issues.apache.org/jira/browse/SLING-8484
> Project: Sling
> Issue Type: Improvement
> Affects Versions: Feature Model Launcher 1.0.2
> Reporter: David Bosschaert
> Assignee: David Bosschaert
> Priority: Major
> Fix For: Feature Model Launcher 1.0.4
>
>
> If the launcher is not able to start all bundles successfully, the launcher
> should fail with a non-zero exit code.
> This functionality should not be the default. It needs to be enabled with a
> flag, for example -foe (Fail on Error)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
