[PATCH 1/3] test_fs: Allow running unprivileged

Simon Glass sjg at chromium.org
Sun Aug 18 17:47:01 CEST 2024


Hi Richard,

On Sun, 18 Aug 2024 at 16:37, Richard Weinberger <richard at sigma-star.at> 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 to worry. :)
>
> > specifically what the code is for. Also, once you get all this in, we
> > can perhaps remove the fallbacks. I still get errors from qemu-img on
>
> I hope we can remove all these fall backs.
> IMHO, unit tests have to behave always the same way.

Indeed.

>
> > Ubuntu unless I chmod a+r /boot/* but presumably you don't use that
> > tool.
>
> On recent Ubuntu releases this should be resolved:
> https://bugs.launchpad.net/ubuntu/+source/libguestfs/+bug/1673431
>
> > For mkinage, it would be better to use the version that is built in
> > CI, for sandbox. That is how the tools tests work.
>
> True that!
> Off the head, I don't know how to achieve this, but my knowledge of
> the Azure pipeline setup is limited.

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 as
easily add tools/

          export PATH=${UBOOT_TRAVIS_BUILD_DIR}/scripts/dtc:${PATH}

Regards,
Simon


More information about the U-Boot mailing list