diff options
author | Stanley Cheung <stanley.cheung@gmail.com> | 2018-08-10 17:09:03 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-08-10 17:09:03 -0700 |
commit | 70c8a58b2918a5369905e5a203d7ce7897b6207e (patch) | |
tree | 152db531c986b3d8055998563be93406cd85a717 | |
parent | 84ac8c2e5c4dc6db38f16e683b75e75aae09eae5 (diff) | |
parent | 9993869d042c59b79bac11c45f4c58ace5dbeccd (diff) |
Merge pull request #16320 from stanley-cheung/remove-php-macos-build-artifact
No need to build artifact for PHP MacOS
-rw-r--r-- | tools/run_tests/artifacts/artifact_targets.py | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/tools/run_tests/artifacts/artifact_targets.py b/tools/run_tests/artifacts/artifact_targets.py index a37098d9bf..bdeb258e1f 100644 --- a/tools/run_tests/artifacts/artifact_targets.py +++ b/tools/run_tests/artifacts/artifact_targets.py @@ -290,15 +290,9 @@ class PHPArtifact: return [] def build_jobspec(self): - if self.platform == 'linux': - return create_docker_jobspec( - self.name, 'tools/dockerfile/grpc_artifact_linux_{}'.format( - self.arch), - 'tools/run_tests/artifacts/build_artifact_php.sh') - else: - return create_jobspec( - self.name, ['tools/run_tests/artifacts/build_artifact_php.sh'], - use_workspace=True) + return create_docker_jobspec( + self.name, 'tools/dockerfile/grpc_artifact_linux_{}'.format( + self.arch), 'tools/run_tests/artifacts/build_artifact_php.sh') class ProtocArtifact: @@ -400,6 +394,5 @@ def targets(): PythonArtifact('windows', 'x64', 'Python37'), RubyArtifact('linux', 'x64'), RubyArtifact('macos', 'x64'), - PHPArtifact('linux', 'x64'), - PHPArtifact('macos', 'x64') + PHPArtifact('linux', 'x64') ]) |