On Thu, Jul 25, 2019 at 04:33:21PM +0300, Sam Protsenko wrote:
> Hi Simon,
> 
> On Sat, Jul 6, 2019 at 7:53 PM Simon Glass <s...@chromium.org> wrote:
> >
> > Hi,
> >
> > On Tue, 2 Jul 2019 at 12:18, Sam Protsenko <semen.protse...@linaro.org> 
> > wrote:
> > >
> > > Signed-off-by: Sam Protsenko <semen.protse...@linaro.org>
> > > ---
> > >  test/py/tests/{ => test_android}/test_avb.py | 0
> > >  1 file changed, 0 insertions(+), 0 deletions(-)
> > >  rename test/py/tests/{ => test_android}/test_avb.py (100%)
> > >
> > > diff --git a/test/py/tests/test_avb.py 
> > > b/test/py/tests/test_android/test_avb.py
> > > similarity index 100%
> > > rename from test/py/tests/test_avb.py
> > > rename to test/py/tests/test_android/test_avb.py
> > > --
> >
> > I'd prefer to keep tsts in one place. What is the goal of this patch?
> >
> 
> Similar patch was merged recently (adding test_android/ dir):
> 
>     https://patchwork.ozlabs.org/patch/1128013/
> 
> So we need either:
>   1. move that test out of test_android/ dir, and drop this one
>   2. or apply this patch
> 
> What would you prefer? (I've explained my perspective on that matter
> earlier in this thread). We should do either (1) or (2) to keep the
> consistency.

Note that we've also had vboot as a sub-directory since it was
introduced and test_fs is also a sub-directory.  My two cents is that
sub-directories help with organization as the number of tests increase.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to