lunny/git-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Makefile: silence GOIMPORTS invocation

Browse Source 
Because we have to manually determine a list of paths to pass to
'goimports', the command-line invocation can be quite lengthy.

It isn't useful to see a large amount of output scroll by in your
terminal when trying to build Git LFS when what you really want to see
is the 'go build' that gets executed.

So, for that reason, let's prefix the $(GOIMPORTS) run with a '@', which
will silence the command invocation on the command line.

If an error occurs, it will be bubbled up, and we'll be able to see
that while the 'set -x'-style output will remain hidden.
This commit is contained in:
Taylor Blau 2018-07-31 16:20:11 -05:00
parent 6c394eff86
commit 3c0c8ac3c5
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Makefile
View File

@ -311,7 +311,7 @@ vendor : glide.lock
.PHONY : fmt
ifeq ($(shell test -x "`which $(GOIMPORTS)`"; echo $$?),0)
fmt : $(SOURCES) | lint
$(GOIMPORTS) $(GOIMPORTS_EXTRA_OPTS) $?;
@$(GOIMPORTS) $(GOIMPORTS_EXTRA_OPTS) $?;
else
fmt : $(SOURCES) | lint
@echo "git-lfs: skipping fmt, no goimports found at \`$(GOIMPORTS)\` ..."