<div dir="ltr">Ah, the NEWS file has a mistake - it says libinput instead of libaudit. </div> <div class="gmail_quote"><div dir="ltr">On Tue, 6 Oct 2015 at 19:34 Joakim Tjernlund <<a href="mailto:joakim.tjernlund@transmode.se">joakim.tjernlund@transmode.se</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, 2015-10-05 at 20:03 +0000, Robert Ancell wrote: > Thanks for making the patch, however this has already been fixed in LightDM 1.6.3. Thanks for telling me but you could have mentioned this in the release notes. > > On Tue, 6 Oct 2015 at 09:02 Joakim Tjernlund <<a href="mailto:joakim.tjernlund@transmode.se" target="_blank">joakim.tjernlund@transmode.se</a>> wrote: > > A few places are missing #if HAVE_LIBAUDIT wrapped around them. > > This patch fixes the build. > > > > --- src/<a href="http://session-child.c.org" rel="noreferrer" target="_blank">session-child.c.org</a> 2015-10-01 20:51:35.830722781 +0200 > > +++ src/session-child.c 2015-10-01 20:54:09.539968600 +0200 > > @@ -416,8 +416,9 @@ > > ut.ut_tv.tv_usec = tv.tv_usec; > > > > updwtmpx ("/var/log/btmp", &ut); > > - > > +#if HAVE_LIBAUDIT > > audit_event (AUDIT_USER_LOGIN, username, -1, remote_host_name, tty, FALSE); > > +#endif > > } > > > > /* Check account is valid */ > > @@ -733,8 +734,9 @@ > > g_printerr ("Failed to write utmpx: %s\n", strerror (errno)); > > endutxent (); > > updwtmpx ("/var/log/wtmp", &ut); > > - > > +#if HAVE_LIBAUDIT > > audit_event (AUDIT_USER_LOGIN, username, uid, remote_host_name, tty, TRUE); > > +#endif > > } > > > > waitpid (child_pid, &return_code, 0); > > @@ -771,8 +773,9 @@ > > g_printerr ("Failed to write utmpx: %s\n", strerror (errno)); > > endutxent (); > > updwtmpx ("/var/log/wtmp", &ut); > > - > > +#if HAVE_LIBAUDIT > > audit_event (AUDIT_USER_LOGOUT, username, uid, remote_host_name, tty, TRUE); > > +#endif > > } > > } > > > > _______________________________________________ > > LightDM mailing list > > <a href="mailto:LightDM@lists.freedesktop.org" target="_blank">LightDM@lists.freedesktop.org</a> > > <a href="http://lists.freedesktop.org/mailman/listinfo/lightdm" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/lightdm</a> > > </blockquote></div>