From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:43319) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RKbmS-0003XQ-J7 for qemu-devel@nongnu.org; Sun, 30 Oct 2011 16:14:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RKbmD-0002pC-Na for qemu-devel@nongnu.org; Sun, 30 Oct 2011 16:14:50 -0400 From: Alexander Graf Date: Sun, 30 Oct 2011 21:23:09 +0100 Message-Id: <1320006193-15219-19-git-send-email-agraf@suse.de> In-Reply-To: <1320006193-15219-1-git-send-email-agraf@suse.de> References: <1320006193-15219-1-git-send-email-agraf@suse.de> Subject: [Qemu-devel] [PATCH 18/22] PPC: Fail configure when libfdt is not available List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Blue Swirl , qemu-ppc@nongnu.org We have several targets in the PPC tree now that basically require libfdt to function properly, namely the pseries and the e500 targets. This dependency will rather increase than decrease in the future, so I want to make sure that people building shiny new 1.0 actually have libfdt installed to get rid of a few ifdefs in the code. Warning: This patch will likely make configure fail for people who don't select their own --target-list, but don't have libfdt development packages installed. However, we really need this new dependency to move on. Signed-off-by: Alexander Graf --- v1 -> v2: - no paranthesis - no fdt check for config_pseries - add . in error message --- configure | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/configure b/configure index f8dd4db..18537e8 100755 --- a/configure +++ b/configure @@ -3422,7 +3422,16 @@ case "$target_arch2" in fi fi esac -if test "$target_arch2" = "ppc64" -a "$fdt" = "yes"; then +if test "$fdt" != "yes" && test "$target_arch2" = "ppc" -o \ + "$target_arch2" = "ppc64" -o "$target_arch2" = "ppcemb"; then + echo + echo "Error: libfdt missing" + echo "The PowerPC target requires libfdt to work properly." + echo "Please make sure to have it and its development packages installed." + echo + exit 1 +fi +if test "$target_arch2" = "ppc64"; then echo "CONFIG_PSERIES=y" >> $config_target_mak fi if test "$target_bigendian" = "yes" ; then -- 1.6.0.2