Re: [PATCH] initscripts: RDEPENDS sed to remove "sed: command not found".

[ChenQi <Qi.Chen@windriver.com>]

On 03/26/2013 04:20 PM, anders.roxell@gmail.com wrote:
> From: Anders Roxell <anders.roxell@enea.com>
>
> Log:
> /etc/rcS.d/S37populate-volatile.sh: line 13: sed: command not found
> /etc/rcS.d/S37populate-volatile.sh: line 152: sed: command not found
> /etc/rcS.d/S37populate-volatile.sh: line 152: sed: command not found
>
> Signed-off-by: Anders Roxell <anders.roxell@enea.com>
> ---
>   meta/recipes-core/initscripts/initscripts_1.0.bb |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb b/meta/recipes-core/initscripts/initscripts_1.0.bb
> index 649e182..bfd3d28 100644
> --- a/meta/recipes-core/initscripts/initscripts_1.0.bb
> +++ b/meta/recipes-core/initscripts/initscripts_1.0.bb
> @@ -3,7 +3,7 @@ DESCRIPTION = "Initscripts provide the basic system startup initialization scrip
>   SECTION = "base"
>   LICENSE = "GPLv2"
>   LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
> -PR = "r140"
> +PR = "r141"
>   
>   INHIBIT_DEFAULT_DEPS = "1"
>   
> @@ -39,6 +39,7 @@ KERNEL_VERSION = ""
>   
>   inherit update-alternatives
>   DEPENDS_append = " update-rc.d-native"
> +RDEPENDS_${PN} = " sed"
>   
>   ALTERNATIVE_PRIORITY = "90"
>   ALTERNATIVE_${PN} = "functions"

busybox also provides sed if CONFIG_SED is enabled.
I'm wondering whether it's necessary to insert dependency here.
If so, then I think other packages which provide init scripts may also 
rdepend on 'busybox' or 'coreutils' which provide basic commands like 
'chown' or 'sed'.

Best Regards,
Chen Qi