[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: gnulib-tool warnings
From: |
Bruno Haible |
Subject: |
Re: gnulib-tool warnings |
Date: |
Mon, 25 Jan 2010 22:44:10 +0100 |
User-agent: |
KMail/1.9.9 |
Jim Meyering wrote:
> Then how about an option to make any warning (or at least those warnings)
> be treated as an error? Otherwise, it's just too easy to overlook
> the warnings in the reams of usual output.
If someone is not looking at the output and wants to see warnings nevertheless,
he can use the option '--quiet' or even '--quiet --quiet'.
If someone wants to ensure things stop when there was a warning, he can do so
by testing if gnulib-tool emitted something to standard error.
To make analysis of standard error easier and avoid confusion between errors
and warnings (a common problem with automake, IIRC), let me make gnulib-tool's
warning output more consistent:
2010-01-25 Bruno Haible <address@hidden>
gnulib-tool: Make warning diagnostics consistent.
* gnulib-tool (func_warning): New function.
Use it everywhere where gnulib-tool produces output to stderr and it is
not a fatal error.
--- gnulib-tool.orig Mon Jan 25 22:33:33 2010
+++ gnulib-tool Mon Jan 25 22:32:09 2010
@@ -490,6 +490,13 @@
func_exit 1
}
+# func_warning message
+# Outputs to stderr a warning message,
+func_warning ()
+{
+ echo "gnulib-tool: warning: $1" 1>&2
+}
+
# func_readlink SYMLINK
# outputs the target of the given symlink.
if (type -p readlink) > /dev/null 2>&1; then
@@ -1135,7 +1142,7 @@
func_exit 1
fi
if test -z "$pobase" && test -n "$po_domain"; then
- echo "gnulib-tool: warning: --po-domain has no effect without a --po-base
option" 1>&2
+ func_warning "--po-domain has no effect without a --po-base option"
fi
# Determine the minimum supported autoconf version from the project's
@@ -1311,7 +1318,7 @@
# Verify that building the module description with 'patch' succeeds.
func_lookup_file "modules/$module"
else
- echo "gnulib-tool: module $module doesn't exist" 1>&2
+ func_warning "module $module doesn't exist"
module=
fi
}
@@ -1689,7 +1696,7 @@
# Duplicate dependencies are harmless, but Jim wants a warning.
duplicated_deps=`echo "$deps" | LC_ALL=C sort | LC_ALL=C uniq -d`
if test -n "$duplicated_deps"; then
- echo "warning: module $module has duplicated dependencies: "`echo
$duplicated_deps` 1>&2
+ func_warning "module $module has duplicated dependencies: "`echo
$duplicated_deps`
fi
for dep in $deps; do
if test -n "$incobsolete" \
@@ -3885,19 +3892,19 @@
GPLv2+)
case "$license" in
GPLv2+ | LGPLv2+) ;;
- *) echo "warning: module $requested_module depends on a
module with an incompatible license: $module" 1>&2 ;;
+ *) func_warning "module $requested_module depends on a
module with an incompatible license: $module" ;;
esac
;;
LGPL)
case "$license" in
LGPL | LGPLv2+) ;;
- *) echo "warning: module $requested_module depends on a
module with an incompatible license: $module" 1>&2 ;;
+ *) func_warning "module $requested_module depends on a
module with an incompatible license: $module" ;;
esac
;;
LGPLv2+)
case "$license" in
LGPLv2+) ;;
- *) echo "warning: module $requested_module depends on a
module with an incompatible license: $module" 1>&2 ;;
+ *) func_warning "module $requested_module depends on a
module with an incompatible license: $module" ;;
esac
;;
esac
Re: [PATCH] xstrtoll: new module, Richard W.M. Jones, 2010/01/25