Re: [meta-webserver][PATCH] Add nginx http webserver

[Paul Eggleton <paul.eggleton@linux.intel.com>]

On Monday 02 December 2013 15:56:34 Sebastien MENNETRIER wrote:
> 2013/12/2 Koen Kooi <koen@dominion.thruhere.net>
> > Sébastien Mennetrier schreef op 02-12-13 15:11:
> > > +    install -d ${D}${sysconfdir}/init.d +    cat ${WORKDIR}/init | \ +
> > > sed -e 's,/usr/sbin/,${sbindir}/,g' \ +            -e
> > > 's,/etc/,${sysconfdir}/,g' > ${D}/${sysconfdir}/init.d/nginx +    chmod
> > > 755 ${D}/${sysconfdir}/init.d/nginx
> > 
> > I think using sed -i will be easier to understand, I haven't seen any pipe
> > based modifications in an OE recipe yet.
> > 
 > This method is used in apache2 recipe... But I can modify if it's
> necessary.

Right, it's probably not ideal there either (and it was probably me that wrote 
that...) When you do change this for the nginx recipe, be sure to sed -i the 
destination file after copying (rather than the source before) so that re-
executing do_install will work.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre