From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6680759771290664960 X-Received: by 2002:a2e:8550:: with SMTP id u16mr16248104ljj.11.1556089920105; Wed, 24 Apr 2019 00:12:00 -0700 (PDT) X-BeenThere: isar-users@googlegroups.com Received: by 2002:a19:f807:: with SMTP id a7ls1369960lff.11.gmail; Wed, 24 Apr 2019 00:11:59 -0700 (PDT) X-Google-Smtp-Source: APXvYqxHZ261q9K/jeq5iAqkurBgGriUAnZLQL228zj0RQ3lVs7gXL+88LjRj60m4WBcP2WqsbZy X-Received: by 2002:ac2:518b:: with SMTP id u11mr17224395lfi.46.1556089919523; Wed, 24 Apr 2019 00:11:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1556089919; cv=none; d=google.com; s=arc-20160816; b=lgvelqshdDMEof2znJOe9aIzpRtk8enp2VN1JvIhmX2AhlAFnS9FV588c9NeuIw8xE fntu/eWrEzcL6c2lGXkq9LVgh456QyYTTCKHtN0X+eqgei0Vf675oAr9f07ty/8x/4f2 VbLbkh2UQzXYwX6phaC/GNuQ4Pc3PlSyzXUzMtiMac++lQeJPYC7jtX9ZjV7LLZDUs8T 6zNz6mFkxcBQR9EF6Jfw7utfutRsi5IZ/4oKtBrtN2cjI5+elHlscON2N05NnQWIiSDV iyeu02H+qrQkAfLEVGEb1LhpdiIZ70UXekoCT3eRTUO23ZpQ1CjW+OoA1Fa7H20i7rhd 7lZw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:content-language:in-reply-to:mime-version :user-agent:date:message-id:from:references:cc:to:subject; bh=ED8iDA7omU7ZTv5I8f9Tl5Umn/58WcFyOEp686xPCZE=; b=PEKbXmnyJRDRrZnNGCh407GBmNVW/FExMWnEVido1RNP1fn4McyUTgPf/TXLaCQW0+ 9VeN5d8L+oYbDX9XD7P2biRGusIUItzXdPShIUYO+cohPRmSRiOFeZlP6/z0aTEZ6qyv Ll6UTzGDN47UaxrFbhQDhTiGK9/NktuTTd4j7aubG+v7U2Ab7TDDEV7ZQDOETwQG1CSV wMHxGIxZmjtFuJfWCjSrfy8V8aZ6UlmRDOKMlZO1lhEjAlgfBNLcQluIJyPLw4ucGvmm wA6fKjI8EVb9JHpP44M2rlfaH2HshZqqLAFvcW4WUrUmJcAlx6HPi+unUlLp1f8cyArO aLYg== ARC-Authentication-Results: i=1; gmr-mx.google.com; spf=pass (google.com: domain of claudius.heine.ext@siemens.com designates 192.35.17.14 as permitted sender) smtp.mailfrom=claudius.heine.ext@siemens.com Return-Path: Received: from david.siemens.de (david.siemens.de. [192.35.17.14]) by gmr-mx.google.com with ESMTPS id v89si816451lje.4.2019.04.24.00.11.59 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 24 Apr 2019 00:11:59 -0700 (PDT) Received-SPF: pass (google.com: domain of claudius.heine.ext@siemens.com designates 192.35.17.14 as permitted sender) client-ip=192.35.17.14; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of claudius.heine.ext@siemens.com designates 192.35.17.14 as permitted sender) smtp.mailfrom=claudius.heine.ext@siemens.com Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66]) by david.siemens.de (8.15.2/8.15.2) with ESMTPS id x3O7BwkY001318 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 24 Apr 2019 09:11:58 +0200 Received: from [139.25.69.232] (linux-ses-ext02.ppmd.siemens.net [139.25.69.232]) by mail2.sbs.de (8.15.2/8.15.2) with ESMTP id x3O7Bv9n025000; Wed, 24 Apr 2019 09:11:57 +0200 Subject: Re: [PATCH 0/6] pre-processing pipeline and transient package replacement To: "Maxim Yu. Osipov" , isar-users@googlegroups.com Cc: Claudius Heine References: <20190418112545.1201-1-claudius.heine.ext@siemens.com> <1732702c-ab31-eca1-e889-43ee78e67428@ilbers.de> From: Claudius Heine Message-ID: <3899d35f-f79d-b459-caf0-ba04e87d18b6@siemens.com> Date: Wed, 24 Apr 2019 09:11:57 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: <1732702c-ab31-eca1-e889-43ee78e67428@ilbers.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit X-TUID: JuBeLLfU5lQ+ Hi Maxim, On 22/04/2019 16.09, Maxim Yu. Osipov wrote: > The patch set doesn't apply against current 'next' (it would be more > convenient to mention that this patchset depends on your previous series > [PATCH v2 0/8] Cleanup rootfs creation). I did in v1 of that patchset: > this patchset contains some patches that where developed while > implementing the preprocessing image pipeline. They are universally > useful, but do prepare for the next steps. But you are correct that I did not repeat myself in this one. Maybe we should start looking into how to improve this process wise. Would it help to post a git url for each patchset? Maybe it might be possible to merge bigger patchset partially, so each commit that looks good, instead of rejecting always whole patchsets? The reason why I split patchsets up is that I hope that those will be merged faster and thus lowering the work to constantly rebase everything on the current next. So for example if patch 5 of 8 makes a problem, just merge 1-4 and let the dev resent the fixed patchset containing just patch 5 to 8. If that would be done then I would have no need to try split patchsets up myself. Claudius > > Maxim. > > On 4/18/19 1:25 PM, claudius.heine.ext@siemens.com wrote: >> From: Claudius Heine >> >> Hi, >> >> here is the tested, documented and slightly modified version of the >> previous >> RFC patch set. >> >> The CI has shown that the RFC patchset was not rebuildable. This >> was patched by consolidating the  `do_rootfs_prepare`, >> `do_roofs_configure` >> and `do_rootfs_install` into just one task `do_rootfs_install` that >> now calls >> the functions of both previous tasks as well as the `rootfs_prepare` >> function >> that previously was its own task `do_rootfs_prepare`. >> >> Otherwise I added documentation and more verbose commit messages. >> >> regards, >> Claudius >> >> Claudius Heine (6): >>    split up isar-bootstrap helper and implement pre-process pipeline >>    meta: remove transient package support >>    meta/classes: add image-locales-extension class >>    meta/classes: add image-account-extension class >>    doc: update description of image customization >>    doc: some fixes >> >>   doc/user_manual.md                            |  52 +++- >>   meta/classes/image-account-extension.bbclass  | 253 ++++++++++++++++++ >>   meta/classes/image-locales-extension.bbclass  |  97 +++++++ >>   meta/classes/image.bbclass                    |  30 +-- >>   meta/classes/isar-bootstrap-helper.bbclass    | 143 ---------- >>   meta/classes/rootfs.bbclass                   | 176 +++++++++++- >>   .../isar-bootstrap/isar-bootstrap.inc         |   5 + >>   .../buildchroot/buildchroot-host.bb           |  16 +- >>   .../buildchroot/buildchroot-target.bb         |   2 - >>   .../buildchroot/buildchroot.inc               |  26 +- >>   meta/recipes-devtools/sdkchroot/sdkchroot.bb  |  47 ++-- >>   .../isar-cfg-localepurge/files/postinst       |  15 -- >>   .../isar-cfg-localepurge.bb                   |  70 ----- >>   .../isar-cfg-rootpw/files/postinst.tmpl       |  19 -- >>   .../isar-cfg-rootpw/isar-cfg-rootpw.bb        |  19 -- >>   15 files changed, 624 insertions(+), 346 deletions(-) >>   create mode 100644 meta/classes/image-account-extension.bbclass >>   create mode 100644 meta/classes/image-locales-extension.bbclass >>   delete mode 100644 meta/classes/isar-bootstrap-helper.bbclass >>   delete mode 100644 >> meta/recipes-support/isar-cfg-localepurge/files/postinst >>   delete mode 100644 >> meta/recipes-support/isar-cfg-localepurge/isar-cfg-localepurge.bb >>   delete mode 100644 >> meta/recipes-support/isar-cfg-rootpw/files/postinst.tmpl >>   delete mode 100644 >> meta/recipes-support/isar-cfg-rootpw/isar-cfg-rootpw.bb >> > > -- 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