Github user jeking3 commented on a diff in the pull request:
https://github.com/apache/thrift/pull/1451#discussion_r161777001
--- Diff: build/docker/ubuntu-trusty/Dockerfile ---
@@ -1,223 +0,0 @@
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-
-#
-# Apache Thrift Docker build environment for Ubuntu Trusty
-# Using all stock Ubuntu Trusty packaging except for:
-# - d: does not come with Ubuntu so we're installing 2.070.0
-# - dart: does not come with Ubuntu so we're installing 1.20.1
-# - dotnetcore, disabled because netcore is for 1.0.0-preview and 2.0.0 is
out
-# - haxe, disabled because the distro comes with 3.0.0 and it cores while
installing
-# - node.js, disabled because it is at 0.10.0 in the distro which is too
old (need 4+)
-# - ocaml, disabled because it fails to install properly
-#
--- End diff --
Recommend adding that php is disabled because we require 5.6 and Trusty
comes with 5.5.9, and comment out the PHP lines, but not delete the whole thing.
---
