From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6935855593929310208 Date: Thu, 15 Apr 2021 02:01:49 -0700 (PDT) From: "vijaikumar....@gmail.com" To: isar-users Message-Id: In-Reply-To: <20210405083459.12277-1-Vijaikumar_Kanagarajan@mentor.com> References: <20210405083459.12277-1-Vijaikumar_Kanagarajan@mentor.com> Subject: Re: [PATCH v5 0/3] Custom ${S} series MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_1138_1458639927.1618477309906" X-TUID: wIqeXhRByMNa ------=_Part_1138_1458639927.1618477309906 Content-Type: multipart/alternative; boundary="----=_Part_1139_808457916.1618477309906" ------=_Part_1139_808457916.1618477309906 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Can this be merged? If there are no other review comments? Thanks, Vijai Kumar K On Monday, April 5, 2021 at 2:05:22 PM UTC+5:30 vijaikumar_...@mentor.com wrote: > Changes since V4: > - Added P3 containing detailed documentation of apt:// in user manual. > > Changes since V3: > - Fix build issue with qemumipsel-stretch. > (Need to pass --only source with --print-uris, since we are dealing > with source package) > > CI build succeeded: http://ci.isar-build.org:8080/job/isar_vkk_devel/95/ > > Changes since V2: > - Introduce P2 to demonstrate apt:// without ${PV} set. > - Address review comment from Henning. > - Fix issue where do_apt_unpack is triggered for non apt:// > SRC_URIs. > > CI build going on: http://ci.isar-build.org:8080/job/isar_vkk_devel/93/ > > Changes since V1: > > - Introduce a separate do_apt_unpack task > > Vijai Kumar K (3): > dpkg-base: Introduce do_apt_unpack > Modify hello to demonstrate apt:// without ${PV} > docs: Add detailed documentation about apt:// > > doc/user_manual.md | 61 +++++++++++++++++-- > .../recipes-app/hello/{hello.inc => hello.bb} | 0 > meta-isar/recipes-app/hello/hello_2.10.bb | 11 ---- > meta-isar/recipes-app/hello/hello_2.9.bb | 8 --- > meta/classes/dpkg-base.bbclass | 29 +++++++-- > 5 files changed, 82 insertions(+), 27 deletions(-) > rename meta-isar/recipes-app/hello/{hello.inc => hello.bb} (100%) > delete mode 100644 meta-isar/recipes-app/hello/hello_2.10.bb > delete mode 100644 meta-isar/recipes-app/hello/hello_2.9.bb > > -- > 2.17.1 > > ------=_Part_1139_808457916.1618477309906 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 7bit Can this be merged? If there are no other review comments?

Thanks,
Vijai Kumar K

On Monday, April 5, 2021 at 2:05:22 PM UTC+5:30 vijaikumar_...@mentor.com wrote:
Changes since V4:
- Added P3 containing detailed documentation of apt:// in user manual.

Changes since V3:
- Fix build issue with qemumipsel-stretch.
(Need to pass --only source with --print-uris, since we are dealing
with source package)

CI build succeeded: http://ci.isar-build.org:8080/job/isar_vkk_devel/95/

Changes since V2:
- Introduce P2 to demonstrate apt:// without ${PV} set.
- Address review comment from Henning.
- Fix issue where do_apt_unpack is triggered for non apt://
SRC_URIs.

CI build going on: http://ci.isar-build.org:8080/job/isar_vkk_devel/93/

Changes since V1:

- Introduce a separate do_apt_unpack task

Vijai Kumar K (3):
dpkg-base: Introduce do_apt_unpack
Modify hello to demonstrate apt:// without ${PV}
docs: Add detailed documentation about apt://

doc/user_manual.md | 61 +++++++++++++++++--
.../recipes-app/hello/{hello.inc => hello.bb} | 0
meta-isar/recipes-app/hello/hello_2.10.bb | 11 ----
meta-isar/recipes-app/hello/hello_2.9.bb | 8 ---
meta/classes/dpkg-base.bbclass | 29 +++++++--
5 files changed, 82 insertions(+), 27 deletions(-)
rename meta-isar/recipes-app/hello/{hello.inc => hello.bb} (100%)
delete mode 100644 meta-isar/recipes-app/hello/hello_2.10.bb
delete mode 100644 meta-isar/recipes-app/hello/hello_2.9.bb

--
2.17.1

------=_Part_1139_808457916.1618477309906-- ------=_Part_1138_1458639927.1618477309906--