On Wed, 2016-01-27 at 15:47 +0530, Chetan Mehrotra wrote:
> FWIW Oak has a PathUtils [1]. Looking at code it has a minor
> dependency on Guava which can be fixed. So you can inline this class
> in your bundle if required

Thanks for the info.

Right now I feel more comfortable with using a minimal PathBuilder
class, but for the future we can inline the PathUtils class in the
sling bundle if we need more complex handling.

Robert

> 
> Chetan Mehrotra
> [1] https://github.com/apache/jackrabbit-oak/blob/trunk/oak-commons/s
> rc/main/java/org/apache/jackrabbit/oak/commons/PathUtils.java
> 
> 
> On Tue, Jan 26, 2016 at 9:36 PM, Robert Munteanu <romb...@apache.org>
> wrote:
> > On Tue, 2016-01-26 at 16:15 +0100, Bertrand Delacretaz wrote:
> > > Hi Robert,
> > > 
> > > > ...
> > > >   https://issues.apache.org/jira/browse/SLING-5455
> > > >   https://svn.apache.org/r1726814 ...
> > > 
> > > I have second thoughts about this, sorry ;-)
> > > 
> > > Touching our "sacred" API bundle just for this doesn't feel
> > > right...how about creating a new bundles/extensions/sling-commons 
> > > or
> > > sling-util bundle? It will be almost empty for now but that's a
> > > good
> > > place to add such minor utilities later, without touching
> > > "important"
> > > bundles.
> > 
> > Moving out of API is fine for me.
> > 
> > Keeping in mind that this will be used by the resourceresolver
> > bundle,
> > is bundles/extensions the right place ? ( not a rhetorical
> > question, I
> > really don't know ).
> > 
> > As for the bundle name, I'm not a very big fan of commons/util
> > since
> > they tend to become onfocused, but I don't have a better idea right
> > now.
> > 
> > I've reopened SLING-5455 to track the creation of the new bundle.
> > 
> > Robert
> > 
> > > 
> > > WDYT?
> > > 
> > > -Bertrand
> > 

Reply via email to