[jira] [Comment Edited] (SLING-5337) Extract Startup API from Launchpad API

["Carsten Ziegeler (JIRA)" <ji...@apache.org>]

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

Carsten Ziegeler edited comment on SLING-5337 at 12/4/15 8:13 AM:
------------------------------------------------------------------

It seems, in the settings module, the startup handler is used to do some run mode magic which I think we can remove


was (Author: cziegeler):
It seems the startup handler is doing some run mode magic which I dont understand. Maybe we really don't need it

> Extract Startup API from Launchpad API
> --------------------------------------
>
>                 Key: SLING-5337
>                 URL: https://issues.apache.org/jira/browse/SLING-5337
>             Project: Sling
>          Issue Type: Improvement
>          Components: Launchpad
>    Affects Versions: Launchpad API 1.2.0
>            Reporter: Oliver Lietz
>            Assignee: Oliver Lietz
>             Fix For: Launchpad API 1.3.0, Startup API 1.0.0
>
>
> {{StartupHandler}}, {{StartupListener}}, {{StartupMode}} and {{StartupService}} in {{org.apache.sling.launchpad.api}} should be moved to {{org.apache.sling.startup.api}} (keeping extending but deprecated classes there for backward compatibility) to decouple Startup from Launchpad.
> A Startup implementation in Sling needs to be provided by every launcher (Launchpad, Crankstart, Karaf)\[1\] whereas {{LaunchpadContentProvider}} is only useful in contexts of Sling's own launcher Launchpad for installation purpose.
> [~cziegeler], [~bdelacretaz]: anything from your POVs to consider?
> \[1\] We discussed to make Startup optional in {{Settings}} but that won't work as we have modules which rely already on it (e.g. {{discovery}}).



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