diff options
| author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2012-11-21 22:19:57 -0500 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2012-11-26 10:31:11 -0800 |
| commit | cad06d4d78b52b99d451063db2e6d08c34697d32 (patch) | |
| tree | 9361c4acd21c897e2816ac8f35717e05f1163776 | |
| parent | ed9fe755130891fc878bb2433204faffb534697b (diff) | |
| download | git-cad06d4d78b52b99d451063db2e6d08c34697d32.tar.gz git-cad06d4d78b52b99d451063db2e6d08c34697d32.tar.xz | |
Makefile: hide stderr of curl-config test
You will get
$ make distclean 2>&1 | grep curl
/bin/sh: curl-config: not found
/bin/sh: curl-config: not found
/bin/sh: curl-config: not found
/bin/sh: curl-config: not found
/bin/sh: curl-config: not found
$
if you don't have a curl development package installed.
The intent is not to alarm the user, but just to test if there is
a new enough curl installed. However, if you look at search engine
suggested completions, the above "error" messages are confusing
people into thinking curl is a hard requirement.
Redirect this error output to /dev/null as it is not necessary to be
shown to the end users.
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1564,7 +1564,7 @@ else REMOTE_CURL_NAMES = $(REMOTE_CURL_PRIMARY) $(REMOTE_CURL_ALIASES) PROGRAM_OBJS += http-fetch.o PROGRAMS += $(REMOTE_CURL_NAMES) - curl_check := $(shell (echo 070908; curl-config --vernum) | sort -r | sed -ne 2p) + curl_check := $(shell (echo 070908; curl-config --vernum) 2>/dev/null | sort -r | sed -ne 2p) ifeq "$(curl_check)" "070908" ifndef NO_EXPAT PROGRAM_OBJS += http-push.o |