The constants should really be defined in the BVP class, this is the service declaring them
I don't understand what you mean by "changes to existing code" - this is new, isn't it? Carsten 2013/9/6 Bertrand Delacretaz <bdelacre...@apache.org> > On Thu, Sep 5, 2013 at 1:28 PM, Bertrand Delacretaz > <bdelacre...@apache.org> wrote: > > ...Maybe > > > > addContextBindings(Bindings b, ScriptEngineFactory f, String context) > > ... > > FYI I didn't do it like that in the end, as that would have caused > more changes than I like to existing code. The new service API is at > > https://svn.apache.org/repos/asf/sling/trunk/bundles/scripting/api/src/main/java/org/apache/sling/scripting/api/BindingsValuesProvidersByContext.java > > Apart from that this is now done, see SLING-3038 for details, > > -Bertrand > -- Carsten Ziegeler cziege...@apache.org