[Buildroot] [PATCH 4/7] package/libedit2: new package
arnout at mind.be
Wed Feb 6 19:40:59 UTC 2013
On 06/02/13 20:00, Yann E. MORIN wrote:
> Arnout, All,
> On Monday 04 February 2013 Arnout Vandecappelle wrote:
>> On 04/02/13 17:01, Yann E. MORIN wrote:
>>> +define LIBEDIT2_FIX_VIS_H_INCLUDE
>>> + $(SED) 's/^#include <vis.h>$$/#include <bsd\/vis.h>/;' $(@D)/libedit/filecomplete.c
>> If you use s,foo,bar, instead of s/foo/bar/ you don't need to quote /.
> Granted. I usualy do not do that, as I sometime have paths that do contain
> commas, and would broke anyway, so I always explicitly quote '/'.
@ is a good alternative. But feel free to keep the quoting if you prefer.
>> Also, the ; is redundant.
> I like it that all my sed expressions are properly terminated, even if
> it is not mandatory. It is very annoying to me that, when there are two
> or more expressions, they are all terminated, save for the last one...
OK, I thought it was a mistake.
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot