<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.32.2">
</HEAD>
<BODY>
On Thu, 2011-08-11 at 10:18 -0700, Alan Coopersmith wrote:
<BLOCKQUOTE TYPE=CITE>
<PRE>
On 08/11/11 09:48, Mat&#283;j Cepl wrote:
&gt; From: Adam Jackson &lt;<A HREF="mailto:ajax@nwnk.net">ajax@nwnk.net</A>&gt;
&gt; 
&gt; <A HREF="https://bugzilla.redhat.com/489999">https://bugzilla.redhat.com/489999</A>
&gt; Avoid collision with the ConsoleKit which doesn't start its session when
&gt; XDG_SESSION_COOKIE is set.
&gt; 
&gt; Signed-off-by: Mat&#283;j Cepl &lt;<A HREF="mailto:mcepl@redhat.com">mcepl@redhat.com</A>&gt;
&gt; ---
&gt;  startx.cpp |    1 +
&gt;  1 files changed, 1 insertions(+), 0 deletions(-)
&gt; 
&gt; diff --git a/startx.cpp b/startx.cpp
&gt; index 9f6561d..f2aa630 100644
&gt; --- a/startx.cpp
&gt; +++ b/startx.cpp
&gt; @@ -11,6 +11,7 @@ XCOMM
&gt;  XCOMM Site administrators are STRONGLY urged to write nicer versions.
&gt;  XCOMM
&gt;  
&gt; +unset XDG_SESSION_COOKIE
&gt;  unset DBUS_SESSION_BUS_ADDRESS
&gt;  unset SESSION_MANAGER
&gt;  

Seems reasonable to me, so:
Reviewed-by: Alan Coopersmith &lt;<A HREF="mailto:alan.coopersmith@oracle.com">alan.coopersmith@oracle.com</A>&gt;

But please include a hint to which module a patch goes with when sending
patches for non-Xserver modules.   We assume unadorned &quot;[PATCH]&quot; is
Xserver since it's the bulk of traffic, so prefer something like
&quot;[PATCH xinit]&quot; for the rest - including it in the brackets allows
it to show up in the e-mail, but not go into the git logs where it
will already be obvious which module it's for.  You can automate this
with git - for instance, my app/xinit/.git/config contains:

[format]
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; subjectprefix = PATCH:xinit
</PRE>
</BLOCKQUOTE>
<BR>
Mat&#283;j,<BR>
<BR>
You can use &quot;git config format.subjectprefix &quot;PATCH $(mod_name)&quot; if you want to script the whole tree.<BR>
<BR>
All this and other info are in a wiki:<BR>
<A HREF="http://www.x.org/wiki/Development/Documentation/SubmittingPatches">http://www.x.org/wiki/Development/Documentation/SubmittingPatches</A><BR>
<BR>
<BLOCKQUOTE TYPE=CITE>
<PRE>

-- 
        -Alan Coopersmith-        <A HREF="mailto:alan.coopersmith@oracle.com">alan.coopersmith@oracle.com</A>
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
<A HREF="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</A>: X.Org development
Archives: <A HREF="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</A>
Info: <A HREF="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</A>
</PRE>
</BLOCKQUOTE>
<BR>
</BODY>
</HTML>