Browse Source

package/gdb: append to dependencies in conditional

Repeat after me: "Forcing the value of <pkg>_DEPENDENCIES inside a
conditional is the root of all evil."

Repeat after me: "Forcing the value of <pkg>_DEPENDENCIES inside a
conditional is the root of all evil."

Repeat after me: "Forcing the value of <pkg>_DEPENDENCIES inside a
conditional is the root of all evil."

Repeat after me: "Forcing the value of <pkg>_DEPENDENCIES inside a
conditional is the root of all evil."

Enough? :-)

Due to this mistake, any other GDB_DEPENDENCIES defined before this
assignment were lost. For example, the host-flex host-bison added
inside the GDB_FROM_GIT==y condition were ignored if
BR2_PACKAGE_GDB_DEBUGGER.

Fixes the build of all ARC configurations that have
BR2_PACKAGE_GDB_DEBUGGER enabled.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
(cherry picked from commit 97f3ad7af3dd54a15a10aa35786e7fa08cf5e7b1)
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Thomas Petazzoni 3 years ago
parent
commit
65391de0e8
1 changed files with 1 additions and 1 deletions
  1. 1 1
      package/gdb/gdb.mk

+ 1 - 1
package/gdb/gdb.mk

@@ -156,7 +156,7 @@ ifeq ($(BR2_PACKAGE_GDB_DEBUGGER),y)
 GDB_CONF_OPTS += \
 GDB_CONF_OPTS += \
 	--enable-gdb \
 	--enable-gdb \
 	--with-curses
 	--with-curses
-GDB_DEPENDENCIES = ncurses \
+GDB_DEPENDENCIES += ncurses \
 	$(if $(BR2_PACKAGE_LIBICONV),libiconv)
 	$(if $(BR2_PACKAGE_LIBICONV),libiconv)
 else
 else
 GDB_CONF_OPTS += \
 GDB_CONF_OPTS += \