Hello,
On 16/08/2023 12:50:30-0700, Joe Slater via lists.openembedded.org wrote:
> From: Joe Slater <joe.sla...@windriver.com>
>
> Depending on the version of glibc, localtime_r() must
> be preceded by a call to tzset() or it will ignore any
> value of TZ in the environment. This problem will only
> be seen when building file-native on outdated hosts.
>
> Signed-off-by: Joe Slater <joe.sla...@windriver.com>
> ---
> meta/recipes-devtools/file/file_5.45.bb | 4 ++-
> .../recipes-devtools/file/files/print_c.patch | 32 +++++++++++++++++++
> 2 files changed, 35 insertions(+), 1 deletion(-)
> create mode 100644 meta/recipes-devtools/file/files/print_c.patch
>
This causes:
ERROR: file-native-5.45-r0 do_patch: QA Issue: Fuzz detected:
Applying patch print_c.patch
patching file src/print.c
Hunk #1 succeeded at 289 with fuzz 1 (offset 3 lines).
The context lines in the patches can be updated with devtool:
devtool modify file-native
devtool finish --force-patch-refresh file-native <layer_path>
Don't forget to review changes done by devtool!
> diff --git a/meta/recipes-devtools/file/file_5.45.bb
> b/meta/recipes-devtools/file/file_5.45.bb
> index a7127023cb..8477668658 100644
> --- a/meta/recipes-devtools/file/file_5.45.bb
> +++ b/meta/recipes-devtools/file/file_5.45.bb
> @@ -11,7 +11,9 @@ LIC_FILES_CHKSUM =
> "file://COPYING;beginline=2;md5=0251eaec1188b20d9a72c502ecfdd
> DEPENDS = "file-replacement-native"
> DEPENDS:class-native = "bzip2-replacement-native"
>
> -SRC_URI = "git://github.com/file/file.git;branch=master;protocol=https"
> +SRC_URI = "git://github.com/file/file.git;branch=master;protocol=https \
> + file://print_c.patch \
> + "
>
> SRCREV = "4cbd5c8f0851201d203755b76cb66ba991ffd8be"
> S = "${WORKDIR}/git"
> diff --git a/meta/recipes-devtools/file/files/print_c.patch
> b/meta/recipes-devtools/file/files/print_c.patch
> new file mode 100644
> index 0000000000..4526655d22
> --- /dev/null
> +++ b/meta/recipes-devtools/file/files/print_c.patch
> @@ -0,0 +1,32 @@
> +From e329257b8e22362b62e6c930447ef6feadd63f32 Mon Sep 17 00:00:00 2001
> +From: Joe Slater <joe.sla...@windriver.com>
> +Date: Mon, 7 Aug 2023 22:37:19 +0000
> +Subject: [PATCH] print.c: initialize timezone data for localtime_r()
> +
> +The man page for localtime() points out that while it acts
> +like tzset() has been called, localtime_r() might not. We
> +have a local version of localtime_r() that avoids this, but
> +we do not compile it.
> +
> +Upstream-Status: Submitted [f...@astron.com]
> +
> +Signed-off-by: Joe Slater <joe.sla...@windriver.com>
> +---
> + src/print.c | 1 +
> + 1 file changed, 1 insertion(+)
> +
> +diff --git a/src/print.c b/src/print.c
> +index 18f42a05..b1de4908 100644
> +--- a/src/print.c
> ++++ b/src/print.c
> +@@ -286,6 +286,7 @@ file_fmtdatetime(char *buf, size_t bsize, uint64_t v,
> int flags)
> + }
> +
> + if (flags & FILE_T_LOCAL) {
> ++ tzset();
> + tm = localtime_r(&t, &tmz);
> + } else {
> + tm = gmtime_r(&t, &tmz);
> +--
> +2.35.5
> +
> --
> 2.39.1
>
>
>
>
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#186366):
https://lists.openembedded.org/g/openembedded-core/message/186366
Mute This Topic: https://lists.openembedded.org/mt/100786945/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
