From 6d80ff135458f168ad86f21d48d6b295c69826cf Mon Sep 17 00:00:00 2001
From: Loic Dachary <ldachary@redhat.com>
Date: Wed, 2 Sep 2015 16:00:10 +0200
Subject: [PATCH] tools: fix do_autogen.sh -R

The R letter was missing from the getopts flags. Also sort the flags
alphabetically to make it easier to spot that kind of lossage.

Signed-off-by: Loic Dachary <ldachary@redhat.com>
---
 do_autogen.sh | 55 +++++++++++++++++++--------------------------------
 1 file changed, 20 insertions(+), 35 deletions(-)

diff --git a/do_autogen.sh b/do_autogen.sh
index febdfcb1705bb..9a4505b98372b 100755
--- a/do_autogen.sh
+++ b/do_autogen.sh
@@ -4,23 +4,24 @@ usage() {
     cat <<EOF
 do_autogen.sh: make a ceph build by running autogen, etc.
 
--h:                              this help message
+-C <parameter>                   add parameters to configure
+-c                               use cryptopp
 -d <level>                       debug build
                                  level 0: no debug
                                  level 1: -g
                                  level 3: -Wextra
                                  level 4: even more...
--T                               --without-tcmalloc
 -e <path>                        dump encoded objects to <path>
--P                               profiling build
--p                               google profiler
--O <level>                       optimize
--c                               use cryptopp
--C <parameter>                   add parameters to configure
+-h                               this help message
 -j                               with java
--R                               without rocksdb
 -J				 --with-jemalloc
 -L				 --without-lttng
+-O <level>                       optimize
+-p                               google profiler
+-P                               profiling build
+-R                               without rocksdb
+-T                               --without-tcmalloc
+-v                               verbose output
 
 EOF
 }
@@ -35,41 +36,25 @@ verbose=0
 profile=0
 rocksdb=1
 CONFIGURE_FLAGS="--disable-static --with-lttng"
-while getopts  "d:e:hHrTPJLjpcvO:C:" flag
+while getopts  "C:cd:e:hjJLO:pPRTv" flag
 do
     case $flag in
+    C) CONFIGURE_FLAGS="$CONFIGURE_FLAGS $OPTARG";;
+    c) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --with-cryptopp --without-nss";;
     d) debug_level=$OPTARG;;
-
+    e) encode_dump=$OPTARG;;
+    h) usage ; exit 0;;
+    j) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --enable-cephfs-java";;
+    J) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --with-jemalloc";;
+    L) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --without-lttng";;
     O) CFLAGS="${CFLAGS} -O$OPTARG";;
-
-    c) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --with-cryptopp --without-nss";;
-
-    C) CONFIGURE_FLAGS="$CONFIGURE_FLAGS $OPTARG";;
-
-    P) profile=1;;
     p) with_profiler="--with-profiler" ;;
-
-    h) usage
-        exit 0;;
-
-    T) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --without-tcmalloc";;
-
-    j) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --enable-cephfs-java";;
-
+    P) profile=1;;
     R) rocksdb=0;;
-
+    T) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --without-tcmalloc";;
     v) verbose=1;;
 
-    e) encode_dump=$OPTARG;;
-
-    J) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --with-jemalloc";;
-
-    L) CONFIGURE_FLAGS="$CONFIGURE_FLAGS --without-lttng";;
-
-    *)
-        echo
-        usage
-        exit 1;;
+    *) echo ; usage ; exit 1;;
     esac
 done
 
-- 
2.39.5