[prev in list] [next in list] [prev in thread] [next in thread]
List: openembedded-devel
Subject: [oe] [meta-oe][PATCH] rng-tools: Fixes for latest OE layouts
From: martin.jansa () gmail ! com (Martin Jansa)
Date: 2013-02-25 19:26:29
Message-ID: 20130225192629.GD3239 () jama
[Download RAW message or body]
On Thu, Jan 10, 2013 at 12:27:54PM +0100, Martin Jansa wrote:
> On Wed, Jan 09, 2013 at 07:34:05AM -0700, Gary Thomas wrote:
> > On 2013-01-09 07:21, Martin Jansa wrote:
> > > On Sun, Dec 16, 2012 at 07:00:22PM +0100, Martin Jansa wrote:
> > > > On Fri, Dec 14, 2012 at 02:05:48PM -0700, Gary Thomas wrote:
> > > > > The current recipe has a few installation errors:
> > > > > * The manipulation of configurable paths drops a '/'
> > > > > * The init script defaults are placed in the wrong file
> > > > > * Finer control over init script start/stop
> > > > > * The default device is non-standard
> > > > >
> > > > > This patch fixes these little nits and makes the package usable again.
> > > > >
> > > > > Signed-off-by: Gary Thomas <gary at mlbassoc.com>
> > > > > ---
> > > > > meta-oe/recipes-support/rng-tools/files/default | 3 ++-
> > > > > meta-oe/recipes-support/rng-tools/rng-tools_2.bb | 5 +++--
> > > > > 2 files changed, 5 insertions(+), 3 deletions(-)
> > > > >
> > > > > diff --git a/meta-oe/recipes-support/rng-tools/files/default \
> > > > > b/meta-oe/recipes-support/rng-tools/files/default index ab7cd93..7aede9b \
> > > > > 100644
> > > > > --- a/meta-oe/recipes-support/rng-tools/files/default
> > > > > +++ b/meta-oe/recipes-support/rng-tools/files/default
> > > > > @@ -1,2 +1,3 @@
> > > > > # Specify rng device
> > > > > -RNG_DEVICE=/dev/hwrng
> > > > > +#RNG_DEVICE=/dev/hwrng
> > > > > +RNG_DEVICE=/dev/urandom
> > > > > diff --git a/meta-oe/recipes-support/rng-tools/rng-tools_2.bb \
> > > > > b/meta-oe/recipes-support/rng-tools/rng-tools_2.bb index ef753cf..b94854a \
> > > > > 100644
> > > > > --- a/meta-oe/recipes-support/rng-tools/rng-tools_2.bb
> > > > > +++ b/meta-oe/recipes-support/rng-tools/rng-tools_2.bb
> > > > > @@ -15,12 +15,13 @@ inherit autotools update-rc.d
> > > > > do_install_append() {
> > > > > install -d "${D}${sysconfdir}/init.d"
> > > > > install -m 0755 ${WORKDIR}/init ${D}${sysconfdir}/init.d/rng-tools
> > > > > - sed -i -e 's,/etc/,${sysconfdir}/,' -e 's,/usr/sbin/,${sbindir},' \
> > > > > \ + sed -i -e 's,/etc/,${sysconfdir}/,' -e \
> > > > > 's,/usr/sbin/,${sbindir}/,' \ ${D}${sysconfdir}/init.d/rng-tools
> > > > >
> > > > > install -d "${D}${sysconfdir}/default"
> > > > > - install -m 0644 ${WORKDIR}/default ${D}${sysconfdir}/default
> > > > > + install -m 0644 ${WORKDIR}/default ${D}${sysconfdir}/rng-tools
> > > > > }
> > > > >
> > > > > INITSCRIPT_NAME = "rng-tools"
> > > > > INITSCRIPT_PARAMS = "defaults"
> > > > > +INITSCRIPT_PARAMS = "start 30 S . stop 30 0 6 1 ."
> > > >
> > > > Why do you add 2nd INITSCRIPT_PARAMS ?
> > >
> > > ping :)
> >
> > Sorry, must have missed your query (holidays).
> >
> > That second line should really replace the first for finer grain
> > control over the start/stop choice. I can fix this and resend the patch.
>
> Yes, please do if you want it applied.
Last call for update. Next week I'll remove it from my queue and mark it
as "Changes requested" on patchwork when not updated.
>
> Cheers,
> --
> Martin 'JaMa' Jansa jabber: Martin.Jansa at gmail.com
--
Martin 'JaMa' Jansa jabber: Martin.Jansa at gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://layers.openembedded.org/pipermail/openembedded-devel/attachments/20130225/be268844/attachment.sig>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic