commit 23949f9a4c30b5dfa67a88b3dd2523f09f163d91
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Sun Sep 25 00:41:25 2011 -0700

    configure.ac: Bump to 1.1.11
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 225493d3c6be64dafec69095f254a1bee6386e33
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Sep 20 17:39:34 2011 -0700

    fix whitespace issues
    
    Performed with:
      find * -type f | xargs perl -i -p -e 's{[ \t]+$}{}'
      find * -type f | xargs perl -i -p -e 's{^[ ]+\t}{\t}'
    git diff -w & git diff -b show no diffs from this change
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 8016dfd9080393c23d19b1c34e81d1f544655d5a
Author: Matěj Cepl <mcepl@redhat.com>
Date:   Mon Aug 8 21:26:47 2011 +0200

    Use explicitly /bin/sh for running of Xwilling script via su.
    
    su nobody -c <command> uses a default shell of the user, which in case
    of nobody is (at least on Linux distros) /sbin/nologin.
    
    Signed-off-by: Matěj Cepl <mcepl@redhat.com>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewd-by: Mark Kettenis <kettenis@openbsd.org>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 0782f1c24c658c06736907dd143f8e67359ac235
Author: Michał Górny <mgorny@gentoo.org>
Date:   Wed Aug 31 18:02:53 2011 +0200

    Support installing systemd unit file.
    
    Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339
    Signed-off-by: Michał Górny <mgorny@gentoo.org>
    Reviewed-by: Gaetan Nadon <memsize@videotron.ca>
    Tested-by: Gaetan Nadon <memsize@videotron.ca>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 5574b79c2d6c89f96484b890872ba1594fb6ddd2
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 21:52:20 2011 -0700

    Simplify FailedLogin code
    
    Relies on username going out of scope to discard the pointer returned by
    pam_get_item that pam_end frees at the bottom of the loop.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 29e9393697d0b7158c2c1e0856180713c7c8f925
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 21:39:16 2011 -0700

    Stop using username retrieved from PAM before pam_end frees it.
    
    The first time a failed login message was syslogged it had the right
    username, but subsequent ones kept reusing that pointer, even though
    PAM had freed it and it may have been reused and filled with something
    else, resulting in garbage user names for the later login failures.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit eca379146cf487cbd05e1698ea12c1835327d1d4
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 20:46:57 2011 -0700

    Add _X_ATTRIBUTE_PRINTF to LogVWrite declaration
    
    Suggested by gcc -Wmissing-format-attribute
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 7b8a4bb753d9da4cde60379050493d811c9b3044
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 20:45:32 2011 -0700

    Replace GCC_PRINTFLIKE with xproto's _X_ATTRIBUTE_PRINTF
    
    The existing GCC_PRINTFLIKE declaration seems to have been broken, since
    it only defined GCC_PRINTFLIKE if GCC_PRINTFLIKE was already defined.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 7843f1e00caba67025ac75c4985ddd3af7924a9c
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 20:38:29 2011 -0700

    Use C99 named initializers for XSetWindowAttributes structure
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 3c3f9cde9dda4c74178cc5428ebf8967c26a516a
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jun 2 18:33:56 2011 -0700

    Bug 13455: allowNullPasswd: true doesn't work until the second login attempt
    
    Assign allow_null_passwd and allow_root_login variables' values before call to
    pam_authenticate()
    
    https://bugs.freedesktop.org/show_bug.cgi?id=13455
    
    Based on a fix suggested by: Tim Shumakov <ars_1@mail.ru>
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 7d69ff7b9dfcd3e4410f181225f66f51089eef67
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri May 27 22:46:09 2011 -0700

    On Solaris, make sure PAM_TTY is only ever set to an actual /dev path
    
    Solaris PAM & auditing insist this is a device file that can be found in
    the filesystem, so we can't use the display name (":0") like we've
    been doing (and continue to do on other OS'es).
    
    Fixes Solaris bug 6960231:
    xdm login fails with: pam_unix_cred: cannot load ttyname with c2audit loaded
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 68517be0c7ae67989bc3cd845c8aa79ef7040618
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue May 17 19:37:15 2011 -0700

    xdm.man: address only needs 2 S's, not 3
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit bd0acffc9e69c4ca39d0881984ae587ad4385934
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 10:08:25 2011 -0700

    freeifaddrs() when done with it in DefineSelf()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 96362c12ccd08fcb50f697106622d0c0f8a6007a
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 09:39:21 2011 -0700

    Add Solaris 11 libsocket to list of libraries to search for getifaddrs()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Acked-by: Gaetan Nadon <memsize@videotron.ca>

commit 1953570d9cd7c4e57fc270e6146aa13e794be0de
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 10:52:18 2011 -0700

    Only greeter needs XPM CFLAGS & LIBS, not xdm daemon
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Julien Cristau <jcristau@debian.org>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Acked-by: Gaetan Nadon <memsize@videotron.ca>

commit 47c1b146c786f057026ea2ad89b8cb4c51ee3d50
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 10:44:06 2011 -0700

    Stop casting return values of malloc and friends
    
    It's not needed on modern mallocs that return void *, and can hide
    missing prototype errors that cause the compiler to assume int is
    returned (a bad thing to assume in 64-bit builds).
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Julien Cristau <jcristau@debian.org>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

commit f29b73ba549fbdcbe1361c016fc40cc424c32e9b
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 10:33:50 2011 -0700

    Stop casting free() arguments to (char *)
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Julien Cristau <jcristau@debian.org>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

commit 6cb6fcd3a95bc125615d4a6736712bcc45ad7685
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 10:24:06 2011 -0700

    Stop checking for NULL before calling free()
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Julien Cristau <jcristau@debian.org>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

commit d39dbde390f0f609c03dbd79f0bcc6a3486ca8a3
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat May 14 09:29:30 2011 -0700

    Fix potential memory leak in allocation failure cleanup path
    
    Error: Memory leak (CWE 401)
       Memory leak of pointer 'newargv' allocated with realloc(((i8*)argv), ((i * 4) + 8))
            at line 239 of /export/alanc/X.Org/git/app/xdm/xdm/util.c in function 'parseArgs'.
              'newargv' allocated at line 231 with realloc(((i8*)argv), ((i * 4) + 8)).
              newargv leaks when newargv != 0 at line 234
                  and save == 0 at line 234.
    
    [This bug was found by the Parfait 0.3.7 bug checking tool.
     For more information see http://labs.oracle.com/projects/parfait/ ]
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Julien Cristau <jcristau@debian.org>
    Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 4b9cbfd3f4d16000120c3241503c7a6102ae8afe
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Mar 4 20:31:01 2011 -0500

    man: replace hard coded section number (1) with __appmansuffix__
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit f394b7e3c7d0437a4415fe99965b866e35066713
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Mon Feb 21 16:52:19 2011 -0800

    greeter: Make sure cursor stays in bounds too
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 7d5c2364cc6c2b7338b4df9f1be0b621ca9fd2e7
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Mon Feb 21 16:43:41 2011 -0800

    greeter: Replace several #ifdef XPM blocks with LOGO_W macro
    
    Refactor code assuming compiler is smart enough to optimize out any
    that reduce to + 0 or - 0 if building without XPM defined.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit f59d8f32c755b8747acf71aa3ab4c4809bff4ebf
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Mon Feb 21 15:54:00 2011 -0800

    greeter: Make sure text input areas stay inside the lines
    
    Regression introduced by f8585c60831a8e5ddebce18bdd7e78d217a822c5
    broke bounds checking for text displayed in input fields.
    
    Since this makes each TEXT_WIDTH value only be used once, just use
    them directly instead of storing the result in a variable.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 3297eb892017c850f25d3dc4a37095612a20a381
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Mon Feb 21 15:53:23 2011 -0800

    greeter: Add echoPasswdChar resource to set character to display
    
    Stop hardcoding it to '*'.   Allows setting a resource with no value to
    to advance the cursor without drawing text (as was previously the default).
    
    Resolves https://bugs.freedesktop.org/show_bug.cgi?id=32794
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 472dc516e7d0190d81fdedd817261ee41bf8ada6
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Feb 19 22:33:33 2011 -0800

    greeter: inline valueMask for xpm window to avoid shadowing argument
    
    The Realize function names one of its arguments valueMask, and the
    valueMask variable in the inner scope is just used as a static set
    of mask flags, so just pass them directly to the function to reduce
    confusion and clear the gcc -Wshadow warning:
    
    Login.c: In function `Realize':
    Login.c:1901: warning: declaration of 'valueMask' shadows a parameter
    Login.c:1871: warning: shadowed declaration is here
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit b0dd4a3da0001a4510bdca6726a8ce6bfe632c28
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Feb 19 22:30:06 2011 -0800

    greeter: Remove unused extern dpy that is shadowed by local variables
    
    Resolves these warnings from gcc -Wshadow:
    greet.c: In function `InitGreet':
    greet.c:244: warning: declaration of 'dpy' shadows a global declaration
    greet.c:146: warning: shadowed declaration is here
    greet.c: In function `CloseGreet':
    greet.c:326: warning: declaration of 'dpy' shadows a global declaration
    greet.c:146: warning: shadowed declaration is here
    greet.c: At top level:
    greet.c:431: warning: declaration of 'dpy' shadows a global declaration
    greet.c:146: warning: shadowed declaration is here
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

commit 5bad823955542b99292783391b3f63a049fa2efc
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Feb 19 14:12:56 2011 -0800

    Delete some unused variables
    
    auth.c: In function `DefineSelf':
    auth.c:1045: warning: unused variable `n'
    error.c: In function `Debug':
    error.c:116: warning: unused variable `buf'
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

commit 3e3a45176d232c4e3f23ed6bc0b46bf660ddf271
Author: Servaas Vandenberghe <vdb128@picaros.org>
Date:   Sat Feb 19 14:06:57 2011 -0800

    Add resource reservAttempts to allow a few successive crash-type exits
    
    https://bugs.freedesktop.org/show_bug.cgi?id=20546
    https://bugs.freedesktop.org/attachment.cgi?id=43050
    
    Signed-off-by: Servaas Vandenberghe <vdb128@picaros.org>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit d0870e66dd9d8a41c9b6de65c4a9243c0a3fdc6e
Author: Servaas Vandenberghe <vdb128@picaros.org>
Date:   Sat Feb 19 13:15:58 2011 -0800

    crash detect: use StopDisplay().
    
    This patch solves a benign bug: on server crash the display is removed
    but the server exit code is ignored.  Proposed solution: substitute
    RemoveDisplay() by StopDisplay() thus enabling the wait for child exit
    synchronism.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=20546
    https://bugs.freedesktop.org/attachment.cgi?id=43049
    
    Signed-off-by: Servaas Vandenberghe <vdb128@picaros.org>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit c4031966dcbc47b2cc85b83ad78efcc64455cf72
Author: Fernando Tarlá Cardoso Lemos <fernandotcl@gmail.com>
Date:   Sat Feb 19 17:53:08 2011 -0200

    Fix missing linking dependency on -ldl.
    
    Recent versions of GCC ship with a linker that doesn't add the deps
    of the DSOs to the linking process. This behavior is also found in
    GNU gold. This change fixes building with those linkers.
    
    Some references:
    
    http://wiki.debian.org/ToolChain/DSOLinking
    https://fedoraproject.org/wiki/UnderstandingDSOLinkChange
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Gaetan Nadon <memsize@videotron.ca>

commit 208c7df5572f6ad8a63a4ee0fb1d5fd68c809653
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Feb 18 18:05:04 2011 -0800

    xdm itself doesn't need to link with Xinerama library
    
    Xinerama calls are made in the greeter & chooser, but not core daemon
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 250c9931f45c73dda9d07c6bddc734c1445cfa5c
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Feb 18 18:00:11 2011 -0800

    config: rename XDM_PRINT to XDM_TOOLKIT
    
    Name is a leftover of checking for non-Xprint Xaw7 vs. Xprint Xaw8,
    but xdm doesn't print, just needs a toolkit to display with.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit edd8d3622528351770fd9b5dc2242b16e6f70904
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Jan 12 15:29:50 2011 -0500

    config: replace deprecated AC_HELP_STRING with AS_HELP_STRING
    
    This silences an Automake warning.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit cf45a0ceee92c2fb048ff8882cf0bea200506d11
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Nov 16 11:42:50 2010 -0500

    config: AM_PROG_CC_C_O is no longer needed
    
    All per-target flags have been removed now that each target
    sits in its own directory.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit e509059f307f81d055a1ab01f4cb3a64b3157dce
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Aug 25 20:44:37 2010 -0400

    config: move appdefault makefile to app-defaults dir.
    
    For this package it is best not to break the pattern
    of each directory having its own makefile.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 8ee6a2678f9045c31366525bcaae5baf59a20930
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Aug 25 20:19:43 2010 -0400

    config: mv xdm in its own directory
    
    Prevents the use of per target flags.
    Clearly shows which libraries and compiler options apply.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit dd8a5b9f50deb9d91bb33cdd12fe083d39cd7d23
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Aug 24 11:32:53 2010 -0400

    config: move common headers into the include directory
    
    They provide interfaces to multiple executables
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit eb0e5a3a665cd4dbd376f1a1cbd6d0c7ace22572
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 23 20:04:15 2010 -0400

    config: move xdmshell into its own directory
    
    Prevents the use of per target flags.
    Clearly shows which libraries and compiler options apply.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 2d520f71a0c89faf5522f5e37c09bdd5f933a574
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 23 14:42:23 2010 -0400

    config: move chooser into its own directory
    
    Prevents the use of per target flags.
    Clearly shows which libraries and compiler options apply.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit f3b0019752e5b683c6e273eba15a4ecc9f55c5ae
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sat Nov 13 16:17:44 2010 -0500

    config: move man pages into their own directory
    
    Replace lowercase $(xdmlibdir) with $(XDMLIBDIR)
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 1dc0ef42cf009596db60a9fd08435621f70b81e7
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Nov 12 17:07:38 2010 -0500

    config: use AM_V_GEN instead of customized macros for AM_SILENT_RULES
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 083f3d0ee8ecbb48b86dddfcf2f172fafb579f08
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Nov 23 15:15:33 2010 -0500

    config: relocate key_setnet in section "Check for header files"
    
    No configuration changes
    
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit e4535a52148d58c3fe91baebdcfe0187a07519b9
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Nov 12 16:25:53 2010 -0500

    config: relocate AC_FUNC_FORK in section "Check for system library functions"
    
    No configuration changes.
    
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 63e882f86809419f8efdf2717050c96a15345c7b
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Nov 12 16:21:54 2010 -0500

    config: comment libxtrans macros as they are order sensitive
    
    XTRANS_SECURE_RPC_FLAGS must be called after XTRANS_CONNECTION_FLAGS
    which finds libraries like -lsocket, -lnsl or -lws32_32.
    
    The found libraries are pre-pended to $LIBS which is not obvious.
    
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 73763eb718cba54963e0cf71b5316fdce2762c1f
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Oct 25 14:19:20 2010 -0400

    config: rework check for random number generator
    
    Following a similar pattern to PAM and SELinux checks.
    Unchanged: if a user has requested the use of a random device
    and none can be found, the configuration aborts.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 10117ab7bc89ca65d313e0c6675e27d336a96925
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Sep 6 15:08:36 2010 -0400

    config: integrate check for arc4random in appropriate sections
    
    The AC_SEARCH_LIBS is done in "Check for libraries" section.
    Move AC_SUBST and AM_CONDITIONAL over there as well.
    Move AC_CHECK_FUNCS for arc4random to "library functions" section
    No function change.
    
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 5d7d55646d4867e7b3f736d318575554c2414361
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Oct 22 20:39:27 2010 -0400

    config: upgrade the checking for SELinux support
    
    Use "auto" rather than "try" for consistency with other modules.
    Update help string to mention default value.
    Unchanged: if configured --with-selinux and -lselinux is misisng,
    abort the configuration.
    
    Change HAVE_SELINUX to USE_SELINUX to be consistent with
    USE_PAM, USE_XFT, USE_XPM and USE_XINERAMA.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 5718767b882dc46f9393fd6b42f443227061a807
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Nov 22 16:08:18 2010 -0500

    config: upgrade the checking for PAM support
    
    Use "auto" rather than "try" for consistency with other modules.
    Update help string to mention default value.
    Unchanged: if configured --with-pam and -lpam is misisng,
    abort the configuration.
    
    Continue with USE_PAM rather than HAVE_PAM_OPEN_SESSION
    as we are checking for a feature rather a function alone.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 529a045fa6a25334be469f633531fe1324dcf14a
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Fri Oct 22 13:51:43 2010 -0400

    config: isolating -lselinux to a single target is not required
    
    Due to previous patches, -lselinux, if installed on the configured system,
    is included in $LIBS already.
    
    Reviewed-by: Mikhail Gusarov <dottedmag@dottedmag.net>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 0b1a949b7c1ce09136d3e4098b68a733d7bb32e2
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Thu Sep 2 19:45:54 2010 -0400

    config: use AC_CHECK_FUNCS for mkstemp, getifaddrs and getspnam
    
    Using AC_CHECK_FUNCS defines HAVE_xxx macro in config.h following
    a consistent naming convention.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit b6309b26edd477f45dd34b6d746eb2ac1e3fe9fc
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Thu Sep 2 17:47:55 2010 -0400

    config: use AC_CHECK_FUNCS for setusercontext
    
    AC_CHECK_FUNCS defines HAVE_SETUSERCONTEXT
    
    The replaced code made an incorrect assumption that it was only
    availble on BSD-like systems.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 80b44cfae4ab244ab4fd9d0ed0e089d946526755
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Sep 1 09:37:42 2010 -0400

    config: use AC_CHECK_FUNCS for setprocttile
    
    AC_CHECK_FUNCS defines HAVE_SETPROCTITLE
    
    The replaced code made an incorrect assumption that it was only
    availble on BSD-like systems.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit d9ab9445d1c52174788fc5df34e5379747623410
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Sep 7 16:47:26 2010 -0400

    config: add "Check for system library functions"
    
    Relocate the existing AC_CHECK_FUNCS statement.
    
    The functions are checked after the libraries checks to benefit
    from the $LIBS updated values.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 28d6283c5bfa9a7d19635b95de26907987799e2b
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Wed Oct 20 15:35:51 2010 -0400

    config: relocate headers, typedefs, and compiler characteristics
    
    Those checks are next following the "Check for system libraries" section.
    They are done before xorg specific configuration.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 849a74c4146d1f1cb961026be2540b4a9970fa26
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Sep 7 16:08:06 2010 -0400

    config: add "Check for system libraries" section
    
    This section gathers all the system libraries needed for
    xdm features. They may be in standard C library on some
    platforms and in discrete libraries on others.
    
    Any xorg library is excluded from this section.
    XTRANS macros will be invoked at a later time and contribute
    additional libraries.
    
    Although a function is used to search for the library, this
    should not be confused with function checking (AC_CHECK_FUNC).
    
    Sample configuration output on GNU/Linux:
    checking for library containing getifaddrs... none required
    checking for library containing getspnam... none required
    checking for library containing setproctitle... no
    checking for library containing setusercontext... no
    checking for library containing arc4random... -lbsd
    checking for library containing crypt... -lcrypt
    
    In the Makefile, libs are pre-pended:
    LIBS = -lcrypt -lbsd
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 4455d61291b7662e652953ca326627104e0d5494
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Sep 7 15:45:44 2010 -0400

    config: relocate the "Check for programs section"
    
    Following the usual layout, programs are check before the xorg specifics start.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 2ff31d55be7fb62aece1324ea14fed9cdac23789
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Sep 7 15:38:15 2010 -0400

    config: libtool: disable production of static libraries.
    
    Regroup libtool statements under the "Initialize" libtool section
    
    The greeter is only used as a shared library that is dynamically loaded.
    
    Alan Coopersmith wrote:
    "I don't think it's intended to be used as a shared library, but instead to
    allow sites to drop in their own greeter module for greater customization
    than allowed via just .Xresources."
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit b4d166d8c4f8ebc28fcd4d6f24065951989d11b2
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Sep 7 14:36:03 2010 -0400

    config: replace deprecated AM_CONFIG_HEADER with AC_CONFIG_HEADERS
    
    Regroup AC statements under the Autoconf initialization section.
    Regroup AM sttaements under the Automake initialization section.
    Add missing AC_CONFIG_SRCDIR
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 75a285c2c7cd8c3567126dafbdc047d2bdac421c
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Aug 31 17:25:02 2010 -0400

    config: look for PAM support on darwin as well
    
    There is an assumption that PAM support is not available on darwin.
    A quick browse on the net indicates otherwise.
    
    For systems with PAM, if any, this authentication method will
    be used, as per original configuration intent.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 22e7cc3458107662de736f9c82679d32197625bc
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Tue Aug 31 10:52:58 2010 -0400

    config: remove unused OS_CFLAGS variable
    
    Now unused as a result of previous patches
    
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit c76330b71a7d65620e411df7b327ad82bff97d90
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 30 19:51:24 2010 -0400

    config: use AC_CHECK_HEADERS for sys/param.h
    
    This is the FreeBSD porting guide recommended
    method to check for BSD 4.3 Net2 or newer systems.
    
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 96380df139761274e03babf71b7cb6692acb847a
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 30 15:51:16 2010 -0400

    config: definition of CSRG_BASED in configure.ac is flawed and redundant
    
    CSRG_BASED is defined in both the xserver using platform checks
    and in Xosdefs.h using compiler macros.
    
    The one from X11/Xosdefs.h will be used through already included headers:
    # if defined(__OpenBSD__) || defined(__NetBSD__) || defined(__FreeBSD__) \
    	|| defined(__APPLE__) || defined(__DragonFly__)
    #  ifndef CSRG_BASED
    #   define CSRG_BASED
    #  endif
    # endif
    
    Reviewed-by: Mark Kettenis <kettenis@openbsd.org>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 1d4f032110924d90cb2302d5dcc241e961043a37
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 30 09:32:06 2010 -0400

    config: remove unused define NO_LASTLOG for Darwin
    
    This is a leftover from commit 6ae3766444c62ee when sessreg
    was broken out so other display managers like gdm could use it
    without having to install all of xdm.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 8b7999f8e0f797a593ac2f4697ba3be983b421ae
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 30 07:39:56 2010 -0400

    config: use libtool -export-dynamic option for reverse dependencies
    
    This replaces -rdynamic which is a GNU/Linux only solution.
    
    "If symbols from your executable are needed to satisfy unresolved
    references in a library you want to dlopen you will have to use
    the flag -export-dynamic. You should use -export-dynamic while
    linking the executable that calls dlopen."
    
    It is used by the xserver, in Xdmx for example.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 50c96170ad42321310c346cf412f9ae7e80ec2a7
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 19:56:59 2010 -0400

    config: AC_LIBTOOL_DLOPEN is required for dynamic linking
    
    It adds the following tests in the configuration :
    checking for shl_load... no
    checking for shl_load in -ldld... no
    checking for dlopen... no
    checking for dlopen in -ldl... yes
    checking whether a program can dlopen itself... yes
    checking whether a statically linked program can dlopen itself... no
    
    This has not resulted in any compiler/linker flags change on
    a GNU/Linux platform.
    
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 5d79c0fad950ee6b83011a6c0242c9dc00c711a7
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 09:05:52 2010 -0400

    config: remove AC_LIBTOOL_WIN32_DLL
    
    As of 2002, this is no longer required and is a no-op
    in libtool v 1.5 and higher.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 0a90d3a5f580337e5d21eac4316b282e81bae68f
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 09:02:19 2010 -0400

    remove AC_C_INLINE as all compilers support inline keyword.
    
    This macro is not used anywhere else in xorg.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 4afafa90e79882b464b4aaee73826fb95136e7b5
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 09:00:45 2010 -0400

    config: remove AC_PROG_CC as it overrides AC_PROG_C_C99
    
    XORG_STRICT_OPTION from XORG_DEFAULT_OPTIONS calls
    AC_PROG_C_C99. This sets gcc with -std=gnu99.
    If AC_PROG_CC macro is called afterwards, it resets CC to gcc.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit f56891b74f40df456e2e447745f70665d87d8596
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 08:58:48 2010 -0400

    config: remove AC_PROG_INSTALL provided by XORG_DEFAULTS
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit a0e103250ffa2d652fa4b2a6e8c719d54f616988
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 29 08:27:28 2010 -0400

    greeter: transfer -DGREET_LIB from configure.ac to Makefile.am
    
    It does not depend on module configuration.
    It should not mixed with flags required by dependencies.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 7e223d3ac6c0d549a7d6e4dcc86a053e19594028
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sat Aug 28 20:11:28 2010 -0400

    greeter: only build the shared library version
    
    There is no evidence that the static version is needed since monolith
    as hinted by the comment in configure.ac.
    
    The greeter code is now all managed by the greeter Makefile.
    The configure option enable-dynamic-greeter no longer needed.
    GREET_USER_STATIC is no longer needed.
    GREET_LIB is still needed in greet.h as it is compiled with
    both session.c for xdm execuatable and greeter shared lib.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 674f08cbcbf9c4a29560e8b32e0335d9638b6868
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sat Aug 28 19:43:03 2010 -0400

    config: remove copying Chooser.ad to Chooser
    
    This was removed from all other apps with resources in app-defaults.
    The Chooser resource file is a file in git that gets installed
    and distributed.
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit b8ca599af4a22c5cd1b2aaff203607fe8e5b99d4
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Mon Aug 23 07:02:44 2010 -0400

    man: build man page using sed rather than the C preprocessor
    
    This fixes misalignments to correctly display \n and \ for sample file.
    No source code changes needed to fix that.
    
    Replace XCOMM IMakefile comment with #.
    
    Replace #if with:
    .if !'x.VARNAME'x.' .ig
    [man page text]
    ..
    
    If $(VARNAME) is empty, the man page text is shown
    If $(VARNAME) is not empty, the man page text is skipped
    
    Or replace #if with:
    .if 'VARNAME'' .ig
    [man page text]
    ..
    
    If $(VARNAME) is empty, the man page text is skipped
    If $(VARNAME) is not empty, the man page text is shown
    
    In DisplayManager.randomDevice, the default /dev/urandom is no longer
    displayed with double-quotes which is consistent with other
    default values. This was a cpp side-effect.
    
    Tested on Debian with groff and Solaris with nroff
    
    Co-authored-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Gaetan Nadon <memsize@videotron.ca>

commit 86f354f0a970e2c39e65b44832aad6ad69f30dc5
Author: Gaetan Nadon <memsize@videotron.ca>
Date:   Sun Aug 22 17:47:39 2010 -0400

    man: conditionally include DisplayManager.randomFile resource
    
    This resource follows the same pattern as prndSocket and prngPort.
    It is displayed to the user only if ARC4_RANDOM is not defined.
    It seems to be a simple omission in the man page.
    Refer to resource.c:
    
    char	*randomFile;
    char	*randomDevice;
    char	*prngdSocket;
[--snip--]
