kudu-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danburk...@apache.org
Subject incubator-kudu git commit: Cleanup from-source installation instructions
Date Fri, 22 Jan 2016 17:20:26 GMT
Repository: incubator-kudu
Updated Branches:
  refs/heads/master a65bbcc8e -> dda74081f


Cleanup from-source installation instructions

We were previously recommending setting both CMAKE_INSTALL_PREFIX and DESTDIR to
/opt/kudu, which would result in artifacts being installed into
/opt/kudu/opt/kudu. Additionally, we listed the installation step as optional,
but included it in the overall build script which skips optional steps.

Change-Id: Ib8d7d5e564d5b03a6f3bd092d8e1f2ad1aa17405
Reviewed-on: http://gerrit.cloudera.org:8080/1872
Reviewed-by: Misty Stanley-Jones <misty@apache.org>
Tested-by: Dan Burkert <dan@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/dda74081
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/dda74081
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/dda74081

Branch: refs/heads/master
Commit: dda74081f91b1086c85b4b9a2aaa9fc392c62c9c
Parents: a65bbcc
Author: Dan Burkert <dan@cloudera.com>
Authored: Thu Jan 21 23:33:07 2016 -0800
Committer: Dan Burkert <dan@cloudera.com>
Committed: Fri Jan 22 17:19:47 2016 +0000

----------------------------------------------------------------------
 docs/installation.adoc | 40 +++++++++++++++-------------------------
 1 file changed, 15 insertions(+), 25 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/dda74081/docs/installation.adoc
----------------------------------------------------------------------
diff --git a/docs/installation.adoc b/docs/installation.adoc
index a93f834..23577ec 100644
--- a/docs/installation.adoc
+++ b/docs/installation.adoc
@@ -249,9 +249,7 @@ $ build-support/enable_devtoolset.sh thirdparty/build-if-necessary.sh
 
 . Build Kudu, using the utilities installed in the previous step. Choose a build
 directory for the intermediate output, which can be anywhere in your filesystem
-except for the `kudu` directory itself. Edit the install prefix to the location
-where you would like the Kudu binaries, libraries, and headers installed during
-the `make install` step. The default value is `/usr/local/`.
+except for the `kudu` directory itself.
 +
 [source,bash]
 ----
@@ -260,13 +258,13 @@ cd build
 ../build-support/enable_devtoolset.sh \
   ../thirdparty/installed/bin/cmake \
   -DCMAKE_BUILD_TYPE=release \
-  -DCMAKE_INSTALL_PREFIX=/opt/kudu \
   ..
 make -j4
 ----
-[[build_install_kudu]]
+
 . Optional: Install Kudu binaries, libraries, and headers.
-If you do not specify a `DESTDIR`, `/usr/local/` is the default.
+If you do not specify an installation directory through the `DESTDIR`
+environment variable, `/usr/local/` is the default.
 +
 [source,bash]
 ----
@@ -307,7 +305,6 @@ cd build
   -DCMAKE_BUILD_TYPE=release \
   ..
 make -j4
-make install
 ----
 ====
 
@@ -345,23 +342,19 @@ $ thirdparty/build-if-necessary.sh
 
 . Build Kudu, using the utilities installed in the previous step. Choose a build
 directory for the intermediate output, which can be anywhere in your filesystem
-except for the `kudu` directory itself. Edit the install prefix to the location
-where you would like the Kudu binaries, libraries, and headers installed during
-the `make install` step. The default value is `/usr/local/`.
+except for the `kudu` directory itself.
 +
 [source,bash]
 ----
 mkdir -p build
 cd build
-../thirdparty/installed/bin/cmake \
-  -DCMAKE_BUILD_TYPE=release \
-  -DCMAKE_INSTALL_PREFIX=/opt/kudu \
-  ..
+../thirdparty/installed/bin/cmake -DCMAKE_BUILD_TYPE=release ..
 make -j4
 ----
-[[build_install_kudu]]
+
 . Optional: Install Kudu binaries, libraries, and headers.
-If you do not specify a `DESTDIR`, `/usr/local/` is the default.
+If you do not specify an installation directory through the `DESTDIR`
+environment variable, `/usr/local/` is the default.
 +
 [source,bash]
 ----
@@ -397,7 +390,6 @@ cd build
   -DCMAKE_BUILD_TYPE=release \
   ..
 make -j4
-make install
 ----
 ====
 
@@ -445,9 +437,7 @@ $ thirdparty/build-if-necessary.sh
 
 . Build Kudu, using the utilities installed in the previous step. Choose a build
 directory for the intermediate output, which can be anywhere in your filesystem
-except for the `kudu` directory itself. Edit the install prefix to the location
-where you would like the Kudu binaries, libraries, and headers installed during
-the `make install` step. The default value is `/usr/local/`.
+except for the `kudu` directory itself.
 +
 [source,bash]
 ----
@@ -456,13 +446,13 @@ cd build
 BOOST_ROOT=../../boost_1_59_0 \
   ../thirdparty/installed/bin/cmake \
   -DCMAKE_BUILD_TYPE=release \
-  -DCMAKE_INSTALL_PREFIX=/opt/kudu \
   ..
 make -j4
 ----
-[[build_install_kudu]]
+
 . Optional: Install Kudu binaries, libraries, and headers.
-If you do not specify a `DESTDIR`, `/usr/local/` is the default.
+If you do not specify an installation directory through the `DESTDIR`
+environment variable, `/usr/local/` is the default.
 +
 [source,bash]
 ----
@@ -472,7 +462,8 @@ sudo make DESTDIR=/opt/kudu install
 .SLES Build Script
 ====
 This script provides an overview of the procedure to build Kudu on SLES, and
-can be used as the basis for an automated deployment scenario.
+can be used as the basis for an automated deployment scenario. It skips
+the steps marked *Optional* above.
 
 [source,bash]
 ----
@@ -496,7 +487,6 @@ BOOST_ROOT=../../boost_1_59_0 \
   -DCMAKE_BUILD_TYPE=release \
   ..
 make -j4
-make install
 ----
 ====
 


Mime
View raw message