diff options
author | Darren Hart <dvhart@linux.intel.com> | 2011-05-13 10:36:03 -0700 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2011-05-19 23:40:38 +0100 |
commit | b158757ef92f1e8bf62cf20bcb4d2bf2a212d6e0 (patch) | |
tree | 1590100433696b193b169672cc6b975d87f6b439 | |
parent | dca0c4846577205458615f1937290b954e77c28b (diff) | |
download | poky-b158757ef92f1e8bf62cf20bcb4d2bf2a212d6e0.tar.gz |
send-pull-request: whitespace cleanup
Indent with tabs, not spaces, for consistency with other bash scripts.
(From OE-Core rev: 187217ec380e89d81a53d9fd5f007d8f95356029)
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
Acked-by: Otavio Salvador <otavio@ossystems.com.br>
Cc: Otavio Salvador <otavio@ossystems.com.br>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-x | scripts/send-pull-request | 250 |
1 files changed, 125 insertions, 125 deletions
diff --git a/scripts/send-pull-request b/scripts/send-pull-request index 3af2a9fa7e..76dd7a2d91 100755 --- a/scripts/send-pull-request +++ b/scripts/send-pull-request | |||
@@ -31,72 +31,72 @@ EOM | |||
31 | # $2: The regex to match and strip from the line with email addresses | 31 | # $2: The regex to match and strip from the line with email addresses |
32 | harvest_recipients() | 32 | harvest_recipients() |
33 | { | 33 | { |
34 | TO_CC=$1 | 34 | TO_CC=$1 |
35 | REGX=$2 | 35 | REGX=$2 |
36 | export IFS=$',\n' | 36 | export IFS=$',\n' |
37 | for PATCH in $PDIR/*.patch; do | 37 | for PATCH in $PDIR/*.patch; do |
38 | # Grab To addresses | 38 | # Grab To addresses |
39 | for EMAIL in $(sed '/^---$/q' $PATCH | grep -e "$REGX" | sed "s/$REGX//"); do | 39 | for EMAIL in $(sed '/^---$/q' $PATCH | grep -e "$REGX" | sed "s/$REGX//"); do |
40 | if [ "$TO_CC" == "TO" ] && [ "${TO/$EMAIL/}" == "$TO" ] && [ -n "$EMAIL" ]; then | 40 | if [ "$TO_CC" == "TO" ] && [ "${TO/$EMAIL/}" == "$TO" ] && [ -n "$EMAIL" ]; then |
41 | if [ -z "$TO" ]; then TO=$EMAIL; else TO="$TO,$EMAIL"; fi | 41 | if [ -z "$TO" ]; then TO=$EMAIL; else TO="$TO,$EMAIL"; fi |
42 | elif [ "$TO_CC" == "CC" ] && [ "${CC/$EMAIL/}" == "$CC" ] && [ -n "$EMAIL" ]; then | 42 | elif [ "$TO_CC" == "CC" ] && [ "${CC/$EMAIL/}" == "$CC" ] && [ -n "$EMAIL" ]; then |
43 | if [ -z "$CC" ]; then CC=$EMAIL; else CC="$CC,$EMAIL"; fi | 43 | if [ -z "$CC" ]; then CC=$EMAIL; else CC="$CC,$EMAIL"; fi |
44 | fi | 44 | fi |
45 | done | 45 | done |
46 | done | 46 | done |
47 | unset IFS | 47 | unset IFS |
48 | } | 48 | } |
49 | 49 | ||
50 | 50 | ||
51 | # Parse and verify arguments | 51 | # Parse and verify arguments |
52 | while getopts "af:ghp:t:" OPT; do | 52 | while getopts "af:ghp:t:" OPT; do |
53 | case $OPT in | 53 | case $OPT in |
54 | a) | 54 | a) |
55 | AUTO=1 | 55 | AUTO=1 |
56 | ;; | 56 | ;; |
57 | f) | 57 | f) |
58 | FROM="$OPTARG" | 58 | FROM="$OPTARG" |
59 | ;; | 59 | ;; |
60 | g) | 60 | g) |
61 | PULL_MTA="git" | 61 | PULL_MTA="git" |
62 | ;; | 62 | ;; |
63 | h) | 63 | h) |
64 | usage | 64 | usage |
65 | exit 0 | 65 | exit 0 |
66 | ;; | 66 | ;; |
67 | p) | 67 | p) |
68 | PDIR=${OPTARG%/} | 68 | PDIR=${OPTARG%/} |
69 | if [ ! -d $PDIR ]; then | 69 | if [ ! -d $PDIR ]; then |
70 | echo "ERROR: pull-dir \"$PDIR\" does not exist." | 70 | echo "ERROR: pull-dir \"$PDIR\" does not exist." |
71 | usage | 71 | usage |
72 | exit 1 | 72 | exit 1 |
73 | fi | 73 | fi |
74 | ;; | 74 | ;; |
75 | t) | 75 | t) |
76 | if [ -n "$TO" ]; then | 76 | if [ -n "$TO" ]; then |
77 | TO="$TO,$OPTARG" | 77 | TO="$TO,$OPTARG" |
78 | else | 78 | else |
79 | TO="$OPTARG" | 79 | TO="$OPTARG" |
80 | fi | 80 | fi |
81 | ;; | 81 | ;; |
82 | esac | 82 | esac |
83 | done | 83 | done |
84 | 84 | ||
85 | if [ -z "$PDIR" ]; then | 85 | if [ -z "$PDIR" ]; then |
86 | echo "ERROR: you must specify a pull-dir." | 86 | echo "ERROR: you must specify a pull-dir." |
87 | usage | 87 | usage |
88 | exit 1 | 88 | exit 1 |
89 | fi | 89 | fi |
90 | 90 | ||
91 | 91 | ||
92 | # Verify the cover letter is complete and free of tokens | 92 | # Verify the cover letter is complete and free of tokens |
93 | CL="$PDIR/0000-cover-letter.patch" | 93 | CL="$PDIR/0000-cover-letter.patch" |
94 | for TOKEN in SUBJECT BLURB; do | 94 | for TOKEN in SUBJECT BLURB; do |
95 | grep -q "*** $TOKEN HERE ***" "$CL" | 95 | grep -q "*** $TOKEN HERE ***" "$CL" |
96 | if [ $? -eq 0 ]; then | 96 | if [ $? -eq 0 ]; then |
97 | echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')." | 97 | echo "ERROR: Please edit $CL and try again (Look for '*** $TOKEN HERE ***')." |
98 | exit 1 | 98 | exit 1 |
99 | fi | 99 | fi |
100 | done | 100 | done |
101 | 101 | ||
102 | 102 | ||
@@ -104,39 +104,39 @@ done | |||
104 | # In addition to To and CC headers/lines, the common Signed-off-by, Tested-by, | 104 | # In addition to To and CC headers/lines, the common Signed-off-by, Tested-by, |
105 | # etc. (*-by) will be added to CC. | 105 | # etc. (*-by) will be added to CC. |
106 | if [ $AUTO -eq 1 ]; then | 106 | if [ $AUTO -eq 1 ]; then |
107 | harvest_recipients TO "^[Tt][Oo]: *" | 107 | harvest_recipients TO "^[Tt][Oo]: *" |
108 | harvest_recipients CC "^[Cc][Cc]: *" | 108 | harvest_recipients CC "^[Cc][Cc]: *" |
109 | harvest_recipients CC "^.*-[Bb][Yy]: *" | 109 | harvest_recipients CC "^.*-[Bb][Yy]: *" |
110 | fi | 110 | fi |
111 | 111 | ||
112 | case "$PULL_MTA" in | 112 | case "$PULL_MTA" in |
113 | git) | 113 | git) |
114 | FROM="$(git config sendemail.from)" | 114 | FROM="$(git config sendemail.from)" |
115 | AUTO_TO="$(git config sendemail.to)" | 115 | AUTO_TO="$(git config sendemail.to)" |
116 | if [ -n "$AUTO_TO" ]; then | 116 | if [ -n "$AUTO_TO" ]; then |
117 | if [ -n "$TO" ]; then | 117 | if [ -n "$TO" ]; then |
118 | TO="$TO,$AUTO_TO" | 118 | TO="$TO,$AUTO_TO" |
119 | else | 119 | else |
120 | TO="$AUTO_TO" | 120 | TO="$AUTO_TO" |
121 | fi | 121 | fi |
122 | fi | ||
123 | ;; | ||
124 | sendmail) | ||
125 | if [ -z "$FROM" ]; then | ||
126 | FROM="$(git config user.name) <$(git config user.email)>" | ||
127 | if [ -z "$FROM" ]; then | ||
128 | echo "ERROR: unable to determine a FROM address" | ||
129 | usage | ||
130 | exit 1 | ||
131 | fi | ||
122 | fi | 132 | fi |
123 | ;; | 133 | ;; |
124 | sendmail) | ||
125 | if [ -z "$FROM" ]; then | ||
126 | FROM="$(git config user.name) <$(git config user.email)>" | ||
127 | if [ -z "$FROM" ]; then | ||
128 | echo "ERROR: unable to determine a FROM address" | ||
129 | usage | ||
130 | exit 1 | ||
131 | fi | ||
132 | fi | ||
133 | ;; | ||
134 | esac | 134 | esac |
135 | 135 | ||
136 | if [ -z "$TO" ] && [ -z "$CC" ]; then | 136 | if [ -z "$TO" ] && [ -z "$CC" ]; then |
137 | echo "ERROR: you have not specified any recipients." | 137 | echo "ERROR: you have not specified any recipients." |
138 | usage | 138 | usage |
139 | exit 1 | 139 | exit 1 |
140 | fi | 140 | fi |
141 | 141 | ||
142 | 142 | ||
@@ -155,55 +155,55 @@ echo "Continue? [y/N] " | |||
155 | read cont | 155 | read cont |
156 | 156 | ||
157 | if [ "$cont" == "y" ] || [ "$cont" == "Y" ]; then | 157 | if [ "$cont" == "y" ] || [ "$cont" == "Y" ]; then |
158 | ERROR=0 | 158 | ERROR=0 |
159 | case "$PULL_MTA" in | 159 | case "$PULL_MTA" in |
160 | git) | 160 | git) |
161 | export IFS=$',' | 161 | export IFS=$',' |
162 | GIT_TO=$(for R in $TO; do echo -n "--to='$R' "; done) | 162 | GIT_TO=$(for R in $TO; do echo -n "--to='$R' "; done) |
163 | GIT_CC=$(for R in $CC; do echo -n "--cc='$R' "; done) | 163 | GIT_CC=$(for R in $CC; do echo -n "--cc='$R' "; done) |
164 | unset IFS | 164 | unset IFS |
165 | for PATCH in $PDIR/*patch; do | 165 | for PATCH in $PDIR/*patch; do |
166 | # We harvest the emails manually, so force git not to. | 166 | # We harvest the emails manually, so force git not to. |
167 | eval "git send-email $GIT_TO $GIT_CC --no-chain-reply-to --suppress-cc=all $PATCH" | 167 | eval "git send-email $GIT_TO $GIT_CC --no-chain-reply-to --suppress-cc=all $PATCH" |
168 | if [ $? -eq 1 ]; then | 168 | if [ $? -eq 1 ]; then |
169 | ERROR=1 | 169 | ERROR=1 |
170 | fi | 170 | fi |
171 | done | 171 | done |
172 | ;; | 172 | ;; |
173 | sendmail) | 173 | sendmail) |
174 | for PATCH in $PDIR/*patch; do | 174 | for PATCH in $PDIR/*patch; do |
175 | # Insert To and CC headers via formail to keep them separate and | 175 | # Insert To and CC headers via formail to keep them separate and |
176 | # appending them to the sendmail command as -- $TO $CC has | 176 | # appending them to the sendmail command as -- $TO $CC has |
177 | # proven to be an exercise in futility. | 177 | # proven to be an exercise in futility. |
178 | # | 178 | # |
179 | # Clear the From header, leaving it up to sendmail to insert an | 179 | # Clear the From header, leaving it up to sendmail to insert an |
180 | # appropriate one. Insert the original sender (per git) into the | 180 | # appropriate one. Insert the original sender (per git) into the |
181 | # body of the message. | 181 | # body of the message. |
182 | # | 182 | # |
183 | # Use tail to remove the email envelope from git or formail as | 183 | # Use tail to remove the email envelope from git or formail as |
184 | # msmtp (sendmail) would choke on them. | 184 | # msmtp (sendmail) would choke on them. |
185 | # | 185 | # |
186 | # Modify the patch date for sequential delivery, but retain the | 186 | # Modify the patch date for sequential delivery, but retain the |
187 | # original date as "Old-Date". | 187 | # original date as "Old-Date". |
188 | DATE=$(date +"%a, %d %b %Y %k:%M:%S %z") | 188 | DATE=$(date +"%a, %d %b %Y %k:%M:%S %z") |
189 | GIT_FROM=$(cat $PATCH | formail -X "From:") | 189 | GIT_FROM=$(cat $PATCH | formail -X "From:") |
190 | cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -I "From: $FROM" -i "Date: $DATE" | sed "0,/^$/s/^$/\n$GIT_FROM\n/" | tail -n +2 | sendmail -t | 190 | cat $PATCH | formail -I "To: $TO" -I "CC: $CC" -I "From: $FROM" -i "Date: $DATE" | sed "0,/^$/s/^$/\n$GIT_FROM\n/" | tail -n +2 | sendmail -t |
191 | if [ $? -eq 1 ]; then | 191 | if [ $? -eq 1 ]; then |
192 | ERROR=1 | 192 | ERROR=1 |
193 | fi | 193 | fi |
194 | done | 194 | done |
195 | ;; | 195 | ;; |
196 | *) | 196 | *) |
197 | echo "ERROR: unknown MTA: $PULL_MTA" | 197 | echo "ERROR: unknown MTA: $PULL_MTA" |
198 | usage | 198 | usage |
199 | exit 1 | 199 | exit 1 |
200 | ;; | 200 | ;; |
201 | esac | 201 | esac |
202 | 202 | ||
203 | if [ $ERROR -eq 1 ]; then | 203 | if [ $ERROR -eq 1 ]; then |
204 | echo "ERROR: Failed to send one or more messages. Check your MTA log for details." | 204 | echo "ERROR: Failed to send one or more messages. Check your MTA log for details." |
205 | fi | 205 | fi |
206 | else | 206 | else |
207 | echo "Send aborted." | 207 | echo "Send aborted." |
208 | fi | 208 | fi |
209 | 209 | ||