[Patch] let sessreg build with utmpx exclusively
Alan Coopersmith
Alan.Coopersmith at Sun.COM
Sat Jan 16 10:13:33 PST 2010
Ed Schouten wrote:
> Hi folks,
>
> We at FreeBSD very recently gained support for utmpx, while removing
> utmp entirely. Our utmpx interface is very compact and should be quite
> pleasant to use. There aren't a lot of extensions to POSIX, which is
> described here:
>
> http://www.opengroup.org/onlinepubs/9699919799/basedefs/utmpx.h.html
> http://www.opengroup.org/onlinepubs/9699919799/functions/endutxent.html
>
> It seens the sessreg app is a bit antique and really depends on utmp
> being there. I've attached a patch to make it build with FreeBSD HEAD
> again.
The utmpx support was written on Solaris, which while it dropped the utmp
& wtmp files years ago, left the headers & functions for backwards
compatibility, so we never had to support building without them.
The patch looks pretty good, and I've confirmed it still builds fine on
Solaris. I do see one small style nit - the #ifdef/#endif indentation
doesn't match here, which could be confusing:
+# ifdef HAVE_UTMPX_NAME
+ if (utmpx_file != NULL)
+#endif
Can you add a 'Signed-off-by' tag ("git commit --amend -s" should do that),
and then submit the patch as output from git format-patch? That will help
us preserve the authorship and your comments, and I'll push that.
--
-Alan Coopersmith- alan.coopersmith at sun.com
Sun Microsystems, Inc. - X Window System Engineering
More information about the xorg-devel
mailing list