Skip to content

Commit

Permalink
Replaced illegal tab in Makefile with spaces.
Browse files Browse the repository at this point in the history
Commands, such as $(error ...), are not allowed to be indented with tabs
outside of targets, throwing an error instead of outputting the actual
error. The solution is to use innocuous spaces instead. Ideally, spaces
should be used everywhere outside targets, but since make does not mind
it if variable assignments are tab-indented outside targets, a complete
overhaul is not necessary. However, if more errors are added, it might
make more sense to be consistent.

Also, make will already add a period so I removed it.
  • Loading branch information
gustavla committed Feb 25, 2015
1 parent 4a3887a commit d2beb8a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -261,7 +261,8 @@ ifneq (,$(findstring clang++,$(CXX)))
else ifneq (,$(findstring g++,$(CXX)))
STATIC_LINK_COMMAND := -Wl,--whole-archive $(STATIC_NAME) -Wl,--no-whole-archive
else
$(error Cannot static link with the $(CXX) compiler.)
# The following line must not be indented with a tab, since we are not inside a target
$(error Cannot static link with the $(CXX) compiler)
endif

# Debugging
Expand Down

0 comments on commit d2beb8a

Please sign in to comment.