On Thu, Jun 21, 2012 at 6:30 AM, Peter Hutterer <[email protected]> wrote: > Much easier for scripts that try to read the display value off the file > descriptor. Plus, this restores the behaviour we had for this patch in > Fedora since server 1.6 (April 2009). > > Signed-off-by: Peter Hutterer <[email protected]>
Thanks for figuring that out,, Reviewed-by: Dave Airlie <[email protected]> > --- > alanc pointed me to why the \n wasn't part of dynamic_display, thanks. > See http://patchwork.freedesktop.org/patch/9740/ for Jon's original comment. > > Changes to v1: > - don't store \n in dynamic_display, just write it out instead > > os/connection.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/os/connection.c b/os/connection.c > index 039942f..3441472 100644 > --- a/os/connection.c > +++ b/os/connection.c > @@ -353,6 +353,7 @@ NotifyParentProcess(void) > #if !defined(WIN32) > if (dynamic_display[0]) { > write(displayfd, dynamic_display, strlen(dynamic_display)); > + write(displayfd, "\n", 1); > close(displayfd); > } > if (RunFromSmartParent) { > -- > 1.7.10.2 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
