Re: [OE-core] [PATCH 1/8] lib/oe/path: implement is_path_under()

2018-02-25 Thread Paul Eggleton
Hi Ola, On Friday, 23 February 2018 8:42:34 PM NZDT Ola x Nilsson wrote: > > +def is_path_under(path1, possible_parent): > > Changing the signature to is_path_under(possible_parent, *paths) will make > the list management more natural. Right, it does simplify things a little. In fact given the

Re: [OE-core] [PATCH 1/8] lib/oe/path: implement is_path_under()

2018-02-22 Thread Ola x Nilsson
> From: openembedded-core-boun...@lists.openembedded.org > [mailto:openembedded-core-boun...@lists.openembedded.org] On Behalf > Of Paul Eggleton > Sent: den 23 februari 2018 04:02 > To: openembedded-core@lists.openembedded.org > > Signed-off-by: Paul Eggleton >

[OE-core] [PATCH 1/8] lib/oe/path: implement is_path_under()

2018-02-22 Thread Paul Eggleton
In a few places we have checks to see path A is under path B, by adding / to the end of the path B and then seeing if path A starts with the suffixed path B. Unfortunately there are two potential flaws: (1) path A needs to be suffixed with / as well or the directory itself won't match (semantics