blackbox: needs iconv if !locale

Closes #2101

At the same time fix dependencies so it can be built without a local
X server, and a comment is shown if C++ support isn't enabled.

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

CHANGES

@@ -13,14 +13,14 @@
 	New packages: cgilua, copas, coxpcall, ffmpeg, luafilesystem,
 	luasocket, rings, wsapi, xavante, xterm
 
-	Updated/fixed packages: berkeleydb, bash, busybox, cdrkit, dash,
-	dialog, diffutils, distcc, dmalloc, dnsmasq, dropbear, fbv,
-	file, gawk, gstreamer, gzip, intltool, ipsec-tools, iptables,
-	libart, libfuse, libgpg-error, libidn, liblockfile, links,
-	lmbench, lrzsz, make, module-init-tools, nbd, ncurses, netperf,
-	ntfsprogs, openssl, php, qt, quagga, sqlite, sdl, sdl_mixer,
-	sdl_ttf, squashfs, tn5250, tremor, usbutils, xkeyboard-config,
-	xserver_xorg-server
+	Updated/fixed packages: berkeleydb, bash, blackbox, busybox,
+	cdrkit, dash, dialog, diffutils, distcc, dmalloc, dnsmasq,
+	dropbear, fbv, file, gawk, gstreamer, gzip, intltool,
+	ipsec-tools, iptables, libart, libfuse, libgpg-error, libidn,
+	liblockfile, links, lmbench, lrzsz, make, module-init-tools,
+	nbd, ncurses, netperf, ntfsprogs, openssl, php, qt, quagga,
+	sqlite, sdl, sdl_mixer, sdl_ttf, squashfs, tn5250, tremor,
+	usbutils, xkeyboard-config, xserver_xorg-server
 
 	Removed packages: modutils, portage, rxvt
 
@@ -51,6 +51,7 @@
 	#2035: ipsec-tools-0.7.2 fails to build with gcc-4.4.x
 	#2038: Bump ncurses to 5.7
 	#2095: make gconfig: undefined reference to symbol 'dlsym@@GLIBC_2.2.5'
+	#2101: blackbox depends on locale support
 	#2119: Tries to build kernel, although disabled in config
 	#2155: Compression lzo don't set for ubifs
 

package/blackbox/Config.in

@@ -2,8 +2,13 @@ config BR2_PACKAGE_BLACKBOX
 	bool "blackbox"
 	depends on BR2_PACKAGE_XORG7
 	depends on BR2_INSTALL_LIBSTDCPP
+	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
+	select BR2_PACKAGE_XLIB_LIBX11
 	help
 	  Blackbox is a fast, lightweight window manager for the X
 	  Window System.
 
 	  http://blackboxwm.sourceforge.net/
+
+comment "blackbox requires a toolchain with C++ support enabled"
+	depends on BR2_PACKAGE_XORG7 && !BR2_INSTALL_LIBSTDCPP

package/blackbox/blackbox.mk

@@ -14,6 +14,10 @@ BLACKBOX_INSTALL_TARGET:=YES
 BLACKBOX_CONF_OPT:=--x-includes=$(STAGING_DIR)/usr/include/X11 \
 		--x-libraries=$(STAGING_DIR)/usr/lib
 
-BLACKBOX_DEPENDENCIES:=xserver_xorg-server
+BLACKBOX_DEPENDENCIES = xlib_libX11
+
+ifneq ($(BR2_ENABLE_LOCALE),y)
+BLACKBOX_DEPENDENCIES += libiconv
+endif
 
 $(eval $(call AUTOTARGETS,package,blackbox))