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

"Mathieu Dubois-Briand" <[email protected]>
Newsgroups org.openembedded.lists.bitbake-devel
Message-ID <[email protected]>
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.