[vlc-devel] [PATCH 2/3] tools: add GNU sed

[Sean McGovern]

On Tue, Mar 11, 2014 at 3:20 PM, Rafaël Carré <funman at videolan.org> wrote:

> Hello,
>
> We had several occurences of using POSIX sed, e.g. for OSX.
>
> Where do we need GNU sed?
>
> I don't know if this patch is really needed or not but if we use it
> we might want to revisit our sed usage (and use -i without suffix
> generously)
>
> On 03/11/14 20:12, Sean McGovern wrote:
> > ---
> >  extras/tools/bootstrap    |   12 ++++++++++++
> >  extras/tools/packages.mak |    3 +++
> >  extras/tools/tools.mak    |   17 +++++++++++++++++
> >  3 files changed, 32 insertions(+)
> >
>
>
As I mentioned on IRC, extras/ and contrib/ are both chock full of sed -i
(in-place editing) usage -- it's not POSIX, as witnessed here:
http://pubs.opengroup.org/onlinepubs/009695399/utilities/sed.html

-- Sean McG.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20140311/74bd4089/attachment.html>