Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-09-17 Thread Tom Rini
On Mon, Aug 19, 2024 at 11:11:36AM -0600, Tom Rini wrote: > On Sun, Aug 18, 2024 at 10:06:05PM +0200, Richard Weinberger wrote: > > Simon, > > > > Am Sonntag, 18. August 2024, 17:47:01 CEST schrieb Simon Glass: > > > In that file, this is the line that builds the tools: > > > > > > ./to

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-19 Thread Tom Rini
On Sun, Aug 18, 2024 at 10:06:05PM +0200, Richard Weinberger wrote: > Simon, > > Am Sonntag, 18. August 2024, 17:47:01 CEST schrieb Simon Glass: > > In that file, this is the line that builds the tools: > > > > ./tools/buildman/buildman -T0 -o ${UBOOT_TRAVIS_BUILD_DIR} > > -w --board to

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-18 Thread Richard Weinberger
Simon, Am Sonntag, 18. August 2024, 17:47:01 CEST schrieb Simon Glass: > In that file, this is the line that builds the tools: > > ./tools/buildman/buildman -T0 -o ${UBOOT_TRAVIS_BUILD_DIR} > -w --board tools-only > > and this adds the built scripts/dtc to the path, but you could just

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-18 Thread Simon Glass
Hi Richard, On Sun, 18 Aug 2024 at 16:37, Richard Weinberger wrote: > > Simon, > > Am Sonntag, 18. August 2024, 17:25:37 CEST schrieb Simon Glass: > > > So, can we please have mkimage inside the Docker image? > > > > Sorry about all your trouble. Perhaps we should add a comment about > > No need

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-18 Thread Richard Weinberger
Simon, Am Sonntag, 18. August 2024, 17:25:37 CEST schrieb Simon Glass: > > So, can we please have mkimage inside the Docker image? > > Sorry about all your trouble. Perhaps we should add a comment about No need to worry. :) > specifically what the code is for. Also, once you get all this in, we

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-18 Thread Simon Glass
Hi Richard, On Sun, Aug 18, 2024, 03:11 Richard Weinberger wrote: > > Am Freitag, 16. August 2024, 00:14:18 CEST schrieb Tom Rini: > > This brings two new pylint errors: > > test/py/tests/fs_helper.py:47:12: E0704: The raise statement is not inside > > an except clause (misplaced-bare-raise) > >

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-18 Thread Richard Weinberger
Am Freitag, 16. August 2024, 00:14:18 CEST schrieb Tom Rini: > This brings two new pylint errors: > test/py/tests/fs_helper.py:47:12: E0704: The raise statement is not inside an > except clause (misplaced-bare-raise) This raise was on purpose, I wanted the test to fail when an unsupported filesy

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-15 Thread Tom Rini
On Fri, Aug 02, 2024 at 11:33:20AM +0200, Richard Weinberger wrote: > There is no need to mount the filesystem on the host side. > All filesystem tools offer some way to fill the fs without mounting. > > So, create the content on the host side, create and fill the fs > without mounting. > No more

Re: [PATCH 1/3] test_fs: Allow running unprivileged

2024-08-06 Thread Mattijs Korpershoek
Hi Richard, Thank you for the patch. On ven., août 02, 2024 at 11:33, Richard Weinberger wrote: > There is no need to mount the filesystem on the host side. > All filesystem tools offer some way to fill the fs without mounting. > > So, create the content on the host side, create and fill the fs