From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6752900179667451904 X-Received: by 2002:a17:906:1b0c:: with SMTP id o12mr4031975ejg.312.1572364933429; Tue, 29 Oct 2019 09:02:13 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a17:906:e2c1:: with SMTP id gr1ls4975165ejb.16.gmail; Tue, 29 Oct 2019 09:02:12 -0700 (PDT) X-Google-Smtp-Source: APXvYqwVJ903xYsq19G5kArMSAZDj+PbGt620UbSqMpTdMCrBiqWYzBoID1RtxdOxEj0HA03Kk1T X-Received: by 2002:a17:906:f252:: with SMTP id gy18mr4098467ejb.224.1572364932904; Tue, 29 Oct 2019 09:02:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1572364932; cv=none; d=google.com; s=arc-20160816; b=f0GUco1RdRwxWJjDwIoTil4lQgGcuG3V9tMRELGc15rU90QMC1GFo21h61vFoJm2Ee mPoldipuZLnR+zpKD/qbIJYwUbZ4MLCSI6/HS3JGnsnRt13AnqmrdTzzkoV+NfO8h4qo vi7z6woTgPokWasQQ/hsOq/JoqSFE/FiH2dqw+U7gKVShDyB+YgFkXaUPtLmWcD656sE UaM83hmIDI+Y5cP6x7pRquDjWRTHC/hxz3uxg0ljp+XvkMAFe9pLYaEy4lZnDRnAwXzf SBLSnlEf7RO1SFo7IKBg3gyEuZfiFCyL+sFJNjtojelNe1OX6CmlqdEhV1mP8bTgWjVp w4lg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date; bh=m9cEFgN69NMrRUWipP/kZJAjgf5cYgYQru74YZgit14=; b=r1UrUs9W2AGP3F9jCmo7GQEZVQTor61hRCWMWfAxuS4Owqia4wmHg9uBZMKcvQbbyT st/AkP6hn7kuCs2nl9/vZAEDmWJ5shnYk/VaF587riTDOlx24ejzcgfz28LZuTq+Bi+0 uxrtD8piGOY+fUh5EVoYzmR5yE/BO1UTcKckKNIgQAq96Pa5nqdAzyMX7ONyC4WvAryA z/2yFNih/F1AZt4D3EQteZ/ucx2JaxuOMMgicdGFa+42I4ahMghn2AwUa73Myzk3c6KB JCWVWRS0QpljJ0NsCbQcbx8IBUiNpBTQgMYL5cEfw+uX3j5GU9CKJY0/PZdj+HfkHGcu 9sNw== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 194.138.37.40 as permitted sender) smtp.mailfrom=henning.schild@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com Return-Path: Received: from gecko.sbs.de (gecko.sbs.de. [194.138.37.40]) by gmr-mx.google.com with ESMTPS id a15si479087ejj.0.2019.10.29.09.02.12 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 29 Oct 2019 09:02:12 -0700 (PDT) Received-SPF: pass (google.com: domain of henning.schild@siemens.com designates 194.138.37.40 as permitted sender) client-ip=194.138.37.40; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of henning.schild@siemens.com designates 194.138.37.40 as permitted sender) smtp.mailfrom=henning.schild@siemens.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=siemens.com Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66]) by gecko.sbs.de (8.15.2/8.15.2) with ESMTPS id x9TG2BZK032555 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 29 Oct 2019 17:02:12 +0100 Received: from md1za8fc.ad001.siemens.net ([139.25.69.153]) by mail2.sbs.de (8.15.2/8.15.2) with ESMTP id x9TG2BB1019441; Tue, 29 Oct 2019 17:02:11 +0100 Date: Tue, 29 Oct 2019 17:02:11 +0100 From: Henning Schild To: Cc: isar-users Subject: Re: [PATCH 08/17] meta: create DL_DIR support for all apt-get downloaded .debs Message-ID: <20191029170211.3d2d1272@md1za8fc.ad001.siemens.net> In-Reply-To: <5c2908a6-97df-46dd-bb9b-21af77202918@googlegroups.com> References: <20191028170132.28072-1-henning.schild@siemens.com> <20191028170132.28072-9-henning.schild@siemens.com> <5c2908a6-97df-46dd-bb9b-21af77202918@googlegroups.com> X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-TUID: 3r7GXtHNCpLZ On Tue, 29 Oct 2019 05:48:01 -0700 wrote: > On Monday, October 28, 2019 at 6:01:36 PM UTC+1, Henning Schild wrote: > > > > From: Henning Schild > > > > > Hook in between the "apt-get --download-only"s and the "apt-get"s > > and copy out all the debs for later. > > > > Signed-off-by: Henning Schild > > > --- > > meta/classes/deb-dl-dir.bbclass | 14 ++++++++++++++ > > meta/classes/dpkg-base.bbclass | 1 + > > meta/classes/dpkg-gbp.bbclass | 1 + > > meta/classes/dpkg.bbclass | 1 + > > meta/classes/image-locales-extension.bbclass | 2 +- > > meta/classes/image-tools-extension.bbclass | 1 + > > meta/classes/rootfs.bbclass | 12 ++++++++++-- > > meta/conf/bitbake.conf | 1 + > > 8 files changed, 30 insertions(+), 3 deletions(-) > > create mode 100644 meta/classes/deb-dl-dir.bbclass > > > > diff --git a/meta/classes/deb-dl-dir.bbclass > > b/meta/classes/deb-dl-dir.bbclass > > new file mode 100644 > > index 0000000..e58df7a > > --- /dev/null > > +++ b/meta/classes/deb-dl-dir.bbclass > > @@ -0,0 +1,14 @@ > > +# This software is a part of ISAR. > > +# Copyright (C) 2019 Siemens AG > > +# > > +# SPDX-License-Identifier: MIT > > + > > +deb_dl_dir_export() { > > + export pc="${DEBDIR}/${DISTRO}" > > + export rootfs="${1}" > > + mkdir -p "${pc}" > > + flock "${pc}".lock -c ' \ > > + sudo find "${rootfs}"/var/cache/apt/archives/ -type f > > -iname '*\.deb' \ > > + -exec cp -f '{}' "${pc}" \; > > > > use -exec command {} + to reduce the number of fork/execvp pairs? > note: you would need to use cp -t "${pc}" since the "+" modifier > appends matches to the end of the command Thanks, this was actually on my mind when i wrote it but eventually overlooked. I will look into it. Henning > > > + ' > > +} > > diff --git a/meta/classes/dpkg-base.bbclass > > b/meta/classes/dpkg-base.bbclass > > index b39df5c..8815478 100644 > > --- a/meta/classes/dpkg-base.bbclass > > +++ b/meta/classes/dpkg-base.bbclass > > @@ -8,6 +8,7 @@ inherit buildchroot > > inherit debianize > > inherit terminal > > inherit repository > > +inherit deb-dl-dir > > > > DEPENDS ?= "" > > > > diff --git a/meta/classes/dpkg-gbp.bbclass > > b/meta/classes/dpkg-gbp.bbclass index d9c3af0..9e6945b 100644 > > --- a/meta/classes/dpkg-gbp.bbclass > > +++ b/meta/classes/dpkg-gbp.bbclass > > @@ -16,6 +16,7 @@ do_install_builddeps_append() { > > sudo -E chroot ${BUILDCHROOT_DIR} \ > > apt-get install -y -o Debug::pkgProblemResolver=yes \ > > --no-install-recommends --download-only > > ${GBP_DEPENDS} > > + deb_dl_dir_export "${BUILDCHROOT_DIR}" > > sudo -E chroot ${BUILDCHROOT_DIR} \ > > apt-get install -y -o Debug::pkgProblemResolver=yes \ > > --no-install-recommends ${GBP_DEPENDS} > > diff --git a/meta/classes/dpkg.bbclass b/meta/classes/dpkg.bbclass > > index d809cca..b6844e9 100644 > > --- a/meta/classes/dpkg.bbclass > > +++ b/meta/classes/dpkg.bbclass > > @@ -9,6 +9,7 @@ do_install_builddeps() { > > E="${@ bb.build.exec_func('isar_export_proxies', d)}" > > sudo -E chroot ${BUILDCHROOT_DIR} /isar/deps.sh \ > > ${PP}/${PPS} ${DISTRO_ARCH} --download-only > > + deb_dl_dir_export "${BUILDCHROOT_DIR}" > > sudo -E chroot ${BUILDCHROOT_DIR} /isar/deps.sh \ > > ${PP}/${PPS} ${DISTRO_ARCH} > > dpkg_undo_mounts > > diff --git a/meta/classes/image-locales-extension.bbclass > > b/meta/classes/image-locales-extension.bbclass > > index 3c0758f..0f0d0ca 100644 > > --- a/meta/classes/image-locales-extension.bbclass > > +++ b/meta/classes/image-locales-extension.bbclass > > @@ -25,7 +25,7 @@ def get_nopurge(d): > > j.split()[0].split(".")[0], > > j.split()[0])))) > > > > -ROOTFS_INSTALL_COMMAND_BEFORE_CLEAN += > > "image_install_localepurge_download" > > +ROOTFS_INSTALL_COMMAND_BEFORE_EXPORT += > > "image_install_localepurge_download" > > image_install_localepurge_download[weight] = "40" > > image_install_localepurge_download() { > > sudo -E chroot '${ROOTFSDIR}' \ > > diff --git a/meta/classes/image-tools-extension.bbclass > > b/meta/classes/image-tools-extension.bbclass > > index 243fa11..3906585 100644 > > --- a/meta/classes/image-tools-extension.bbclass > > +++ b/meta/classes/image-tools-extension.bbclass > > @@ -34,6 +34,7 @@ do_install_imager_deps() { > > --allow-unauthenticated --allow-downgrades > > --download-only install \ > > ${IMAGER_INSTALL}' > > > > + deb_dl_dir_export ${BUILDCHROOT_DIR} > > sudo -E chroot ${BUILDCHROOT_DIR} sh -c ' \ > > apt-get -o Debug::pkgProblemResolver=yes > > --no-install-recommends -y \ > > --allow-unauthenticated --allow-downgrades install \ > > diff --git a/meta/classes/rootfs.bbclass > > b/meta/classes/rootfs.bbclass index 095ebc1..841aa89 100644 > > --- a/meta/classes/rootfs.bbclass > > +++ b/meta/classes/rootfs.bbclass > > @@ -1,6 +1,8 @@ > > # This software is a part of ISAR. > > # Copyright (c) Siemens AG, 2019 > > > > +inherit deb-dl-dir > > + > > ROOTFS_ARCH ?= "${DISTRO_ARCH}" > > ROOTFS_DISTRO ?= "${DISTRO}" > > ROOTFS_PACKAGES ?= "" > > @@ -119,8 +121,14 @@ rootfs_install_pkgs_download() { > > /usr/bin/apt-get ${ROOTFS_APT_ARGS} --download-only > > ${ROOTFS_PACKAGES} > > } > > > > -ROOTFS_INSTALL_COMMAND_BEFORE_CLEAN ??= "" > > -ROOTFS_INSTALL_COMMAND += "${ROOTFS_INSTALL_COMMAND_BEFORE_CLEAN}" > > +ROOTFS_INSTALL_COMMAND_BEFORE_EXPORT ??= "" > > +ROOTFS_INSTALL_COMMAND += > > "${ROOTFS_INSTALL_COMMAND_BEFORE_EXPORT}" > > + > > +ROOTFS_INSTALL_COMMAND += "rootfs_export_package_cache" > > +rootfs_export_package_cache[weight] = "5" > > +rootfs_export_package_cache() { > > + deb_dl_dir_export ${ROOTFSDIR} > > +} > > > > ROOTFS_INSTALL_COMMAND += "${@ 'rootfs_install_clean_files' if > > (d.getVar('ROOTFS_CLEAN_FILES') or '').strip() else ''}" > > rootfs_install_clean_files[weight] = "2" > > diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf > > index 4c0a809..56f4fbf 100644 > > --- a/meta/conf/bitbake.conf > > +++ b/meta/conf/bitbake.conf > > @@ -26,6 +26,7 @@ DEPLOY_DIR = "${TMPDIR}/deploy" > > FILESPATH = > > "${FILE_DIRNAME}/${PF}:${FILE_DIRNAME}/${P}:${FILE_DIRNAME}/${PN}:${FILE_DIRNAME}/files:${FILE_DIRNAME}" > > > > FILE_DIRNAME = "${@os.path.dirname(d.getVar('FILE', False))}" > > GITDIR = "${DL_DIR}/git" > > +DEBDIR = "${DL_DIR}/deb" > > DEBSRCDIR = "${DL_DIR}/deb-src" > > P = "${PN}-${PV}" > > PF = "${PN}-${PV}-${PR}" > > -- > > 2.23.0 > > > > >