Re: [PATCH V1] initscripts/urandom: create directory before staging into it

[Ming Liu <ming.liu@windriver.com>]

[-- Attachment #1: Type: text/plain, Size: 1631 bytes --]

On 12/06/2012 03:30 AM, Saul Wold wrote:
> On 12/04/2012 11:57 PM, Ming Liu wrote:
>> /etc/init.d/urandom fails to start/stop because it tries to save
>> random-seed into /var/lib/urandom folder which does not exist in the
>> file system.
>>
>> Fixed by creating /var/lib/urandom at do_install time.
>>
>> Signed-off-by: Ming Liu <ming.liu@windriver.com>
>> ---
>>   meta/recipes-core/initscripts/initscripts_1.0.bb | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb 
>> b/meta/recipes-core/initscripts/initscripts_1.0.bb
>> index d25838b..39be9a8 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 = "r137"
>> +PR = "r138"
>>
>>   INHIBIT_DEFAULT_DEPS = "1"
>>
>> @@ -133,4 +133,6 @@ do_install () {
>>       fi
>>
>>       install -m 0755        ${WORKDIR}/device_table.txt 
>> ${D}${sysconfdir}/device_table
>> +    # Holds state information pertaining to urandom
>> +    install -d ${D}/var/lib/urandom
>
> I thought this would go in volatiles, not here.
Yes, it's a alternative solution or maybe better.
I was going to put it into volatiles, but considering it's a urandom 
specific directory, so I eventually put it here.
OK, I will send the V2 as you suggest.
>
> Sau!
>
>>   }
>>
>
>


[-- Attachment #2: Type: text/html, Size: 4277 bytes --]