wget -q in bootstrap is too silent

wget -q in bootstrap is too silent

[Tim Rühsen]

[-- Attachment #1.1: Type: text/plain, Size: 585 bytes --]

Hi,

we just had an issue with the bootstrap script because wget -q
suppresses error output.

E.g. instead of
"
ERROR: cannot verify translationproject.org's certificate, issued by
'CN=Let\'s Encrypt Authority X3,O=Let\'s Encrypt,C=US':
  Unable to locally verify the issuer's authority.
"
there is no output.

This lead later to the confusing error
"
automake: error: cannot open < lib/gnulib.mk: No such file or directory
"

Instead of -q we could use -nv to avoid disguising the real error.
But there will be some diagnostic noice...

WDYT ?

Regards, Tim


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: wget -q in bootstrap is too silent

[Paul Eggert]

[-- Attachment #1: Type: text/plain, Size: 334 bytes --]

On 10/4/19 8:34 AM, Tim Rühsen wrote:
> Instead of -q we could use -nv to avoid disguising the real error.
> But there will be some diagnostic noice...

At this point the bootstrap script is so noisy that a bit more won't hurt, 
especially if it's in a good cause. Thanks for reporting the problem; I 
installed the attached.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-bootstrap-simplify-debugging-of-wget-failures.txt --]
[-- Type: text/x-patch; name="0001-bootstrap-simplify-debugging-of-wget-failures.txt", Size: 1787 bytes --]

From 60039e9a201ffbbddffb6c1e8ba31c4b9dce9ff2 Mon Sep 17 00:00:00 2001
From: Paul Eggert <eggert@cs.ucla.edu>
Date: Sat, 5 Oct 2019 21:36:15 -0700
Subject: [PATCH] bootstrap: simplify debugging of wget failures
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Problem reported by Tim Rühsen in:
https://lists.gnu.org/r/bug-gnulib/2019-10/msg00000.html
* build-aux/bootstrap (po_download_command_format):
Invoke wget with -nv instead of -q, to make debugging easier.
---
 ChangeLog           | 8 ++++++++
 build-aux/bootstrap | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 325ddf57b..25f1f3e3b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2019-10-05  Paul Eggert  <eggert@cs.ucla.edu>
+
+	bootstrap: simplify debugging of wget failures
+	Problem reported by Tim Rühsen in:
+	https://lists.gnu.org/r/bug-gnulib/2019-10/msg00000.html
+	* build-aux/bootstrap (po_download_command_format):
+	Invoke wget with -nv instead of -q, to make debugging easier.
+
 2019-09-29  Bruno Haible  <bruno@clisp.org>
 
 	avltree-list: Fix compilation warning (introduced on 2014-09-16).
diff --git a/build-aux/bootstrap b/build-aux/bootstrap
index 5b08e7e2d..e273ea732 100755
--- a/build-aux/bootstrap
+++ b/build-aux/bootstrap
@@ -166,7 +166,7 @@ bootstrap_epilogue() { :; }
 # specified directory.  Fill in the first %s with the destination
 # directory and the second with the domain name.
 po_download_command_format=\
-"wget --mirror --level=1 -nd -q -A.po -P '%s' \
+"wget --mirror --level=1 -nd -nv -A.po -P '%s' \
  https://translationproject.org/latest/%s/"
 
 # Prefer a non-empty tarname (4th argument of AC_INIT if given), else
-- 
2.17.1

Re: wget -q in bootstrap is too silent

[Tim Rühsen]

[-- Attachment #1.1: Type: text/plain, Size: 406 bytes --]

Thank you, Paul !

On 06.10.19 06:38, Paul Eggert wrote:
> On 10/4/19 8:34 AM, Tim Rühsen wrote:
>> Instead of -q we could use -nv to avoid disguising the real error.
>> But there will be some diagnostic noice...
> 
> At this point the bootstrap script is so noisy that a bit more won't
> hurt, especially if it's in a good cause. Thanks for reporting the
> problem; I installed the attached.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]