Re: [bitbake-devel] [PATCH v2] parse: warn on trailing whitespace in parsed lines

Jaipaul Cheernam <[email protected]>
Newsgroups org.openembedded.lists.bitbake-devel
Message-ID <DU7PPF66507B2D7DF3AD4504715A950243CF3DB2@DU7PPF66507B2D7.EURP189.PROD.OUTLOOK.COM>
Hi Mathieu,

Yes , if we agree on solution, I will send patch series to oe-core and metaopenembedded as well .

I will prepare and send them tomorrow .


Regards,
Jaipaul

________________________________
From: Mathieu Dubois-Briand <[email protected]>
Sent: Thursday, August 13, 2026 5:07 PM
To: Jaipaul Cheernam <[email protected]>; [email protected] <[email protected]>
Subject: Re: [bitbake-devel] [PATCH v2] parse: warn on trailing whitespace in parsed lines

On Thu Aug 6, 2026 at 1:24 PM CEST, Jaipaul Cheernam wrote:
> rstrip() silently strips trailing whitespace before processing. This
> hides invisible spaces or tabs that end up in patch context and cause
> patches to fail to apply across branches.
>
> Warn on any line with trailing whitespace so developers can fix it early.
>
> Signed-off-by: Jaipaul Cheernam <[email protected]>
> ---

Hi Jaipaul,

Thanks for the v2. I forgot to ask: do you plan to provide an oe-core
patch fixing all whitespace issues? As so far, this is adding a lot of
warnings during the build:

https://autobuilder.yoctoproject.org/valkyrie/#/builders/4/builds/4405

Thanks,
Mathieu

--
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.