From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 10 Jun 2020 12:13:27 -0400 Subject: [PATCH 1/1] Dockerfile: add libguestfs-tools package In-Reply-To: <20200608012520.GA3769@laputa> References: <20200530221658.50842-1-xypron.glpk@gmx.de> <20200605151915.GI32287@bill-the-cat> <20200608012520.GA3769@laputa> Message-ID: <20200610161327.GD24893@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Jun 08, 2020 at 10:25:20AM +0900, AKASHI Takahiro wrote: > On Fri, Jun 05, 2020 at 11:19:15AM -0400, Tom Rini wrote: > > On Sun, May 31, 2020 at 12:16:58AM +0200, Heinrich Schuchardt wrote: > > > > > In some of our Python tests we build disk images. Currently this involves > > > using the sudo command for losetup and mount. For some tests sudo is > > > not necessary if the command guestmount of package libguestfs-tools is > > > available. Another useful tool in libguestfs-tools is virt-make-fs. > > > > > > Generally we should get rid of using a root user for building and testing. > > > As a first step install libguestfs-tools in our Docker image. > > > > > > Signed-off-by: Heinrich Schuchardt > > > --- > > > Dockerfile | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/Dockerfile b/Dockerfile > > > index 4138415..20a861f 100644 > > > --- a/Dockerfile > > > +++ b/Dockerfile > > > @@ -57,6 +57,7 @@ RUN apt-get update && apt-get install -y \ > > > iasl \ > > > imagemagick \ > > > iputils-ping \ > > > + libguestfs-tools \ > > > libisl15 \ > > > liblz4-tool \ > > > libpixman-1-dev \ > > > > I haven't applied this as in order to also convert any of the tests to > > libguestfs-tools based approaches, Azure/GitLab (or Travis even) can be > > As far as I know, the only use of sudo can be seen in tests I added: > a test_fs > b test_efi_secboot > > (a) has a flexible approach like: > if (tool_is_in_path('guestmount'): > ... > else > check_call('sudo mount ...') > > I also took a similar approach in (b) when I posted my patch, > but Heinrich deleted "non-sudo" part from my original code > when it was merged. Ah, I had forgotten that guestmount was part of it already. I'll apply this patch soon then, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: