[PATCH v2] os: print newline after printing display name

Peter Hutterer peter.hutterer at who-t.net
Wed Jun 20 22:30:17 PDT 2012


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 <peter.hutterer at who-t.net>
---
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



More information about the xorg-devel mailing list