Re: [OE-core] [PATCH v3] xmlto: backport a patch to fix build with gcc-14 on host

[Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>]

On Mon, Nov 04, 2024 at 03:59:21PM +0100, Martin Jansa wrote:
> * need to add dependency on flex-native because now when the
>   .l file is modified by the .patch file it will try to regenerate
>   the c code and fail:
> 
> | make[1]: Entering directory 'work/x86_64-linux/xmlto-native/0.0.28-r0/build'
> | /bin/bash ../xmlto-0.0.28/ylwrap ../xmlto-0.0.28/xmlif/xmlif.l .c xmlif/xmlif.c -- /bin/bash 'work/x86_64-linux/xmlto-native/0.0.28-r0/xmlto-0.0.28/missing' flex
> | work/x86_64-linux/xmlto-native/0.0.28-r0/xmlto-0.0.28/missing: line 81: flex: command not found
> | WARNING: 'flex' is missing on your system.
> |          You should only need it if you modified a '.l' file.
> |          You may want to install the Fast Lexical Analyzer package:
> |          <https://github.com/westes/flex>
> 
> * backport
>   https://pagure.io/xmlto/c/32376c053733c6c0ebaca3c25c0725509342fdf3?branch=master
>   as well, so that patched xmlif/xmlif.c is newer than xmlif/xmlif.l and the build
>   won't try to regenerate it with flex as that leads to random build failures reported
>   in:
>   https://lists.openembedded.org/g/openembedded-core/message/206412
>   https://errors.yoctoproject.org/Errors/Details/810853/
>   https://lists.openembedded.org/g/openembedded-core/message/206496
>   https://valkyrie.yoctoproject.org/#/builders/29/builds/355
> 
> Signed-off-by: Martin Jansa <martin.jansa@gmail.com>
> ---
> v3: add touch ${S}/xmlif/xmlif.c
>     to make sure xmlif.c is newer than xmlif.l after do_patch (order of
>     .patch files in SRC_URI isn't enough) to prevent regenerating it
>     with flex-native which isn't in DEPENDS
> 

Hi Martin,

This is still for Kirkstone branch, right ?

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com