diff options
author | Tom Tromey <tom@tromey.com> | 2018-06-28 11:19:33 -0600 |
---|---|---|
committer | Tom Tromey <tom@tromey.com> | 2018-07-17 09:54:17 -0600 |
commit | 056dec39ed11e0027ae399018c3fef3a719718b9 (patch) | |
tree | 5fb766a67cbe830cd8484ccd85e52b652ad6d873 | |
parent | 4735f0edf4265e3e260e10a12921baf17987a020 (diff) | |
download | gdb-056dec39ed11e0027ae399018c3fef3a719718b9.zip gdb-056dec39ed11e0027ae399018c3fef3a719718b9.tar.gz gdb-056dec39ed11e0027ae399018c3fef3a719718b9.tar.bz2 |
Remove --disable-gdbcli
I think it doesn't really make sense to allow building gdb without the
CLI. Perhaps at one time this was a goal, but libgdb is long gone and
the CLI is intrinsic to gdb.
So, this patch removes the implementation of this configure option.
It is still recognized (this is autoconf's default), but does nothing.
This simplifies configure.ac and Makefile.in a bit.
Tested by rebuilding.
gdb/ChangeLog
2018-07-17 Tom Tromey <tom@tromey.com>
* configure.ac: Remove --disable-gdbcli.
* configure: Rebuild.
* Makefile.in (SUBDIR_CLI_DEPS, SUBDIR_CLI_LDFLAGS)
(SUBDIR_CLI_CFLAGS): Remove.
(SFILES): Use SUBDIR_CLI_SRCS.
(COMMON_OBS): Use SUBDIR_CLI_OBS.
-rw-r--r-- | gdb/ChangeLog | 9 | ||||
-rw-r--r-- | gdb/Makefile.in | 6 | ||||
-rw-r--r-- | gdb/NEWS | 5 | ||||
-rwxr-xr-x | gdb/configure | 28 | ||||
-rw-r--r-- | gdb/configure.ac | 23 |
5 files changed, 15 insertions, 56 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 01c1a25..6f5487c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,14 @@ 2018-07-17 Tom Tromey <tom@tromey.com> + * configure.ac: Remove --disable-gdbcli. + * configure: Rebuild. + * Makefile.in (SUBDIR_CLI_DEPS, SUBDIR_CLI_LDFLAGS) + (SUBDIR_CLI_CFLAGS): Remove. + (SFILES): Use SUBDIR_CLI_SRCS. + (COMMON_OBS): Use SUBDIR_CLI_OBS. + +2018-07-17 Tom Tromey <tom@tromey.com> + PR gdb/18624: * coffread.c (coff_symtab_read): Use scoped_free_pendings. diff --git a/gdb/Makefile.in b/gdb/Makefile.in index b145e51..e68aee2 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -253,10 +253,6 @@ SUBDIR_CLI_SRCS = \ SUBDIR_CLI_OBS = $(patsubst %.c,%.o,$(SUBDIR_CLI_SRCS)) -SUBDIR_CLI_DEPS = -SUBDIR_CLI_LDFLAGS = -SUBDIR_CLI_CFLAGS = - # # MI sub directory definitons # @@ -1160,6 +1156,7 @@ SFILES = \ symfile-mem.c \ ui-file.h \ mi/mi-common.c \ + $(SUBDIR_CLI_SRCS) \ $(SUBDIR_TARGET_SRCS) \ $(COMMON_SFILES) \ $(SUBDIR_GCC_COMPILE_SRCS) @@ -1557,6 +1554,7 @@ COMMON_OBS = $(DEPFILES) $(CONFIG_OBS) $(YYOBJ) \ version.o \ xml-builtin.o \ $(patsubst %.c,%.o,$(COMMON_SFILES)) \ + $(SUBDIR_CLI_OBS) \ $(SUBDIR_TARGET_OBS) \ $(SUBDIR_GCC_COMPILE_OBS) @@ -130,13 +130,16 @@ SH-5/SH64 running OpenBSD SH-5/SH64 support in sh*-*-openbsd* the tradeoff that there is a possibility of false hits being reported. -* New configure options +* Configure changes --enable-codesign=CERT This can be used to invoke "codesign -s CERT" after building gdb. This option is useful on macOS, where code signing is required for gdb to work properly. +--disable-gdbcli has been removed + This is now silently accepted, but does nothing. + *** Changes in GDB 8.1 * GDB now supports dynamically creating arbitrary register groups specified diff --git a/gdb/configure b/gdb/configure index 28756ed..3f0a220 100755 --- a/gdb/configure +++ b/gdb/configure @@ -855,7 +855,6 @@ with_auto_load_dir with_auto_load_safe_path enable_targets enable_64_bit_bfd -enable_gdbcli enable_gdbmi enable_tui enable_gdbtk @@ -1547,7 +1546,6 @@ Optional Features: --enable-targets=TARGETS alternative target configurations --enable-64-bit-bfd 64-bit support (on hosts with narrower word sizes) - --disable-gdbcli disable command-line interface (CLI) --disable-gdbmi disable machine-interface (MI) --enable-tui enable full-screen terminal user interface (TUI) --enable-gdbtk enable gdbtk graphical user interface (GUI) @@ -6755,32 +6753,6 @@ _ACEOF fi -# The CLI cannot be disabled yet, but may be in the future. - -# Enable CLI. -# Check whether --enable-gdbcli was given. -if test "${enable_gdbcli+set}" = set; then : - enableval=$enable_gdbcli; case $enableval in - yes) - ;; - no) - as_fn_error $? "the command-line interface cannot be disabled yet" "$LINENO" 5 ;; - *) - as_fn_error $? "bad value $enableval for --enable-gdbcli" "$LINENO" 5 ;; - esac -else - enable_gdbcli=yes -fi - -if test x"$enable_gdbcli" = xyes; then - if test -d $srcdir/cli; then - CONFIG_OBS="$CONFIG_OBS \$(SUBDIR_CLI_OBS)" - CONFIG_DEPS="$CONFIG_DEPS \$(SUBDIR_CLI_DEPS)" - CONFIG_SRCS="$CONFIG_SRCS \$(SUBDIR_CLI_SRCS)" - ENABLE_CFLAGS="$ENABLE_CFLAGS \$(SUBDIR_CLI_CFLAGS)" - fi -fi - # Enable MI. # Check whether --enable-gdbmi was given. if test "${enable_gdbmi+set}" = set; then : diff --git a/gdb/configure.ac b/gdb/configure.ac index 4c20ea5..dfd38a7 100644 --- a/gdb/configure.ac +++ b/gdb/configure.ac @@ -299,29 +299,6 @@ if test "x$targ_defvec" != x; then [Define to BFD's default target vector. ]) fi -# The CLI cannot be disabled yet, but may be in the future. - -# Enable CLI. -AC_ARG_ENABLE(gdbcli, -AS_HELP_STRING([--disable-gdbcli], [disable command-line interface (CLI)]), - [case $enableval in - yes) - ;; - no) - AC_MSG_ERROR([the command-line interface cannot be disabled yet]) ;; - *) - AC_MSG_ERROR([bad value $enableval for --enable-gdbcli]) ;; - esac], - [enable_gdbcli=yes]) -if test x"$enable_gdbcli" = xyes; then - if test -d $srcdir/cli; then - CONFIG_OBS="$CONFIG_OBS \$(SUBDIR_CLI_OBS)" - CONFIG_DEPS="$CONFIG_DEPS \$(SUBDIR_CLI_DEPS)" - CONFIG_SRCS="$CONFIG_SRCS \$(SUBDIR_CLI_SRCS)" - ENABLE_CFLAGS="$ENABLE_CFLAGS \$(SUBDIR_CLI_CFLAGS)" - fi -fi - # Enable MI. AC_ARG_ENABLE(gdbmi, AS_HELP_STRING([--disable-gdbmi], [disable machine-interface (MI)]), |