Re: [PATCH v1] test: Include /sbin to the PATH when creating ext4 disk image

2021-02-05 Thread Simon Glass
Hi Andy, On Fri, 5 Feb 2021 at 12:40, Andy Shevchenko wrote: > > On Fri, Feb 05, 2021 at 07:29:40PM +0200, Andy Shevchenko wrote: > > On Thu, Feb 04, 2021 at 08:17:23PM -0700, Simon Glass wrote: > > > On Wed, 3 Feb 2021 at 08:32, Andy Shevchenko > > > wrote: > > > > ... > > > > > > if os.pa

Re: [PATCH v1] test: Include /sbin to the PATH when creating ext4 disk image

2021-02-05 Thread Andy Shevchenko
On Fri, Feb 05, 2021 at 07:29:40PM +0200, Andy Shevchenko wrote: > On Thu, Feb 04, 2021 at 08:17:23PM -0700, Simon Glass wrote: > > On Wed, 3 Feb 2021 at 08:32, Andy Shevchenko > > wrote: > > ... > > > > if os.path.exists(persistent): > > > c.log.action('Disk image file ' + persist

Re: [PATCH v1] test: Include /sbin to the PATH when creating ext4 disk image

2021-02-05 Thread Andy Shevchenko
On Thu, Feb 04, 2021 at 08:17:23PM -0700, Simon Glass wrote: > On Wed, 3 Feb 2021 at 08:32, Andy Shevchenko > wrote: ... > > if os.path.exists(persistent): > > c.log.action('Disk image file ' + persistent + ' already exists') > > else: > > +root_path = os.path.abspath(

Re: [PATCH v1] test: Include /sbin to the PATH when creating ext4 disk image

2021-02-04 Thread Simon Glass
Hi Andy, On Wed, 3 Feb 2021 at 08:32, Andy Shevchenko wrote: > > On some distributions the mkfs.ext4 is under /sbin and /sbin is not set > for mere users. Include /sbin to the PATH when creating ext4 disk image, > so that users won't get a scary traceback from Python. > > Signed-off-by: Andy Shev

[PATCH v1] test: Include /sbin to the PATH when creating ext4 disk image

2021-02-03 Thread Andy Shevchenko
On some distributions the mkfs.ext4 is under /sbin and /sbin is not set for mere users. Include /sbin to the PATH when creating ext4 disk image, so that users won't get a scary traceback from Python. Signed-off-by: Andy Shevchenko --- test/py/tests/test_env.py | 3 +++ 1 file changed, 3 insertio