From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6683827867816558592 X-Received: by 2002:a19:3f54:: with SMTP id m81mr34210503lfa.98.1556616849463; Tue, 30 Apr 2019 02:34:09 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a2e:8013:: with SMTP id j19ls3516268ljg.0.gmail; Tue, 30 Apr 2019 02:34:08 -0700 (PDT) X-Google-Smtp-Source: APXvYqz1ERGev7I5CFTpOMpbNW8FcPtprCef3V92aGTZHMHY0UCDP2wztsz6NYbgd2q9VHPWJDff X-Received: by 2002:a2e:7a03:: with SMTP id v3mr25930242ljc.142.1556616848957; Tue, 30 Apr 2019 02:34:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1556616848; cv=none; d=google.com; s=arc-20160816; b=l/9gKbOIRN+ICEzvHLrsYCvsU+Ye5cHIC+pRzzUT7Wcqc5YWq4e46hboiQfglWjkDP x4hqrVbpqlB33mKYf/AJ1MGKIn6evJrtGh7TECl13Fo9UoxbJYThbWpgIBWAR2jKoNYL yGBuFB1hQMp0YHoTfZAzOO+lkW3wwT0kAI4iyvNLQxi4YGL7oMJgcb4j1Zj6/XoLJrPk LQk1B2qYGxGIbCTGD0o298jt1MhkP+INc2caMIhc8/tqzWrwBgntJa2BttiXDQ7QB2Ks 1E7GaupWE1rZn+eMumbnbJQAmkbXSLK3AL3PZiMC9MAPLT/OEu5BUHP2eLJpO7xxtXUc t59A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=user-agent:in-reply-to:content-disposition:content-description :mime-version:references:message-id:subject:cc:to:from:date; bh=X14TbayRuop/cnpYjHA4XQ2B821gRzOKiUrB5iVe848=; b=cQBWoYFbdLjmIeFZSv6ZBL0wfDQGipmtTULo0bA6dvZ6UFyv6Y4idkupUQJi9jS/CM JnnG+znkPYzkkQhD27L73GxczuutJjxHT9jK5GDnec39mJqDA8oj34GmSOQq+BNtJah1 jPsfU5kOYMSl+mbHAKJqUbfSRFbtcNS78SPcjWCTGf6R1/q8vJjF92og9T5XwozSR+Ru 38TJ9q9DFRjHtUjx6ScDfL/c0j68LvMHMzgbRBQUytJ7mpgRoeCmRKrmpqFvakY2nbCY 0MGNxDJ9UQST4xtr0px8KkIohjH3c3dJEI92awaD/vdk39uxvOmPXnlwMTZPO9l3d7fp IcZA== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of andreas.reichel.ext@siemens.com designates 194.138.37.39 as permitted sender) smtp.mailfrom=andreas.reichel.ext@siemens.com Return-Path: Received: from lizzard.sbs.de (lizzard.sbs.de. [194.138.37.39]) by gmr-mx.google.com with ESMTPS id u30si1036632lfn.4.2019.04.30.02.34.08 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 30 Apr 2019 02:34:08 -0700 (PDT) Received-SPF: pass (google.com: domain of andreas.reichel.ext@siemens.com designates 194.138.37.39 as permitted sender) client-ip=194.138.37.39; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of andreas.reichel.ext@siemens.com designates 194.138.37.39 as permitted sender) smtp.mailfrom=andreas.reichel.ext@siemens.com Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66]) by lizzard.sbs.de (8.15.2/8.15.2) with ESMTPS id x3U9Y7EU012450 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 30 Apr 2019 11:34:08 +0200 Received: from iiotirae (golem.ppmd.siemens.net [139.25.69.122]) by mail2.sbs.de (8.15.2/8.15.2) with SMTP id x3U9Y6Of018488; Tue, 30 Apr 2019 11:34:07 +0200 Date: Tue, 30 Apr 2019 11:34:06 +0200 From: Andreas Reichel To: "Maxim Yu. Osipov" Cc: Claudius Heine , claudius.heine.ext@siemens.com, isar-users@googlegroups.com Subject: Re: [PATCH v4 1/9] isar-bootstrap-host: disable DISTRO_APT_KEYS usage Message-ID: <20190430093405.GA14335@iiotirae> References: <20190425134450.13443-1-claudius.heine.ext@siemens.com> <20190425134450.13443-2-claudius.heine.ext@siemens.com> <155626421155.10914.2537647574220599237@ardipi> <89e6b417-265c-b1a6-b151-0938fed5d462@ilbers.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Description: message Content-Disposition: inline In-Reply-To: <89e6b417-265c-b1a6-b151-0938fed5d462@ilbers.de> User-Agent: Mutt/1.11.4 (2019-03-13) X-TUID: S8UzslKtH0Nm On Fri, Apr 26, 2019 at 11:41:16AM +0300, Maxim Yu. Osipov wrote: > Hi Claudius, Andreas, > > @Andreas > Your input is very welcome at this topic as you were busy with all this APT > keys stuff. > Thank you. Well in my eyes, Claudius delivers important changes to Isar which improve code quality and the build steps as a whole. Also as I know him, he is never ever interested in any personal "affairs" whatsoever, which I also learnt from his reviews on my code. He has a sharp mind and always tries to get out the best of the code up to his knowledge and techniques. Furthermore, if he may sometimes sound direct or even annoying to some - this is merely a personal question of how one focuses on mails. I also did not understand everything he criticized on my code in the beginning - but after I understood him, it was always great improvement. So, Maxim, I beg that you do not take anything personal on any mail, regarding any words or writing style but just focus on the code as I always try - which always helps to go on further with the project and to improve it. That's my input. Andreas > On 4/26/19 9:36 AM, Claudius Heine wrote: > > Hi Maxim, > > > > Quoting Maxim Yu. Osipov (2019-04-25 20:20:59) > > > On 4/25/19 3:44 PM, claudius.heine.ext@siemens.com wrote: > > > > From: Claudius Heine > > > > > > > > isar-bootstrap-host only supports bootstrapping Debian root file > > > > systems. Therefore deactivate any DISTRO_APT_KEYS from other > > > > distributions. > > > > > > > > Signed-off-by: Claudius Heine > > > > --- > > > > meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb > > > > index 08b068f..3e96281 100644 > > > > --- a/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb > > > > +++ b/meta/recipes-core/isar-bootstrap/isar-bootstrap-host.bb > > > > @@ -12,6 +12,8 @@ DEPLOY_ISAR_BOOTSTRAP = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}" > > > > ISAR_BOOTSTRAP_LOCK = "${DEPLOY_DIR_BOOTSTRAP}/${HOST_DISTRO}-${HOST_ARCH}.lock" > > > > require isar-bootstrap.inc > > > > +# We only build debian host buildchroot environments > > > > +DISTRO_APT_KEYS = "" > > > > > > From the first glance this modification limits functionality. > > > It looks like a hack and I would suggest to avoid this modification. > > > > Well it is a fix and that limited functionality was already present but > > just implicit, hidden behind some bug and the cleanup just made it > > appear. > > > Could you please point to this hidden, implicit place where mentioned bug > persists? > > I've looked under meta/recipes-core/isar-bootstrap/ > > It seems that keyring stuff is quite symmetrical (in terms of host/target): > > isar/meta/recipes-core/isar-bootstrap$ grep -ri keyring * > isar-bootstrap-host.bb:do_generate_keyring[stamp-extra-info] = > "${DISTRO}-${DISTRO_ARCH}" > isar-bootstrap-host.bb:addtask bootstrap before do_build after > do_generate_keyring > isar-bootstrap.inc:APTKEYRING = "${WORKDIR}/apt-keyring.gpg" > isar-bootstrap.inc:DEBOOTSTRAP_KEYRING = "" > isar-bootstrap.inc: d.setVar("DEBOOTSTRAP_KEYRING", "--keyring > ${APTKEYRING}") > isar-bootstrap.inc: d.setVar("DEBOOTSTRAP_KEYRING", "--keyring > ${APTKEYRING}") > isar-bootstrap.inc:do_generate_keyring[dirs] = "${DL_DIR}" > isar-bootstrap.inc:do_generate_keyring[vardeps] += "DISTRO_APT_KEYS" > isar-bootstrap.inc:do_generate_keyring() { > isar-bootstrap.inc: gpg --no-default-keyring --keyring > "${APTKEYRING}" \ > isar-bootstrap.inc:addtask generate_keyring before do_build after do_unpack > isar-bootstrap.inc: ${DEBOOTSTRAP_KEYRING} \ > isar-bootstrap.inc: ${DEBOOTSTRAP_KEYRING} \ > isar-bootstrap-target.bb:do_generate_keyring[stamp-extra-info] = > "${DISTRO}-${DISTRO_ARCH}" > isar-bootstrap-target.bb:addtask bootstrap before do_build after > do_generate_keyring > isar/meta/recipes-core/isar-bootstrap$ > > > And bootstrapping itself (function isar_bootsrap in isar-bootstrap.inc) > differs only by passing extra '--arch' to target DISTRO_ARCH. Nothing > regarding > > if [ ${IS_HOST} ]; then > ${DEBOOTSTRAP} $debootstrap_args \ > ${@get_distro_components_argument(d, True)} \ > ${DEBOOTSTRAP_KEYRING} \ > "${@get_distro_suite(d, True)}" \ > "${ROOTFSDIR}" \ > "${@get_distro_source(d, True)}" > > else > "${DEBOOTSTRAP}" $debootstrap_args \ > --arch="${DISTRO_ARCH}" \ > ${@get_distro_components_argument(d, > False)} \ > ${DEBOOTSTRAP_KEYRING} \ > "${@get_distro_suite(d, False)}" \ > "${ROOTFSDIR}" \ > "${@get_distro_source(d, False)}" > fi > > > > > > > > > Some time ago I thought about introduction of HOST_DISTRO_APT_KEYS to > > > avoid confusion between target and host apt keys. > > > > Good idea. But that would be a new feature/improvement. > > Yes. But your series is also improvement, isn't? > > I need more arguments for introduction of this limitation. > > Maxim. > > > Also thanks for looking at the code! > > > > Claudius > > > > > > > > > > > Maxim. > > > > > > > > > > > > > inherit isar-bootstrap-helper > > > > do_generate_keyring[stamp-extra-info] = "${DISTRO}-${DISTRO_ARCH}" > > > > > > > > > > > > > -- > > > Maxim Osipov > > > ilbers GmbH > > > Maria-Merian-Str. 8 > > > 85521 Ottobrunn > > > Germany > > > +49 (151) 6517 6917 > > > mosipov@ilbers.de > > > http://ilbers.de/ > > > Commercial register Munich, HRB 214197 > > > General Manager: Baurzhan Ismagulov > > > > > > -- > > > You received this message because you are subscribed to the Google Groups "isar-users" group. > > > To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com. > > > To post to this group, send email to isar-users@googlegroups.com. > > > To view this discussion on the web visit https://groups.google.com/d/msgid/isar-users/ccc13295-982c-7b25-cfc2-e079033689c0%40ilbers.de. > > > For more options, visit https://groups.google.com/d/optout. > > > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: (+49)-8142-66989-54 Fax: (+49)-8142-66989-80 Email: ch@denx.de > > > > PGP key: 6FF2 E59F 00C6 BC28 31D8 64C1 1173 CB19 9808 B153 > > Keyserver: hkp://pool.sks-keyservers.net > > > > > -- > Maxim Osipov > ilbers GmbH > Maria-Merian-Str. 8 > 85521 Ottobrunn > Germany > +49 (151) 6517 6917 > mosipov@ilbers.de > http://ilbers.de/ > Commercial register Munich, HRB 214197 > General Manager: Baurzhan Ismagulov -- Andreas Reichel Dipl.-Phys. (Univ.) Software Consultant Andreas.Reichel@tngtech.com, +49-174-3180074 TNG Technology Consulting GmbH, Betastr. 13a, 85774 Unterfoehring Geschaeftsfuehrer: Henrik Klagges, Dr. Robert Dahlke, Gerhard Mueller Sitz: Unterfoehring * Amtsgericht Muenchen * HRB 135082