diff options
author | Nitin A Kamble <nitin.a.kamble@intel.com> | 2010-06-08 22:04:59 -0700 |
---|---|---|
committer | Saul Wold <Saul.Wold@intel.com> | 2010-06-10 16:30:32 -0700 |
commit | 1d64687d13ff112c56e1f345a0b4c81a531aed19 (patch) | |
tree | 3bbdb1fdff2e6ff4f5c3ccb160c7e66132a4405b /scripts/create-pull-request | |
parent | 046ed7e77578085898c4c7f83e56c170c89fed81 (diff) | |
download | openembedded-core-contrib-1d64687d13ff112c56e1f345a0b4c81a531aed19.tar.gz |
update create_pull_request for distro/master
With this change the create_pull_request will be able to generate pull
requests to master as well as distro/master branch.
Some documentation is added in the Usage messange of the script.
Signed-off-by: Nitin A Kamble <nitin.a.kamble@intel.com>
Diffstat (limited to 'scripts/create-pull-request')
-rwxr-xr-x | scripts/create-pull-request | 77 |
1 files changed, 61 insertions, 16 deletions
diff --git a/scripts/create-pull-request b/scripts/create-pull-request index 22231519569..e8d4115e878 100755 --- a/scripts/create-pull-request +++ b/scripts/create-pull-request @@ -4,38 +4,83 @@ # usage() { - echo "Error: Invalid arguments." echo "Usage: " - echo "$ $0 <commit_id> <contrib_branch>" + echo "$ $0 [-r <relative_to>] [-i <commit_id>] -b <contrib_branch>" + echo " <relative_to> is a commit identifier, like branch-name, HEAD, hex-commit-id" echo " <commit_id> is a commit identifier, like branch-name, HEAD, hex-commit-id" echo " <contrib_branch> is the branch-name in the git.pokylinux.org/poky-contrib tree" + echo " If <relative_to> is not specified then relative to master is assumed" + echo " If <commit_id> is not specified then it is assumed as HEAD" + echo " For Example:" + echo " $0 -r master -i misc -b nitin/misc " + echo " $0 -b nitin/misc " + echo " $0 -r distro/master -i nitin/distro -b nitin/distro " exit 1 } -case $# in - 2) - COMMIT=$1 - CONTRIB_BRANCH=$2 - shift - ;; +while [ $# -ne 0 ] # loop over arguments +do + + case $1 in + -r ) + shift + RELATIVE_TO=$1 + shift + ;; + -i ) + shift + COMMIT_ID=$1 + shift + ;; + -b ) + shift + CONTRIB_BRANCH=$1 + shift + ;; *) + usage + ;; + esac +done + +if [ "${COMMIT_ID}" = "" ]; then + COMMIT_ID=HEAD + echo "Note: <commit_id> parameter assumed as 'HEAD'" +fi + +if [ "${RELATIVE_TO}" = "" ]; then + RELATIVE_TO=master + echo "Note: <relative_to> parameter assumed as 'master'" +fi + +if [ "${CONTRIB_BRANCH}" = "" ]; then + echo: "Error: Parameter <contrib_branch> not specified" usage - ;; -esac +fi -if [ "$COMMIT" = "" ]; then +git --no-pager show ${COMMIT_ID} > /dev/null +if [ "$?" != "0" ]; then + echo "Error: Invalid <commit_id> parameter specified" usage fi -git --no-pager show $COMMIT > /dev/null +git --no-pager show ${RELATIVE_TO} > /dev/null if [ "$?" != "0" ]; then - echo "Invalid Commit." + echo "Error: Invalid <relative_to> parameter specified: ${RELATIVE_TO}" usage fi -git --no-pager diff master..${COMMIT} | diffstat -p1 echo "" -git --no-pager log --no-merges master..${COMMIT} | git --no-pager shortlog +git --no-pager diff ${RELATIVE_TO}..${COMMIT_ID} | diffstat -p1 +echo "" +git --no-pager log --no-merges ${RELATIVE_TO}..${COMMIT_ID} | git --no-pager shortlog +PULL_URL="http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=${CONTRIB_BRANCH}" -echo "Pull URL: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=${CONTRIB_BRANCH}" +echo "Pull URL: ${PULL_URL}" + +wget -q ${PULL_URL} -O - | grep -q "Invalid branch:\ ${CONTRIB_BRANCH}" +if [ "$?" == "0" ]; then + echo "Warning: Branch named '${CONTRIB_BRANCH}' was not found on contrib git tree" + echo "Check your <contrib-branch> parameter" +fi |