[oe] [meta-oe][PATCH] rng-tools: Fixes for latest OE layouts

Martin Jansa martin.jansa at gmail.com
Thu Jan 10 11:27:54 UTC 2013


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.

Cheers,
-- 
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://lists.openembedded.org/pipermail/openembedded-devel/attachments/20130110/74f17e75/attachment-0002.sig>


More information about the Openembedded-devel mailing list