aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>2017-05-19 15:08:45 +0200
committerRainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>2017-05-19 15:08:45 +0200
commit6e7e1744e96abbf1a4229d5f269caf2cc921ec58 (patch)
tree7e9a4ab80d0472d557fb0b98bf47cc867dbdf53c
parent7a6e7fcc77997bf7679cce4f1cfebcd57ba8af70 (diff)
downloadgdb-6e7e1744e96abbf1a4229d5f269caf2cc921ec58.zip
gdb-6e7e1744e96abbf1a4229d5f269caf2cc921ec58.tar.gz
gdb-6e7e1744e96abbf1a4229d5f269caf2cc921ec58.tar.bz2
Fix tui compilation with Solaris libcurses: clear define (PR tui/21482)
On both mainline and the 8.0 branch, gdb compilation fails on Solaris 10 with the native libcurses like this: In file included from /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/gdb_curses.h:42: 0, from /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/tui/tui-data.h:2 6, from /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/tui/tui-disasm.c :31: /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/tui/tui-disasm.c: In function `CORE_A DDR tui_disassemble(gdbarch*, tui_asm_line*, CORE_ADDR, int)': /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/tui/tui-disasm.c:71:19: error: `class string_file' has no member named `wclear'; did you mean `clear'? gdb_dis_out.clear (); ^ /vol/src/gnu/gdb/gdb-8.0-branch/local/gdb/tui/tui-disasm.c:78:19: error: `class string_file' has no member named `wclear'; did you mean `clear'? gdb_dis_out.clear (); ^ make[2]: *** [Makefile:1927: tui-disasm.o] Error 1 It turned out this happens because <curses.h> has #define clear() wclear(stdscr) This can be avoided by defining NOMACROS, which the patch below does. ncurses potentially has a similar problem, which can be avoided by defining NCURSES_NOMACROS. PR tui/21482 * gdb_curses.h (NOMACROS): Define. (NCURSES_NOMACROS): Define.
-rw-r--r--gdb/ChangeLog7
-rw-r--r--gdb/gdb_curses.h7
2 files changed, 14 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 6bffa90..59fa951 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,12 @@
2017-05-19 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
+ PR tui/21482
+ * gdb_curses.h (NOMACROS): Define.
+ (NCURSES_NOMACROS): Define.
+
+2017-05-19 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE>
+
+ PR tui/21482
* tui/tui-windata.c (tui_erase_data_content): Cast last mvwaddstr
arg to char *.
* tui/tui-wingeneral.c (box_win): Likewise.
diff --git a/gdb/gdb_curses.h b/gdb/gdb_curses.h
index 16442c7..72fafe3 100644
--- a/gdb/gdb_curses.h
+++ b/gdb/gdb_curses.h
@@ -32,6 +32,13 @@
#undef KEY_EVENT
#endif
+/* On Solaris and probably other SysVr4 derived systems, we need to define
+ NOMACROS so the native <curses.h> doesn't define clear which interferes
+ with the clear member of class string_file. ncurses potentially has a
+ similar problem and fix. */
+#define NOMACROS
+#define NCURSES_NOMACROS
+
#if defined (HAVE_NCURSES_NCURSES_H)
#include <ncurses/ncurses.h>
#elif defined (HAVE_NCURSES_H)