Github user njayaram2 commented on a diff in the pull request:
https://github.com/apache/madlib/pull/227#discussion_r162219624
--- Diff: tool/docker/base/Dockerfile_postgres_9_6 ---
@@ -1,71 +0,0 @@
-#
-# Licensed to the Apache Software Foundation (ASF) under one
-# or more contributor license agreements. See the NOTICE file
-# distributed with this work for additional information
-# regarding copyright ownership. The ASF licenses this file
-# to you 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.
-
-FROM postgres:9.6
-
-### Get postgres specific add-ons
-RUN apt-get update && apt-get install -y wget \
- build-essential \
- postgresql-server-dev-9.6 \
- postgresql-plpython-9.6 \
- openssl \
- libssl-dev \
- libboost-all-dev \
- m4 \
- wget \
- vim \
- pgxnclient \
- flex \
- bison \
- graphviz
--- End diff --
We should keep `flex`, `bison`, and `graphviz` in
`Dockerfile_centos_7_postgres_9_6` to help developers build docs using the
docker image.
---
