[OE-core] [PATCH] systemd: fix /var/log/journal ownership

Khem Raj raj.khem at gmail.com
Thu Mar 12 07:21:55 UTC 2015


> On Mar 12, 2015, at 12:14 AM, ChenQi <Qi.Chen at windriver.com> wrote:
> 
> On 03/10/2015 11:25 AM, Jonathan Liu wrote:
>> The ownership needs to be explicitly set otherwise it inherits the user
>> and group id of the build user.
>> 
>> Signed-off-by: Jonathan Liu <net147 at gmail.com>
>> ---
>>  meta/recipes-core/systemd/systemd_219.bb | 2 ++
>>  1 file changed, 2 insertions(+)
>> 
>> diff --git a/meta/recipes-core/systemd/systemd_219.bb b/meta/recipes-core/systemd/systemd_219.bb
>> index 24486f1..6a4f4e8 100644
>> --- a/meta/recipes-core/systemd/systemd_219.bb
>> +++ b/meta/recipes-core/systemd/systemd_219.bb
>> @@ -147,6 +147,8 @@ do_install() {
>>  		sed -i s%@UDEVD@%${rootlibexecdir}/systemd/systemd-udevd% ${D}${sysconfdir}/init.d/systemd-udevd
>>  	fi
>>  +	chown root:root ${D}/${localstatedir}/log/journal
>> +
>>          # Delete journal README, as log can be symlinked inside volatile.
>>          rm -f ${D}/${localstatedir}/log/README
>>  
> 
> I think it should be root:systemd-journal.
> What do you think?
> 
> And this patch would also fix the following bug.
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=7293

/var/log is symlinked (its not a copy-bind) into /var/volatile/log and /var/volatile  is mounted as tmpfs on runtime as of now
so how is this patch helping out with anything ? 

> 
> Regards,
> Chen Qi
> 
> -- 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core




More information about the Openembedded-core mailing list