Skip to content

Commit 7de8ffd

Browse files
committed
php: drop php client
Issue vitessio#3230 Now that we support mysql protocol, there's no need to support a custom PHP client any more.
1 parent 2f937a2 commit 7de8ffd

File tree

349 files changed

+3
-51006
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

349 files changed

+3
-51006
lines changed

.gitmodules

-3
Original file line numberDiff line numberDiff line change
@@ -1,3 +0,0 @@
1-
[submodule "php/pdo"]
2-
path = php/pdo
3-
url = https://github.com/pixelfederation/vitess-php-pdo.git

Makefile

+1-14
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ MAKEFLAGS = -s
1818
# Since we are not using this Makefile for compilation, limiting parallelism will not increase build time.
1919
.NOTPARALLEL:
2020

21-
.PHONY: all build build_web test clean unit_test unit_test_cover unit_test_race integration_test proto proto_banner site_test site_integration_test docker_bootstrap docker_test docker_unit_test java_test php_test reshard_tests
21+
.PHONY: all build build_web test clean unit_test unit_test_cover unit_test_race integration_test proto proto_banner site_test site_integration_test docker_bootstrap docker_test docker_unit_test java_test reshard_tests
2222

2323
all: build
2424

@@ -113,10 +113,6 @@ java_test:
113113
go install ./go/cmd/vtgateclienttest ./go/cmd/vtcombo
114114
mvn -f java/pom.xml clean verify
115115

116-
php_test:
117-
go install ./go/cmd/vtgateclienttest
118-
phpunit php/tests
119-
120116
# TODO(mberlin): Remove the manual copy once govendor supports a way to
121117
# install vendor'd programs: https://github.com/kardianos/govendor/issues/117
122118
install_protoc-gen-go:
@@ -167,15 +163,6 @@ $(PROTO_GO_TEMPS): go/vt/.proto.tmp/%.pb.go: proto/%.proto
167163
$(PROTOC_DIR)/protoc -Iproto $< --go_out=plugins=grpc:go/vt/.proto.tmp
168164
sed -i -e 's,import \([a-z0-9_]*\) ".",import \1 "github.com/youtube/vitess/go/vt/proto/\1",g' $@
169165

170-
# Generate the PHP proto files in a Docker container, and copy them back.
171-
php_proto:
172-
docker run -ti --name=vitess_php-proto -v $$PWD/proto:/in vitess/bootstrap:common bash -c 'cd $$VTTOP && mkdir -p proto && cp -R /in/* proto/ && tools/proto-gen-php.sh'
173-
docker cp vitess_php-proto:/vt/src/github.com/youtube/vitess/php/src/descriptor.php php/src/
174-
docker cp vitess_php-proto:/vt/src/github.com/youtube/vitess/php/src/php.php php/src/
175-
rm -r php/src/Vitess/Proto/*
176-
docker cp vitess_php-proto:/vt/src/github.com/youtube/vitess/php/src/Vitess/Proto/. php/src/Vitess/Proto/
177-
docker rm vitess_php-proto
178-
179166
# Helper targets for building Docker images.
180167
# Please read docker/README.md to understand the different available images.
181168

composer.json

-19
This file was deleted.

0 commit comments

Comments
 (0)