diff options
author | Ed Bartosh <ed.bartosh@linux.intel.com> | 2017-05-17 13:14:56 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-05-23 17:45:35 +0100 |
commit | 7e23b174e5fd6fe4a8945e1e276cedd9a90888df (patch) | |
tree | e742a18db1b18096883feb99be9da9e50163153d /scripts/create-pull-request | |
parent | c523b2ba27472479fd7a0f17957ca3e08d7021f3 (diff) | |
download | poky-7e23b174e5fd6fe4a8945e1e276cedd9a90888df.tar.gz |
Revert "create-pull-request: add "-t in-reply-to" option"
Rasons:
- It breaks the script if script is used without -t
- Its functionality covered by the next patch
This reverts commit 3ad3fda6c5f084f4fa1485b30aa333287989bee7.
(From OE-Core rev: 9be68e6f058e2ab149e5c6f045479413cf0db69b)
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/create-pull-request')
-rwxr-xr-x | scripts/create-pull-request | 14 |
1 files changed, 3 insertions, 11 deletions
diff --git a/scripts/create-pull-request b/scripts/create-pull-request index 46d65386a3..e82858bc98 100755 --- a/scripts/create-pull-request +++ b/scripts/create-pull-request | |||
@@ -34,7 +34,7 @@ RFC=0 | |||
34 | usage() { | 34 | usage() { |
35 | CMD=$(basename $0) | 35 | CMD=$(basename $0) |
36 | cat <<EOM | 36 | cat <<EOM |
37 | Usage: $CMD [-h] [-o output_dir] [-m msg_body_file] [-s subject] [-r relative_to] [-i commit_id] [-d relative_dir] -u remote [-b branch] [-t in_reply_to] | 37 | Usage: $CMD [-h] [-o output_dir] [-m msg_body_file] [-s subject] [-r relative_to] [-i commit_id] [-d relative_dir] -u remote [-b branch] |
38 | -b branch Branch name in the specified remote (default: current branch) | 38 | -b branch Branch name in the specified remote (default: current branch) |
39 | -l local branch Local branch name (default: HEAD) | 39 | -l local branch Local branch name (default: HEAD) |
40 | -c Create an RFC (Request for Comment) patch series | 40 | -c Create an RFC (Request for Comment) patch series |
@@ -49,7 +49,6 @@ Usage: $CMD [-h] [-o output_dir] [-m msg_body_file] [-s subject] [-r relative_to | |||
49 | -s subject The subject to be inserted into the summary email | 49 | -s subject The subject to be inserted into the summary email |
50 | -u remote The git remote where the branch is located, or set CPR_CONTRIB_REMOTE in env | 50 | -u remote The git remote where the branch is located, or set CPR_CONTRIB_REMOTE in env |
51 | -d relative_dir Generate patches relative to directory | 51 | -d relative_dir Generate patches relative to directory |
52 | -t in_reply_to Make mails appear as replies to the given Message-Id, to continue patch/series threads | ||
53 | 52 | ||
54 | Examples: | 53 | Examples: |
55 | $CMD -u contrib -b nitin/basic | 54 | $CMD -u contrib -b nitin/basic |
@@ -58,13 +57,12 @@ Usage: $CMD [-h] [-o output_dir] [-m msg_body_file] [-s subject] [-r relative_to | |||
58 | $CMD -u contrib -r master -i misc -b nitin/misc -o pull-misc | 57 | $CMD -u contrib -r master -i misc -b nitin/misc -o pull-misc |
59 | $CMD -u contrib -p "RFC PATCH" -b nitin/experimental | 58 | $CMD -u contrib -p "RFC PATCH" -b nitin/experimental |
60 | $CMD -u contrib -i misc -b nitin/misc -d ./bitbake | 59 | $CMD -u contrib -i misc -b nitin/misc -d ./bitbake |
61 | $CMD -u contrib -p "OE-core][PATCH v2" -t "<cover.11146.git.john.doe@example.com>" | ||
62 | EOM | 60 | EOM |
63 | } | 61 | } |
64 | 62 | ||
65 | REMOTE="$CPR_CONTRIB_REMOTE" | 63 | REMOTE="$CPR_CONTRIB_REMOTE" |
66 | # Parse and validate arguments | 64 | # Parse and validate arguments |
67 | while getopts "b:acd:hi:m:o:p:r:s:u:l:t:" OPT; do | 65 | while getopts "b:acd:hi:m:o:p:r:s:u:l:" OPT; do |
68 | case $OPT in | 66 | case $OPT in |
69 | b) | 67 | b) |
70 | BRANCH="$OPTARG" | 68 | BRANCH="$OPTARG" |
@@ -110,8 +108,6 @@ while getopts "b:acd:hi:m:o:p:r:s:u:l:t:" OPT; do | |||
110 | a) | 108 | a) |
111 | CPR_CONTRIB_AUTO_PUSH="1" | 109 | CPR_CONTRIB_AUTO_PUSH="1" |
112 | ;; | 110 | ;; |
113 | t) | ||
114 | IN_REPLY_TO="$OPTARG" | ||
115 | esac | 111 | esac |
116 | done | 112 | done |
117 | 113 | ||
@@ -209,11 +205,7 @@ if [ -n "$RELDIR" ]; then | |||
209 | fi | 205 | fi |
210 | 206 | ||
211 | # Generate the patches and cover letter | 207 | # Generate the patches and cover letter |
212 | if [ -z "$IN_REPLY_TO" ]; then | 208 | git format-patch $extraopts -M40 --subject-prefix="$PREFIX" -n -o $ODIR --thread=shallow --cover-letter $RELATIVE_TO..$COMMIT_ID > /dev/null |
213 | git format-patch $extraopts -M40 --subject-prefix="$PREFIX" -n -o $ODIR --thread=shallow --in-reply-to="$IN_REPLY_TO" --cover-letter $RELATIVE_TO..$COMMIT_ID > /dev/null | ||
214 | else | ||
215 | git format-patch $extraopts -M40 --subject-prefix="$PREFIX" -n -o $ODIR --thread=shallow --cover-letter $RELATIVE_TO..$COMMIT_ID > /dev/null | ||
216 | fi | ||
217 | 209 | ||
218 | if [ -z "$(ls -A $ODIR 2> /dev/null)" ]; then | 210 | if [ -z "$(ls -A $ODIR 2> /dev/null)" ]; then |
219 | echo "ERROR: $ODIR is empty, no cover letter and patches was generated!" | 211 | echo "ERROR: $ODIR is empty, no cover letter and patches was generated!" |