[linux-lvm] lvm2 portability issues (GNU sed is assumed)
maillist-lvm at barfooze.de
Thu Jan 31 03:31:35 UTC 2013
On 01/30/2013 03:55 AM, Alasdair G Kergon wrote:
> On Tue, Jan 29, 2013 at 08:51:21PM +0100, John Spencer wrote:
>> may i ask kindly to reformulate that statement so that it works with
>> posix sed (as used in busybox 1.20.2) ?
> I'm perfectly happy using modern extensions to these tools:)
> If you want this to work with alternative versions, please send a tested
> patch for us to incorporate. (Either modifications to the commands so
> it works with both or else alternative versions which configure chooses
a couple of regex experts looked at it, and the reason it didnt work is
simply a bug: the \ before d is wrong. apparently the author meant to
escape the makefile variable that follows, but that is not necessary and
so the backslash gets passed on.
find attached a patch that works correctly with gnu sed and busybox sed.
btw: apparently there are 2 other bugs in that sed statement:
| should become \| to work as intended by the author (gnu sed only), and
[ \*] should become [ *].
but apparently this part of the regex isn't used anyway, so it might be
the best to just remove it.
i left that part unfixed, as it is not crucial, so someone else might
want to look at it.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 985 bytes
Desc: not available
More information about the linux-lvm