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