public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: Uladzimir Bely <ubely@ilbers.de>
To: Chengshu Mou <chengshu.mou@siemens.com>,
	isar-users <isar-users@googlegroups.com>
Subject: Re: [PATCH 1/1] fix: unsupported value of disk format in ovf
Date: Wed, 21 Aug 2024 08:12:24 +0300	[thread overview]
Message-ID: <b35d1aaaef806e105ae9638dd343ede3bc516df1.camel@ilbers.de> (raw)
In-Reply-To: <506abd25-6c6e-45ef-8087-89dacd1b0ff0n@googlegroups.com>

On Tue, 2024-08-20 at 00:22 -0700, 'Chengshu Mou' via isar-users wrote:
> Hello guys,
>  
> I have a patch to fix the unsupported value of disk format in ovf
> file generated by Isar. Details are as follows: 
> 
> Affected part: 
> Images for VMware and VirtualBox.
>  
> Modification:
> Add ‘VMDK_SUBFORMAT’ into template vars of ovf.
>  
> Reference issue: 
> https://github.com/ilbers/isar/issues/106.
> 

Hello.

The patch resolves real issue, but it's mailformed and not applicable
as is. Please could you send v2 with "git send-email" but not as
copypaste/attachment in your email client?

> Patch: 
> From 792a013b26a474696d11a31a8bd069554b10b190 Mon Sep 17 00:00:00
> 2001
> From: "Mou Chengshu(Ben)" <chengshu.mou@siemens.com>
> Date: Tue, 20 Aug 2024 13:28:06 +0800
> Subject: [PATCH] fix: unsupported value of disk format in ovf
> 

missing "Signed-off-by"

> ---
>  meta/classes/imagetypes_vm.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes/imagetypes_vm.bbclass
> b/meta/classes/imagetypes_vm.bbclass

wrong line break here

> index 56410de9..4da42447 100644
> --- a/meta/classes/imagetypes_vm.bbclass
> +++ b/meta/classes/imagetypes_vm.bbclass
> @@ -60,7 +60,7 @@ macgen() {
>  
>  OVA_VARS = "OVA_NAME OVA_MEMORY OVA_NUMBER_OF_CPU OVA_VRAM \
>              OVA_FIRMWARE OVA_ACPI OVA_3D_ACCEL \
> -            OVA_SHA_ALG VIRTUAL_MACHINE_IMAGE_FILE"
> +            OVA_SHA_ALG VIRTUAL_MACHINE_IMAGE_FILE VMDK_SUBFORMAT"
>  
>  IMAGE_TEMPLATE_FILES:ova = "${OVF_TEMPLATE_FILE}"
>  IMAGE_TEMPLATE_VARS:ova = "${OVA_VARS}"
> -- 
> 2.39.2
> 
> Please have a look, thanks.
> -- 
> 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 view this discussion on the web visit
> https://groups.google.com/d/msgid/isar-users/506abd25-6c6e-45ef-8087-89dacd1b0ff0n%40googlegroups.com
> .

-- 
Best regards,
Uladzimir.

-- 
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 view this discussion on the web visit https://groups.google.com/d/msgid/isar-users/b35d1aaaef806e105ae9638dd343ede3bc516df1.camel%40ilbers.de.

  reply	other threads:[~2024-08-21  5:12 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-20  7:22 'Chengshu Mou' via isar-users
2024-08-21  5:12 ` Uladzimir Bely [this message]
2024-08-21  9:24   ` [PATCH v2] " Mou Chengshu(Ben)
2024-08-21 10:05     ` 'Jan Kiszka' via isar-users
2024-08-23  2:23       ` Mou Chengshu(Ben)
2024-08-23  6:13         ` 'Jan Kiszka' via isar-users
2024-08-23  7:25           ` Mou Chengshu(Ben)
2024-08-27 16:59             ` Uladzimir Bely

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b35d1aaaef806e105ae9638dd343ede3bc516df1.camel@ilbers.de \
    --to=ubely@ilbers.de \
    --cc=chengshu.mou@siemens.com \
    --cc=isar-users@googlegroups.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox