> -----Original Message----- > From: Marek Vasut <[email protected]> > Sent: den 18 mars 2024 23:00 > To: Christophe Chapuis <[email protected]> > Cc: Peter Kjellerstedt <[email protected]>; openembedded- > [email protected]; Khem Raj <[email protected]>; Fabio Estevam > <[email protected]> > Subject: Re: [oe] [meta-oe][PATCH v4 7/7] lvgl: lv-conf.inc: generalize > sed instructions > > On 3/18/24 4:42 PM, Christophe Chapuis wrote: > > I avoided \s because it's not well known, and I would also prefer using > > [[:space:]] as it is POSIX compliant.
See chapter "5.6 regular expression extensions" in the sed info page for \s and a number of other similar sequences (for anyone used to Perl's regular expressions, \s is the normal way to match whitespace). And true, this is not POSIX, but neither is the -i option that is also used by the same command. > > > > I can propose a patch for this, taking Peter's remarks into account. Thank you. > > Yes please, already reviewed too. > > btw please stop top-posting . //Peter
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#109505): https://lists.openembedded.org/g/openembedded-devel/message/109505 Mute This Topic: https://lists.openembedded.org/mt/104964304/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
