Re: [PATCH v6 1/7] vm-img: move template out of package into SRC_URI file

[Henning Schild <henning.schild@siemens.com>]

Am Fri, 17 Dec 2021 17:48:29 +0300
schrieb Anton Mikanovich <amikan@ilbers.de>:

> From: Henning Schild <henning.schild@siemens.com>
> 
> Instead of placing just one static template into a file coming via a
> package, fetch that template file. That allows a choice in template
> files, or any kind of modification in layers. Where people might not
> find our template to be sufficient.
> 
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
>  meta/classes/vm-img.bbclass                      |  8 +++++---
>  .../vm-img}/vm-template.ovf.tmpl                 |  0
>  .../vm-template/vm-template_0.1.bb               | 16
> ---------------- 3 files changed, 5 insertions(+), 19 deletions(-)
>  rename meta/{recipes-devtools/vm-template/files =>
> classes/vm-img}/vm-template.ovf.tmpl (100%) delete mode 100644
> meta/recipes-devtools/vm-template/vm-template_0.1.bb
> 
> diff --git a/meta/classes/vm-img.bbclass b/meta/classes/vm-img.bbclass
> index c75a544..84ca328 100644
> --- a/meta/classes/vm-img.bbclass
> +++ b/meta/classes/vm-img.bbclass
> @@ -7,8 +7,10 @@
>  inherit buildchroot
>  inherit wic-img
>  
> -IMAGER_BUILD_DEPS += "vm-template"
> -IMAGER_INSTALL += "qemu-utils gawk uuid-runtime vm-template"
> +FILESEXTRAPATHS_prepend := "${FILE_DIRNAME}/vm-img:"

Ok let us see if that will work. If the recipes have issues finding the
template then we should use

FILESEXTRAPATHS_prepend := "${LAYERDIR_core}/classes/vm-img:"

Henning

> +SRC_URI += "file://vm-template.ovf.tmpl"
> +
> +IMAGER_INSTALL += "qemu-utils gawk uuid-runtime"
>  
>  # virtual machine disk settings
>  SOURCE_IMAGE_FILE ?= "${IMAGE_FULLNAME}.wic.img"
> @@ -101,7 +103,7 @@ do_create_ova() {
>          export DISK_UUID=$(uuidgen)
>          export VM_UUID=$(uuidgen)
>          # create ovf
> -        cat /usr/share/vm-template/vm-template.ovf.tmpl | envsubst >
> ${PP_DEPLOY}/${OVA_NAME}.ovf
> +        cat ${PP_WORK}/vm-template.ovf.tmpl | envsubst >
> ${PP_DEPLOY}/${OVA_NAME}.ovf tar -cvf ${PP_DEPLOY}/${OVA_NAME}.ova -C
> ${PP_DEPLOY} ${OVA_NAME}.ovf 
>          # VirtualBox needs here a manifest file. VMware does accept
> that format. diff --git
> a/meta/recipes-devtools/vm-template/files/vm-template.ovf.tmpl
> b/meta/classes/vm-img/vm-template.ovf.tmpl similarity index 100%
> rename from
> meta/recipes-devtools/vm-template/files/vm-template.ovf.tmpl rename
> to meta/classes/vm-img/vm-template.ovf.tmpl diff --git
> a/meta/recipes-devtools/vm-template/vm-template_0.1.bb
> b/meta/recipes-devtools/vm-template/vm-template_0.1.bb deleted file
> mode 100644 index 1d474cd..0000000 ---
> a/meta/recipes-devtools/vm-template/vm-template_0.1.bb +++ /dev/null
> @@ -1,16 +0,0 @@ -# This software is a part of ISAR.
> -#
> -# Copyright (c) Siemens AG, 2020
> -#
> -# SPDX-License-Identifier: MIT
> -
> -inherit dpkg-raw
> -
> -SRC_URI += "file://vm-template.ovf.tmpl"
> -
> -do_install() {
> -    TARGET=${D}/usr/share/vm-template
> -    install -m 0755 -d ${TARGET}
> -    install -m 0740 ${WORKDIR}/vm-template.ovf.tmpl \
> -        ${TARGET}/vm-template.ovf.tmpl
> -}