summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Petazzoni <thomas.petazzoni@free-electrons.com>2010-07-29 09:50:59 +0200
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>2010-07-29 16:04:38 +0200
commita302e3aaa0a7e4eec1a82143efbbd09629b499ea (patch)
treed47a04c400401e7bfed8e77d2dcf81844fcde1be
parentaad29b55a8851dec26399e37f2b87c0b38c98fc3 (diff)
libiconv: add an error when both libiconv and locale are enabled
This error should never show up if all Buildroot dependencies are correct. However, rather than failing horribly later on, catch this particular case early on and error out. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-rw-r--r--package/libiconv/libiconv.mk7
1 files changed, 7 insertions, 0 deletions
diff --git a/package/libiconv/libiconv.mk b/package/libiconv/libiconv.mk
index cc40cd4b6..a3f762ab6 100644
--- a/package/libiconv/libiconv.mk
+++ b/package/libiconv/libiconv.mk
@@ -23,3 +23,10 @@ ifneq ($(BR2_ENABLE_DEBUG),y)
$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libcharset.so.*
endif
touch $@
+
+# Configurations where the toolchain supports locales and the libiconv
+# package is enabled are incorrect, because the toolchain already
+# provides libiconv functionality, and having both confuses packages.
+ifeq ($(BR2_PACKAGE_LIBICONV)$(BR2_ENABLE_LOCALE),yy)
+$(error Libiconv should never be enabled when the toolchain supports locales. Report this failure to Buildroot developers)
+endif