Include solvespace.h before spnav.h in the GTK frontend

Forwarded: no
Last-Update: 2026-10-03

spnav.h includes <X11/Xlib.h>, which does "#define Complex 0". When Eigen
is included afterwards, its AltiVec backend (used on ppc64el) fails to
compile because it uses "bool Complex" as a template parameter name.
Include solvespace.h, and therefore Eigen, before the X11 headers.

Gbp-Pq: Name 08-Include-Eigen-before-X11-headers.patch

diff --git a/src/platform/guigtk.cpp b/src/platform/guigtk.cpp
index 522fb13586cba7d945d896119ece1a6608bd13e9..2e845a3aded3799dd392d926a18b0ea1070c1910 100644 (file)
--- a/src/platform/guigtk.cpp
+++ b/src/platform/guigtk.cpp
@@ -32,6 +32,11 @@
 #   include <gtkmm/filechoosernative.h>
 #endif
 
+// Include solvespace.h (and thus Eigen) before spnav.h: the latter pulls in
+// <X11/Xlib.h>, which defines macros such as `Complex` that break Eigen's
+// AltiVec template code on ppc64el.
+#include "solvespace.h"
+
 #if defined(HAVE_SPACEWARE)
 #   include <spnav.h>
 #   include <gdk/gdk.h>
@@ -48,8 +53,6 @@
 #   endif
 #endif
 
-#include "solvespace.h"
-
 namespace SolveSpace {
 namespace Platform {