Re: [Buildroot] [PATCH] package/elf2flt: handle more relocations

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello,

On Mon, 7 Aug 2023 09:50:20 +0200
Waldemar Brodkorb <wbx@openadk.org> wrote:

> Fixes following autobuild failure:
> http://autobuild.buildroot.net/results/c16/c161f191d85f9d064626ee6fcfebea61d916e434/
> 
> Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>
> ---
>  ...e-relocations-required-to-be-handled.patch | 39 +++++++++++++++++++
>  1 file changed, 39 insertions(+)
>  create mode 100644 package/elf2flt/0008-riscv64-add-more-relocations-required-to-be-handled.patch
> 
> diff --git a/package/elf2flt/0008-riscv64-add-more-relocations-required-to-be-handled.patch b/package/elf2flt/0008-riscv64-add-more-relocations-required-to-be-handled.patch
> new file mode 100644
> index 0000000000..c619a1d068
> --- /dev/null
> +++ b/package/elf2flt/0008-riscv64-add-more-relocations-required-to-be-handled.patch
> @@ -0,0 +1,39 @@
> +From 5acfed0012f2fff9801b25403bd8a5e1c2ccfea2 Mon Sep 17 00:00:00 2001
> +From: Waldemar Brodkorb <wbx@openadk.org>
> +Date: Mon, 7 Aug 2023 09:30:34 +0200
> +Subject: [PATCH 8/8] riscv64: add more relocations required to be handled
> +
> +Signed-off-by: Waldemar Brodkorb <wbx@openadk.org>

Thanks for the patch. However, it will raise at least two warnings from
"make check-package":

(1) Because it has 8/8, but we don't want such numbering in Buildroot patches

(2) Because it lacks an Upstream: tag

Could you address those details? Especially the Upstream: tag means we
expect this patch to have been submitted upstream.

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot