> -----Original Message----- > From: Marek Vasut <ma...@denx.de> > Sent: den 18 mars 2024 23:00 > To: Christophe Chapuis <chris.chap...@gmail.com> > Cc: Peter Kjellerstedt <peter.kjellerst...@axis.com>; openembedded- > de...@lists.openembedded.org; Khem Raj <raj.k...@gmail.com>; Fabio Estevam > <feste...@denx.de> > 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: openembedded-devel+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-