Hi,

On 31 May 2017 at 19:32, Simon Glass sjg@chromium.org wrote:

This error needs to go to stderr otherwise it will not be reported by buildman. Fix it.

Signed-off-by: Simon Glass sjg@chromium.org

Makefile | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile index 89e0451a49..c13937c3a8 100644 --- a/Makefile +++ b/Makefile @@ -1357,12 +1357,13 @@ $(timestamp_h): $(srctree)/Makefile FORCE

checkbinman: tools @if ! ( echo 'import libfdt' | ( PYTHONPATH=tools python )); then \

•           echo '*** binman needs the Python libfdt library. Either '; \
  

•           echo '*** install it on your system, or try:'; \
  

•           echo '***'; \
  

•           echo '*** sudo apt-get install swig libpython-dev'; \
  

•           echo '***'; \
  

•           echo '*** to have U-Boot build its own version.'; \
  

•           echo >&2; \
  

•           echo >&2 '*** binman needs the Python libfdt library.'; \
  

•           echo >&2 '*** Either install it on your system, or try:'; \
  

•           echo >&2 '***'; \
  

•           echo >&2 '*** sudo apt-get install swig libpython-dev'; \
  

•           echo >&2 '***'; \
  

•           echo >&2 '*** to have U-Boot build its own version.'; \
            false; \
    fi
  
  

-- 2.13.0.219.gdb65acc882-goog

I have squashed this into 'fdt: Makefile: Build python libfdt library if needed' since it fixes up that patch.

Applied to u-boot-fdt.