Re: [PATCH 2/6] classes/image: Provide /dev/null for Stretch apt

[Henning Schild <henning.schild@siemens.com>]

Am Mon, 28 Aug 2017 17:20:06 +0200
schrieb "[ext] Henning Schild" <henning.schild@siemens.com>:

> Am Sun, 27 Aug 2017 18:13:35 +0300
> schrieb Alexander Smirnov <asmirnov@ilbers.de>:
> 
> > From: Baurzhan Ismagulov <ibr@ilbers.de>
> > 
> > Signed-off-by: Alexander Smirnov <asmirnov@ilbers.de>
> > Signed-off-by: Baurzhan Ismagulov <ibr@ilbers.de>
> > ---
> >  meta/classes/image.bbclass                              | 3 +++
> >  meta/recipes-devtools/buildchroot/files/configscript.sh | 1 +
> >  2 files changed, 4 insertions(+)
> > 
> > diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> > index f42aa48..c2ff453 100644
> > --- a/meta/classes/image.bbclass
> > +++ b/meta/classes/image.bbclass
> > @@ -20,11 +20,14 @@ do_populate() {
> >              sudo mkdir -p "${S}/${DEBCACHEMNT}"
> >              sudo mount -o bind "${DIR_CACHE}" "${S}/${DEBCACHEMNT}"
> >  
> > +            # apt-get http method, gpg require /dev/null
> > +            # Otherwise, apt http method hangs on Stretch
> >              sudo chroot "${S}" apt-get update -y
> >              for package in ${IMAGE_INSTALL}; do
> >                  sudo chroot "${S}" apt-get install -t
> > "${DEBDISTRONAME}" -y \ --allow-unauthenticated "${package}"
> >              done
> > +            sudo umount "${S}/dev"  
> 
> I see a new umount and a comment, where is the new mount?
> 
> Henning
> 
> >              sudo umount "${S}/${DEBCACHEMNT}"
> >          else
> > diff --git a/meta/recipes-devtools/buildchroot/files/configscript.sh
> > b/meta/recipes-devtools/buildchroot/files/configscript.sh index
> > 5080ae1..08c818b 100644 ---
> > a/meta/recipes-devtools/buildchroot/files/configscript.sh +++
> > b/meta/recipes-devtools/buildchroot/files/configscript.sh @@ -40,6
> > +40,7 @@ export LC_ALL=C LANGUAGE=C
> > LANG=C /var/lib/dpkg/info/dash.preinst install 
> >  # apt-get http method, gpg require /dev/null
> > +# Otherwise, apt http method hangs on Stretch
> >  mount -t devtmpfs -o mode=0755,nosuid devtmpfs /dev

Found it ... and it is not new.

Can we not get the "mount; { stuff } umount;" into one file?

But we should first discuss whether we will drop do_populate
alltogether. See
"[PATCH 3/3] classes: image: remove populate and replace it with a
custom repo"

Henning
 
> >  #configuring packages  
>