[linux-lvm] lvm2 portability issues (GNU sed is assumed)

[John Spencer]

On 01/31/2013 04:31 AM, John Spencer wrote:
> 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
>> between.)
>
> 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.


zdenek, since alasdair does not seem to react, would you mind applying 
this patch for a quick test, see that it generates both linker scripts 
(.export.sym) correctly (exactly the same output than before) and merge 
it ? thanks!

(reattaching the patch)

thanks,
--JS
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-make.tmpl.in-fix-buggy-sed-statement-that-worked-by-.patch
Type: text/x-patch
Size: 985 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/linux-lvm/attachments/20130224/97ad309c/attachment.bin>