Skip to content

Update Makefile to use Docker Compose V2 syntax #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ITJamie wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Makefile to use Docker Compose V2 syntax #75

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 6 additions & 10 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,15 @@
SHELL := $(shell which bash)
# Get docker path or an empty string
DOCKER := $(shell command -v docker)
# Get docker-compose path or an empty string
DOCKER_COMPOSE := $(shell command -v docker-compose)


# Test if the dependencies we need to run this Makefile are installed
deps:
ifndef DOCKER
@echo "Docker is not available. Please install docker"
@exit 1
endif
ifndef DOCKER_COMPOSE
@echo "docker-compose is not available. Please install docker-compose"
@exit 1
endif


# TODO: for future use, will tag docker build
COMMIT := $(shell git rev-list -1 HEAD)
Expand All @@ -31,7 +27,7 @@ LEMUR_GIT_DIR = lemur-build-docker/lemur

clean: deps
rm -rf $(LEMUR_GIT_DIR)
docker-compose down -v || sudo docker-compose down -v
docker compose down -v || sudo docker compose down -v

all: deps
$(MAKE) copy_env_files
Expand All @@ -48,11 +44,11 @@ lemur_checkout:
cd $(LEMUR_GIT_DIR) && git pull

build_containers: deps
docker-compose build || sudo docker-compose build || echo "failed to build containers"
docker compose build || sudo docker compose build || echo "failed to build containers"

restart_containers: deps
docker-compose stop || sudo docker-compose stop || echo "failed to stop containers"
docker-compose up -d || sudo docker-compose up -d || echo "failed to start containers"
docker compose stop || sudo docker compose stop || echo "failed to stop containers"
docker compose up -d || sudo docker compose up -d || echo "failed to start containers"

# TODO: should run with sudo if you do not have access rights to docker
inspect_image_size: deps
Expand Down