[Buildroot] [PATCH 1/1] package/stm32flash: bump to version 0.5
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Mon Feb 18 21:58:58 UTC 2019
Hello Markus,
Thanks for your contribution! I have applied your patch to our next
branch, but I have a few comments, see below.
On Mon, 18 Feb 2019 14:01:53 +0000
"Steinhilber, Markus" <Markus.Steinhilber at erbe-med.com> wrote:
> Bump to latest official version.
>
> Signed-off-by: Markus Steinhilber <markus.steinhilber at erbe-med.com>
Your From: doesn't match your Signed-off-by: line. Your From is:
LastName, FirstName <email>
While your SoB is:
FirstName LastName <email>
We have some checks that verify that both are consistent, so I fixed up
the author name of your commit. Indeed, we have a preference for the:
FirstName LastName <email>
formatting.
> diff --git a/package/stm32flash/stm32flash.hash b/package/stm32flash/stm32flash.hash
> new file mode 100644
> index 0000000000..dc97c10972
> --- /dev/null
> +++ b/package/stm32flash/stm32flash.hash
> @@ -0,0 +1,2 @@
> +# Locally computed:
> +sha256 73afcbd317ca3eb080c14b6ed1451e5a74a3bbae8f8fb5fe24ea496333f03ead
> +stm32flash-1f934ae86babdeea47afdfae1d856d5fd5da6c53.tar.gz
This line was wrapped. I suspect you haven't sent this patch using "git
send-email", and due to this, it was line-wrapped by your e-mail
client. Using "git send-email" would avoid such issues.
A possible follow-up improvement to this package would be to add the
hash of the gpl-2.0.txt license file to the .hash file.
Thanks again for your contribution!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list