[PATCH] bootstrap: don't ignore download_po_files failure.

[Giuseppe Scrivano]

Hello,

I have noticed this error today while I was on a network with a silly
firewall that blocked rsync.  I think `bootstrap' should fail in this
case.

Cheers,
Giuseppe



>From 59fbac2381583a3946adb7e3f2299f8143159b2d Mon Sep 17 00:00:00 2001
From: Giuseppe Scrivano <gscriv...@gnu.org>
Date: Mon, 3 May 2010 22:05:41 +0200
Subject: [PATCH] bootstrap: don't ignore download_po_files failure.

* build-aux/bootstrap (update_po_files): Don't ignore download_po_files
failure.
---
 ChangeLog           |    6 ++++++
 build-aux/bootstrap |    2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 0ec11b9..c5ba140 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-05-03  Giuseppe Scrivano  <gscriv...@gnu.org>
+
+       bootstrap: don't ignore download_po_files failure.
+       * build-aux/bootstrap (update_po_files): Don't ignore download_po_files
+       failure.
+
 2010-05-03  Jim Meyering  <meyer...@redhat.com>
 
        test-xalloc-die: avoid unwarranted test failure on OpenSolaris 5.11
diff --git a/build-aux/bootstrap b/build-aux/bootstrap
index e55b3d2..a9a778a 100755
--- a/build-aux/bootstrap
+++ b/build-aux/bootstrap
@@ -488,7 +488,7 @@ update_po_files() {
   test -d $ref_po_dir || mkdir $ref_po_dir || return
   download_po_files $ref_po_dir $domain \
     && ls "$ref_po_dir"/*.po 2>/dev/null |
-      sed 's|.*/||; s|\.po$||' > "$po_dir/LINGUAS"
+      sed 's|.*/||; s|\.po$||' > "$po_dir/LINGUAS" || return
 
   langs=`cd $ref_po_dir && echo *.po|sed 's/\.po//g'`
   test "$langs" = '*' && langs=x
-- 
1.7.0