public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
From: "Roberto A. Foglietta" <roberto.foglietta@gmail.com>
To: Anton Mikanovich <amikan@ilbers.de>
Cc: isar-users@googlegroups.com, Baurzhan Ismagulov <ibr@ilbers.de>
Subject: Re: [PATCH 1/1] fixed 2 typos in RECIPE-API-CHANGELOG.md (due to 87d31ee)
Date: Tue, 6 Dec 2022 16:16:09 +0100	[thread overview]
Message-ID: <CAJGKYO6T5Z+FcuKBEQ_fu+5YN+-VmkL5pSO5tVPT3k4-dNf-Og@mail.gmail.com> (raw)
In-Reply-To: <29814741-e465-b10f-d80c-0e42309bc7c5@ilbers.de>

[-- Attachment #1: Type: text/plain, Size: 1193 bytes --]

On Tue, 6 Dec 2022 at 14:30, Anton Mikanovich <amikan@ilbers.de> wrote:

>
> This changes are ok, but the patch itself is formatted quite badly.
>

the patch has been formatted by git format-patch

> There is no need in mention fixed hash (87d31ee) in commit title.
> On the other hard, mentioning of fixed hash in commit message should be
> informative to note how it is connected to the current patch, like:
>
> Fixes: 87d31ee ("add interface to passthrough env vars to sbuild")

ok, resending in attachment

>
> And once again: please submit patchsets with 'git send-mail'. This will
> reduce
> manual work during review and applying, and also will allow Patchwork to
> generate correct applicable records.
>

 I sent the patch with elastic mail and probably it went into the spam
folder. As explained in this list in the past, Google does not allow
anymore to use their smtp via git. Thus, I am using
smtp.elasticmail.com but usually the e-mail that I send via that smtp
do not arrive in the list but they end up in moderation. So, as far as
possible, I did. Please check the following thread about:

 https://groups.google.com/g/isar-users/c/xC6-QImeLzc/m/1HATZVvVAgAJ

 Best regards, R-

[-- Attachment #2: 0001-Fixed-2-typos-in-RECIPE-API-CHANGELOG.md.patch --]
[-- Type: text/x-patch, Size: 1654 bytes --]

From ee4b3bb4ddb9825ec8aad30ec4440d4ca5b79db5 Mon Sep 17 00:00:00 2001
From: "Roberto A. Foglietta" <roberto.foglietta@gmail.com>
Date: Tue, 6 Dec 2022 12:53:46 +0100
Subject: [PATCH] Fixed 2 typos in RECIPE-API-CHANGELOG.md

bugfix, typos, documentation

Fixes: 87d31ee ("add interface to passthrough env vars to sbuild")

Signed-off-by: Roberto A. Foglietta <roberto.foglietta@gmail.com>
---
 RECIPE-API-CHANGELOG.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/RECIPE-API-CHANGELOG.md b/RECIPE-API-CHANGELOG.md
index 1b8efa5..0bb8338 100644
--- a/RECIPE-API-CHANGELOG.md
+++ b/RECIPE-API-CHANGELOG.md
@@ -436,8 +436,8 @@ specific bitbake variables as environment variables into the sbuild environment.
 The motivation behind it is to allow the use of external mirrors for programming languages with builtin
 package managers (like rust and go). By that, the variables are also excluded from the bitbake signatures.
 This helps in areas where default mirrors can either not be reached or provide only little throughput.
-Please note, the forwarded variables do not have to exist. While they are not forwared in case they do not
+Please note, the forwarded variables do not have to exist. While they are not forwarded in case they do not
 exist, empty variables are forwarded.
 
-**Note about reproducability**: the forwarded variables must not have any influence on the generated package.
+**Note about reproducibility**: the forwarded variables must not have any influence on the generated package.
 This mechanism must also not be used to inject build configurations. For these cases, templates should be used.
-- 
2.34.1


  reply	other threads:[~2022-12-06 15:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-06 12:24 Roberto A. Foglietta
2022-12-06 13:30 ` Anton Mikanovich
2022-12-06 15:16   ` Roberto A. Foglietta [this message]
2022-12-06 19:56     ` Anton Mikanovich
2022-12-06 20:52       ` Roberto A. Foglietta
2022-12-07  8:32         ` Anton Mikanovich

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=CAJGKYO6T5Z+FcuKBEQ_fu+5YN+-VmkL5pSO5tVPT3k4-dNf-Og@mail.gmail.com \
    --to=roberto.foglietta@gmail.com \
    --cc=amikan@ilbers.de \
    --cc=ibr@ilbers.de \
    --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