Skip to content

Commit

Permalink
[native] Remove centos-container.dockfile
Browse files Browse the repository at this point in the history
This script is out of date.
It uses a Velox circle ci image as the base image.
Presto release team should build and manage the base image.
  • Loading branch information
majetideepak authored and tdcmeehan committed May 15, 2024
1 parent 6784981 commit 760853d
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 31 deletions.
11 changes: 0 additions & 11 deletions presto-native-execution/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -107,17 +107,6 @@ tidy-fix: cmake #: Fix clang-tidy issues in the master branch
tidy-check: cmake #: Check clang-tidy issues in the master branch
pushd "$$(pwd)/.." && presto-native-execution/velox/scripts/check.py tidy master && popd

centos-container: #: Build the linux container for CircleCi
$(MAKE) linux-container CONTAINER_NAME=centos

linux-container:
rm -rf /tmp/docker && \
mkdir -p /tmp/docker && \
$(eval VELOX_SETUP_SCRIPT=$(shell [ $(CONTAINER_NAME) == centos ] && echo setup-centos8.sh; [ $(CONTAINER_NAME) != centos ] && echo setup-$(CONTAINER_NAME).sh;)) \
cp scripts/setup-$(CONTAINER_NAME).sh scripts/$(CONTAINER_NAME)-container.dockfile velox/scripts/${VELOX_SETUP_SCRIPT} velox/scripts/setup-helper-functions.sh /tmp/docker && \
cd /tmp/docker && \
docker build --build-arg cpu_target=$(CPU_TARGET) --tag "prestocpp/prestocpp-$(CPU_TARGET)-$(CONTAINER_NAME):$(USER)-$(shell date +%Y%m%d)" -f $(CONTAINER_NAME)-container.dockfile .

help: #: Show the help messages
@cat $(firstword $(MAKEFILE_LIST)) | \
awk '/^[-a-z]+:/' | \
Expand Down
1 change: 0 additions & 1 deletion presto-native-execution/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,6 @@ A reminder of the available Makefile targets can be obtained using `make help`
header-check Check for license header issues on the current branch
tidy-fix Fix clang-tidy issues in the current branch
tidy-check Check clang-tidy issues in the current branch
linux-container Build the CircleCi linux container from scratch
help Show the help messages
```

Expand Down
19 changes: 0 additions & 19 deletions presto-native-execution/scripts/centos-container.dockfile

This file was deleted.

0 comments on commit 760853d

Please sign in to comment.