From 50f812c612d7e49e1e6cfe17b68b0bbb8a1ff909 Mon Sep 17 00:00:00 2001
From: Todd Sedano <tsedano@pivotal.io>
Date: Fri, 23 Jun 2017 16:37:52 -0700
Subject: [PATCH] Updates readme with correct order of steps

- make distclean can not be called before configure
---
 README.md | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/README.md b/README.md
index 8d8a91e1ab..56ed1f69d2 100644
--- a/README.md
+++ b/README.md
@@ -50,10 +50,10 @@ building, see the README at the following repositories:
     ```
     git clone https://github.com/greenplum-db/gp-xerces
     mkdir gp-xerces/build
-    pushd gp-xerces/build
+    cd gp-xerces/build
     ../configure
     make install
-    popd
+    cd ../..
     ```
 
 1. ORCA requires [CMake](https://cmake.org) and
@@ -65,10 +65,10 @@ building, see the README at the following repositories:
     ```
     git clone https://github.com/greenplum-db/gporca
     mkdir gporca/build
-    pushd gporca/build
+    cd gporca/build
     cmake -GNinja ..
     ninja install
-    popd
+    cd ../..
     ```
     **Note**: Get the latest ORCA `git pull --ff-only` if you see an error message like below:
     ```
@@ -89,9 +89,6 @@ building, see the README at the following repositories:
     
 ### Build the database
 ```
-# Clean environment
-make distclean
-
 # Configure build environment to install at /usr/local/gpdb
 ./configure --with-perl --with-python --with-libxml --prefix=/usr/local/gpdb
 
@@ -139,6 +136,11 @@ To turn ORCA off and use legacy planner for query optimization:
 set optimizer=off;
 ```
 
+If you want to clean all generated files
+```
+make distclean
+```
+
 ## Running tests
 
 * The default regression tests
-- 
GitLab