[OE-core] [PATCH 6/6] systemd: remove unnecessary code

[Maciek Borzecki]

On wto, 2016-02-09 at 07:19 -0800, Khem Raj wrote:
> On Mon, Feb 8, 2016 at 2:09 AM, Maciej Borzecki
> <maciej.borzecki at open-rnd.pl> wrote:
> > Remove unnecessary chown on journal file.
> > 
> > Signed-off-by: Maciej Borzecki <maciej.borzecki at open-rnd.pl>
> > ---
> >  meta/recipes-core/systemd/systemd_228.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/meta/recipes-core/systemd/systemd_228.bb b/meta/recipes-
> > core/systemd/systemd_228.bb
> > index
> > 40ed81a37e78c67bae13f7ee7018d85efa2f6b7d..03434ff9d5ff19fe043332b0b53a4fa96ab2b120
> > 100644
> > --- a/meta/recipes-core/systemd/systemd_228.bb
> > +++ b/meta/recipes-core/systemd/systemd_228.bb
> > @@ -209,7 +209,7 @@ do_install() {
> >                 sed -i s%@UDEVD@%${rootlibexecdir}/systemd/systemd-udevd%
> > ${D}${sysconfdir}/init.d/systemd-udevd
> >         fi
> > 
> > -       chown root:systemd-journal ${D}/${localstatedir}/log/journal
> > +        # chown root:systemd-journal ${D}/${localstatedir}/log/journal
> > 
> 
> journald requires these perms on this directory. Can you describe the
> issue it fixes ?
> 

I should use non-root user a bit more often. Apparently journalctl becomes useless without
systemd-journal chown on /var/log/journal. I'll revise this and the other useradd/sysusers
patch. Thanks for catching this.

 
-- 
Maciej Borzęcki
Open-RnD Sp. z o.o., Poland
www.open-rnd.pl