Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-20 Thread Björn Stenberg
Randy MacLeod wrote: > Yes, Roy will fix this. He's on holiday for a few days. Excellent. > I presume he'll add a check to see which arg to use for find > rather than add a dependency on findutils. I suggest skipping the find command altogether. The run-ptest files are defined to always sit in

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-19 Thread Randy MacLeod
On 13-09-19 07:47 AM, Björn Stenberg wrote: Chris Larson wrote: On Wed, Sep 11, 2013 at 2:17 AM, wrote: +for x in `find -L ./ -name run-ptest -type f -perm /u+x,g+x` +do As far as I can tell, busybox find doesn't support -L, so this adds an implicit, undeclared dependency upon find

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-19 Thread Björn Stenberg
Chris Larson wrote: > On Wed, Sep 11, 2013 at 2:17 AM, wrote: > > > +for x in `find -L ./ -name run-ptest -type f -perm /u+x,g+x` > > +do > > > > As far as I can tell, busybox find doesn't support -L, so this adds an > implicit, undeclared dependency upon findutils. Rongquing, are you p

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-17 Thread Björn Stenberg
Chris Larson wrote: > On Wed, Sep 11, 2013 at 2:17 AM, wrote: > > > +for x in `find -L ./ -name run-ptest -type f -perm /u+x,g+x` > > +do > > > > As far as I can tell, busybox find doesn't support -L, so this adds an > implicit, undeclared dependency upon findutils. That is correct. Bus

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-12 Thread Chris Larson
On Wed, Sep 11, 2013 at 2:17 AM, wrote: > +for x in `find -L ./ -name run-ptest -type f -perm /u+x,g+x` > +do > As far as I can tell, busybox find doesn't support -L, so this adds an implicit, undeclared dependency upon findutils. -- Christopher Larson __

[OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-11 Thread rongqing.li
From: Roy Li 1. ptest files may be installed under /usr/lib64/ for 64bit filesystem or under /usr/lib/ for 64bit multilib filesystem, so we should check both directories 2. If a soft link is linking to a directory under the same directory, we only run once. [YOCTO #5125] [YOCTO #5126] Signed-o

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-11 Thread Rongqing Li
On 09/11/2013 04:40 PM, Richard Purdie wrote: On Wed, 2013-09-11 at 15:34 +0800, rongqing...@windriver.com wrote: From: Roy Li 1. ptest files may be installed under /usr/lib64/ for 64bit filesystem or under /usr/lib/ for 64bit multilib filesystem, so we should check both directories 2. If a

Re: [OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-11 Thread Richard Purdie
On Wed, 2013-09-11 at 15:34 +0800, rongqing...@windriver.com wrote: > From: Roy Li > > 1. ptest files may be installed under /usr/lib64/ for 64bit filesystem > or under /usr/lib/ for 64bit multilib filesystem, so we should check both > directories > > 2. If a soft link is linking to a directory

[OE-core] [PATCH 1/1] ptest-runner: trivial fixes and refine

2013-09-11 Thread rongqing.li
From: Roy Li 1. ptest files may be installed under /usr/lib64/ for 64bit filesystem or under /usr/lib/ for 64bit multilib filesystem, so we should check both directories 2. If a soft link is linking to a directory under the same directory, we only run once. [YOCTO #5125] [YOCTO #5126] Signed-o