Skip to content

Commit

Permalink
fix: make image-{test,release} into PHONY targets
Browse files Browse the repository at this point in the history
  • Loading branch information
gmega committed Jan 17, 2025
1 parent 82f1f9a commit 3ed074e
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ SHELL := bash
tests \
unit-docker \
integration-docker \
image-test \
image-release \
clean

# Runs the unit tests locally.
Expand All @@ -31,18 +33,13 @@ integration:

tests: unit integration

docker/.lastbuilt-test.timestamp: docker/bittorrent-benchmarks.Dockerfile
# Builds the test image required for local dockerized integration tests.
image-test:
docker build -t bittorrent-benchmarks:test -f ./docker/bittorrent-benchmarks.Dockerfile .
touch docker/.lastbuilt-test.timestamp

docker/.lastbuilt-release.timestamp: docker/bittorrent-benchmarks.Dockerfile
image-release:
docker build -t bittorrent-benchmarks:test --build-arg BUILD_TYPE="release" \
-f ./docker/bittorrent-benchmarks.Dockerfile .
touch docker/.lastbuilt-release.timestamp

# Builds the test image required for local dockerized integration tests.
image-test: docker/.lastbuilt-test.timestamp
image-release: docker/.lastbuilt-release.timestamp

# Runs the unit tests in a docker container.
unit-docker: image-test
Expand Down

0 comments on commit 3ed074e

Please sign in to comment.