On 01.11.2011, at 12:28, Blue Swirl <blauwir...@gmail.com> wrote: > On Sun, Oct 30, 2011 at 20:23, Alexander Graf <ag...@suse.de> wrote: >> 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. > > I'd still prefer the submodule approach to help win32 and OSX users, > assuming that libfdt builds on those platforms.
Then pull and reverse this patch please. I want at least the rest in before the freeze. Alex > >> Signed-off-by: Alexander Graf <ag...@suse.de> >> >> --- >> >> 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 >> >>
