On Thursday, March 4, 2021 at 12:40:55 AM UTC+5:30 Henning Schild wrote:
This will kind of allow "tracking" of those "apt://" guys, and we have
<orig-version> to cover that changelog.

So i wonder if we can make recipes-app/hello/ just one file and drop
those inc and DEFAULT_PREFERENCE bits. That would also make a real CI
test.

Yes. That would be a good test case. I will add another patch and send out a v3.
 


Maybe the docs will need updating as well.

ok
 


Henning

Am Wed, 3 Mar 2021 19:59:11 +0100
schrieb "[ext] Henning Schild" <henning...@siemens.com>:

> Am Thu, 4 Mar 2021 00:08:23 +0530
> schrieb Vijai Kumar K <Vijaikumar_...@mentor.com>:
>
> > The Debian source package fetch and unpacking happens now inside the
> > do_apt_fetch task.
> >
> > With the current do_apt_fetch implementation, it is not possible to
> > use a custom source directory(${S}).
> > apt-get source by default extracts the contents of the debian source
> > into folder with name <pkg>_<version>.
> >
> > Add provision for specifying a custom source directory.
> >
> > Add a new task called do_apt_unpack and move unpacking logic there.
> >
> > Signed-off-by: Vijai Kumar K <Vijaikumar_...@mentor.com>
> > ---
> > meta/classes/dpkg-base.bbclass | 24 +++++++++++++++++++++---
> > 1 file changed, 21 insertions(+), 3 deletions(-)
> >
> > diff --git a/meta/classes/dpkg-base.bbclass
> > b/meta/classes/dpkg-base.bbclass index 5c7bddc..a6c2e3b 100644
> > --- a/meta/classes/dpkg-base.bbclass
> > +++ b/meta/classes/dpkg-base.bbclass
> > @@ -58,7 +58,6 @@ do_apt_fetch() {
> > if [ -z "${@d.getVar("SRC_APT", True).strip()}" ]; then
> > return 0
> > fi
> > - rm -rf ${S}
> > dpkg_do_mounts
> > E="${@ isar_export_proxies(d)}"
> > sudo -E chroot ${BUILDCHROOT_DIR} /usr/bin/apt-get update \
> > @@ -69,8 +68,6 @@ do_apt_fetch() {
> > for uri in "${SRC_APT}"; do
> > sudo -E chroot --userspec=$( id -u ):$( id -g )
> > ${BUILDCHROOT_DIR} \ sh -c 'mkdir -p /downloads/deb-src/"$1"/"$2" &&
> > cd /downloads/deb-src/"$1"/"$2" && apt-get -y --download-only
> > --only-source source "$2"' my_script "${DISTRO}" "${uri}"
> > - sudo -E chroot --userspec=$( id -u ):$( id -g )
> > ${BUILDCHROOT_DIR} \
> > - sh -c 'cp /downloads/deb-src/"$1"/"$2"/* ${PP} && cd
> > ${PP} && apt-get -y --only-source source "$2"' my_script "${DISTRO}"
> > "${uri}" done
> > dpkg_undo_mounts
> > @@ -79,6 +76,27 @@ do_apt_fetch() {
> > addtask apt_fetch after do_unpack before do_patch
>
> before do_apt_unpack
>
> > do_apt_fetch[lockfiles] += "${REPO_ISAR_DIR}/isar.lock"
> >
> > +do_apt_unpack() {
> > + rm -rf ${S}
> > + dpkg_do_mounts
> > + E="${@ isar_export_proxies(d)}"
>
> Not sure about this, i would hope the unpack would be "purely offline"
>
> > + for uri in "${SRC_APT}"; do
> > + sudo -E chroot --userspec=$( id -u ):$( id -g )
> > ${BUILDCHROOT_DIR} \
> > + sh -c ' \
> > + set -e
> > + dscfile="$(apt-get -y -qq --print-uris source
> > "${2}" | cut -d " " -f2 | grep -E "*.dsc")"
>
> but maybe this guy needs network
>
> Thanks, looking good so far!
>
> Henning
>
> > + cd ${PP}
> > + cp /downloads/deb-src/"${1}"/"${2}"/* ${PP}
> > + dpkg-source -x "${dscfile}" "${PPS}"' \
> > + my_script "${DISTRO}" "${uri}"
> > + done
> > +
> > + dpkg_undo_mounts
> > +}
> > +
> > +addtask apt_unpack after do_apt_fetch before do_patch
> > +
> > addtask cleanall_apt before do_cleanall
> > do_cleanall_apt[nostamp] = "1"
> > do_cleanall_apt() {
>