Skip to content

Commit

Permalink
Makefile: Fix redirect to stderr to be more portable (crystal-lang#5859)
Browse files Browse the repository at this point in the history
`>/dev/stderr` does not work in some environments. Moreover, all POSIX
compliant shells supports standard `>&2` for redirect stdout to stderr.
  • Loading branch information
jirutka authored and RX14 committed Mar 24, 2018
1 parent dedc726 commit 9adbb92
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ CXXFLAGS += $(if $(debug),-g -O0)
ifeq (${LLVM_CONFIG},)
$(error Could not locate llvm-config, make sure it is installed and in your PATH, or set LLVM_CONFIG)
else
$(shell echo $(shell printf '\033[33m')Using $(LLVM_CONFIG) [version=$(shell $(LLVM_CONFIG) --version)]$(shell printf '\033[0m') >/dev/stderr)
$(shell echo $(shell printf '\033[33m')Using $(LLVM_CONFIG) [version=$(shell $(LLVM_CONFIG) --version)]$(shell printf '\033[0m') >&2)
endif

.PHONY: all
Expand Down

0 comments on commit 9adbb92

Please sign in to comment.