Refactor package.sh:

* lint
 * adjust exit codes (1 for usage, 2 for maven, 3 for rpmbuild)
 * variable naming consistency
 * add option for package release version
 * revise synopsis and usage
This commit is contained in:
David Amorim Faria 2015-11-16 16:02:46 +01:00
parent 62f57d2355
commit e66dd8c1e0

View File

@ -18,26 +18,37 @@
function usage() { function usage() {
echo "" echo ""
echo "usage: ./package.sh [-p|--pack] [-h|--help] [ARGS]" echo "usage: ./package.sh [-h|--help] -d|--distribution <name> [-r|--release <version>] [-p|--pack oss|OSS|noredist|NOREDIST] [-s|--simulator default|DEFAULT|simulator|SIMULATOR]"
echo "" echo ""
echo "The commonly used Arguments are:" echo "The supported arguments are:"
echo "-p|--pack oss|OSS To package with only redistributable libraries (default)" echo " To package with only redistributable libraries (default)"
echo "-p|--pack noredist|NOREDIST To package with non-redistributable libraries" echo " -p|--pack oss|OSS"
echo "-d centos7|centos63|fedora20|fedora21 To build a package for a distribution" echo " To package with non-redistributable libraries"
echo "-s simulator|SIMULATOR To build for Simulator" echo " -p|--pack noredist|NOREDIST"
echo " To build a package for a distribution (mandatory)"
echo " -d|--distribution centos7|centos63|fedora20|fedora21"
echo " To set the package release version (optional)"
echo " (default is 1 for normal and prereleases, empty for SNAPSHOT)"
echo " -r|--release version(integer)"
echo " To build for Simulator (optional)"
echo " -s|--simulator default|DEFAULT|simulator|SIMULATOR"
echo " To display this information"
echo " -h|--help"
echo "" echo ""
echo "Examples: ./package.sh -p|--pack oss|OSS" echo "Examples: ./package.sh --pack oss"
echo " ./package.sh -p|--pack noredist|NOREDIST" echo " ./package.sh --pack noredist"
echo " ./package.sh (Default OSS)" echo " ./package.sh --pack oss --distribution centos7 --release 42"
exit 1 echo " ./package.sh --distribution centos7 --release 42"
echo " ./package.sh --distribution centos7"
} }
# packaging # packaging
# $1 redist flag # $1 redist flag
# $2 simulator flag # $2 simulator flag
# $3 distribution name # $3 distribution name
# $4 package release version
function packaging() { function packaging() {
CWD=`pwd` CWD=$(pwd)
RPMDIR=$CWD/../dist/rpmbuild RPMDIR=$CWD/../dist/rpmbuild
PACK_PROJECT=cloudstack PACK_PROJECT=cloudstack
if [ -n "$1" ] ; then if [ -n "$1" ] ; then
@ -48,116 +59,127 @@ function packaging() {
fi fi
DISTRO=$3 DISTRO=$3
MVN=`which mvn` MVN=$(which mvn)
if [ -z "$MVN" ] ; then if [ -z "$MVN" ] ; then
MVN=`locate bin/mvn | grep -e mvn$ | tail -1` MVN=$(locate bin/mvn | grep -e mvn$ | tail -1)
if [ -z "$MVN" ] ; then if [ -z "$MVN" ] ; then
echo "mvn not found\n cannot retrieve version to package\n RPM Build Failed" echo -e "mvn not found\n cannot retrieve version to package\n RPM Build Failed"
exit 2 exit 2
fi fi
fi fi
VERSION=`(cd ../; $MVN org.apache.maven.plugins:maven-help-plugin:2.1.1:evaluate -Dexpression=project.version) | grep --color=none '^[0-9]\.'` VERSION=$(cd ../; $MVN org.apache.maven.plugins:maven-help-plugin:2.1.1:evaluate -Dexpression=project.version | grep --color=none '^[0-9]\.')
if echo $VERSION | grep -q SNAPSHOT ; then if echo "$VERSION" | grep -q SNAPSHOT ; then
REALVER=`echo $VERSION | cut -d '-' -f 1` REALVER=$(echo "$VERSION" | cut -d '-' -f 1)
DEFVER="-D_ver $REALVER" if [ -n "$4" ] ; then
DEFPRE="-D_prerelease 1" DEFPRE="-D_prerelease $4"
DEFREL="-D_rel SNAPSHOT" DEFREL="-D_rel SNAPSHOT$4"
else
DEFPRE="-D_prerelease 1"
DEFREL="-D_rel SNAPSHOT"
fi
else else
REALVER=`echo $VERSION` REALVER="$VERSION"
DEFVER="-D_ver $REALVER" if [ -n "$4" ] ; then
DEFREL="-D_rel 1" DEFREL="-D_rel $4"
else
DEFREL="-D_rel 1"
fi
fi fi
DEFVER="-D_ver $REALVER"
echo Preparing to package Apache CloudStack ${VERSION} echo "Preparing to package Apache CloudStack $VERSION"
mkdir -p $RPMDIR/SPECS mkdir -p "$RPMDIR/SPECS"
mkdir -p $RPMDIR/BUILD mkdir -p "$RPMDIR/BUILD"
mkdir -p $RPMDIR/RPMS mkdir -p "$RPMDIR/RPMS"
mkdir -p $RPMDIR/SRPMS mkdir -p "$RPMDIR/SRPMS"
mkdir -p $RPMDIR/SOURCES/$PACK_PROJECT-$VERSION mkdir -p "$RPMDIR/SOURCES/$PACK_PROJECT-$VERSION"
echo ". preparing source tarball" echo ". preparing source tarball"
(cd ../; tar -c --exclude .git --exclude dist . | tar -C $RPMDIR/SOURCES/$PACK_PROJECT-$VERSION -x ) (cd ../; tar -c --exclude .git --exclude dist . | tar -C "$RPMDIR/SOURCES/$PACK_PROJECT-$VERSION" -x )
(cd $RPMDIR/SOURCES/; tar -czf $PACK_PROJECT-$VERSION.tgz $PACK_PROJECT-$VERSION) (cd "$RPMDIR/SOURCES/"; tar -czf "$PACK_PROJECT-$VERSION.tgz" "$PACK_PROJECT-$VERSION")
echo ". executing rpmbuild" echo ". executing rpmbuild"
cp $DISTRO/cloud.spec $RPMDIR/SPECS cp "$DISTRO/cloud.spec" "$RPMDIR/SPECS"
(cd $RPMDIR; rpmbuild --define "_topdir $RPMDIR" "${DEFVER}" "${DEFREL}" ${DEFPRE+"${DEFPRE}"} ${DEFOSSNOSS+"$DEFOSSNOSS"} ${DEFSIM+"$DEFSIM"} -bb SPECS/cloud.spec)
(cd "$RPMDIR"; rpmbuild --define "_topdir ${RPMDIR}" "${DEFVER}" "${DEFREL}" ${DEFPRE+"$DEFPRE"} ${DEFOSSNOSS+$DEFOSSNOSS} ${DEFSIM+"$DEFSIM"} -bb SPECS/cloud.spec)
if [ $? -ne 0 ]; then if [ $? -ne 0 ]; then
echo "RPM Build Failed " echo "RPM Build Failed "
exit 1 exit 3
else else
echo "RPM Build Done" echo "RPM Build Done"
fi fi
exit exit
} }
TARGETDISTRO="" TARGETDISTRO=""
sim="" SIM=""
packageval="" PACKAGEVAL=""
RELEASE=""
SHORTOPTS="hp:d:" SHORTOPTS="hp:s:d:r:"
LONGOPTS="help,pack:,simulator:distribution" LONGOPTS="help,pack:simulator:distribution:release:"
ARGS=$(getopt -s bash -u -a --options $SHORTOPTS --longoptions $LONGOPTS --name $0 -- "$@") ARGS=$(getopt -s bash -u -a --options "$SHORTOPTS" --longoptions "$LONGOPTS" --name "$0" -- "$@")
eval set -- "$ARGS" eval set -- "$ARGS"
echo "$ARGS" echo "$ARGS"
while [ $# -gt 0 ] ; do while [ $# -gt 0 ] ; do
case "$1" in case "$1" in
-h | --help) -h | --help)
usage usage
exit 0 exit 0
;; ;;
-p | --pack) -p | --pack)
echo "Doing CloudStack Packaging ....." echo "Packaging CloudStack..."
packageval=$2 PACKAGEVAL=$2
echo "$packageval" echo "$PACKAGEVAL"
if [ "$packageval" == "oss" -o "$packageval" == "OSS" ] ; then if [ "$PACKAGEVAL" == "oss" -o "$PACKAGEVAL" == "OSS" ] ; then
packageval="" PACKAGEVAL=""
elif [ "$packageval" == "noredist" -o "$packageval" == "NOREDIST" ] ; then elif [ "$PACKAGEVAL" == "noredist" -o "$PACKAGEVAL" == "NOREDIST" ] ; then
packageval="noredist" PACKAGEVAL="noredist"
else else
echo "Error: Incorrect value provided in package.sh script, Please see help ./package.sh --help|-h for more details." echo "Error: Unsupported value for --pack"
usage
exit 1 exit 1
fi fi
shift shift
;; ;;
-s | --simulator) -s | --simulator)
sim=$2 SIM=$2
echo "$sim" echo "$SIM"
if [ "$sim" == "default" -o "$sim" == "DEFAULT" ] ; then if [ "$SIM" == "default" -o "$SIM" == "DEFAULT" ] ; then
sim = "false" SIM="false"
elif [ "$sim" == "simulator" -o "$sim" == "SIMULATOR" ] ; then elif [ "$SIM" == "simulator" -o "$SIM" == "SIMULATOR" ] ; then
sim="simulator" SIM="simulator"
else else
echo "Error: Incorrect value provided in package.sh script for -o, Please see help ./package.sh --help|-h for more details." echo "Error: Unsupported value for --simulator"
exit 1 usage
exit 1
fi fi
shift shift
;; ;;
-d | --distribution) -d | --distribution)
TARGETDISTRO=$2 TARGETDISTRO=$2
if [ -z "$TARGETDISTRO" ] ; then
echo "Error: Missing target distribution"
usage
exit 1
fi
shift
;;
-r | --release)
RELEASE=$2
shift shift
;; ;;
-) -)
echo "Unrecognized option..." echo "Error: Unrecognized option"
usage usage
exit 1 exit 1
;; ;;
*) *)
shift shift
;; ;;
esac esac
done done
if [ -z "$TARGETDISTRO" ] packaging "$PACKAGEVAL" "$SIM" "$TARGETDISTRO" "$RELEASE"
then
echo "Missing target distribution"
usage
exit 1
fi
packaging "$packageval" "$sim" "$TARGETDISTRO"