On Solaris we need to explicitly include X11.h in a couple of places
because the other headers don't pull that in.

A couple of questions:

1)  We need to link with "rt"  -lrt on Solaris.  I'm not familiar
enough with cmake to know exactly how I should add this flag for
Solaris.  Any suggestions?

2)  How does one get commit privileges for this project?

Here is the patch:


>From 21dcb56a72c67e68cd35d28362a8d47487bd3bb3 Mon Sep 17 00:00:00 2001
From: David Sundstrom <su...@peapod.net>
Date: Fri, 28 Oct 2011 11:27:09 -0500
Subject: [PATCH] Add missing includes for Solaris

---
 client/X11/xf_window.c       |    1 +
 libfreerdp-kbd/layouts_xkb.c |    1 +
 2 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/client/X11/xf_window.c b/client/X11/xf_window.c
index 3dee4b7..da217f3 100644
--- a/client/X11/xf_window.c
+++ b/client/X11/xf_window.c
@@ -17,6 +17,7 @@
  * limitations under the License.
  */

+#include <X11/Xlib.h>
 #include <X11/Xutil.h>
 #include <X11/Xatom.h>

diff --git a/libfreerdp-kbd/layouts_xkb.c b/libfreerdp-kbd/layouts_xkb.c
index ceed0ae..df40ec0 100644
--- a/libfreerdp-kbd/layouts_xkb.c
+++ b/libfreerdp-kbd/layouts_xkb.c
@@ -22,6 +22,7 @@
 #include <string.h>

 #ifdef WITH_XKBFILE
+#include <X11/Xlib.h>
 #include <X11/XKBlib.h>
 #include <X11/extensions/XKBfile.h>
 #include <X11/extensions/XKBrules.h>
--
1.7.6.1

------------------------------------------------------------------------------
The demand for IT networking professionals continues to grow, and the
demand for specialized networking skills is growing even more rapidly.
Take a complimentary Learning@Cisco Self-Assessment and learn 
about Cisco certifications, training, and career opportunities. 
http://p.sf.net/sfu/cisco-dev2dev
_______________________________________________
Freerdp-devel mailing list
Freerdp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freerdp-devel

Reply via email to