Skip to content

Commit

Permalink
Avoid running Makefile.main ::build ::tests on web
Browse files Browse the repository at this point in the history
The purpose of Makefile.web is to test and build
Lookout web, but it was also testing and building
the whole Lookout because Makefile.web includes
Makefile, and that one includes src-d/ci/Makefile.main

This commit overrides Makefile.main::build,
Makefile.main::test and Makefile.main::dependencies
targets to avoid calling them when using Makefile.web

Signed-off-by: David Pordomingo <[email protected]>
  • Loading branch information
dpordomingo committed Apr 16, 2019
1 parent 6949f7b commit 7840930
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions Makefile.web
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,19 @@

-include Makefile

# Makefile.main::test -> this::test
# The @echo replaces the commands defined by `Makefile.main::$(COMMANDS)`
$(COMMANDS):
@echo

# The @echo replaces the commands defined by `Makefile.main::dependencies`
dependencies:
@echo

# The @echo replaces the commands defined by `Makefile.main::test`
test: web-test
@echo

# this::build -> Makefile.main::build -> Makefile.main::$(COMMANDS)
# The @echo forces this prerequisites to be run before `Makefile.main::build` ones.
build: web-build web-pack
@echo

# To be used as -tags
WITH_STATIC_TAG := with_static
Expand Down

0 comments on commit 7840930

Please sign in to comment.