<feed xmlns='http://www.w3.org/2005/Atom'>
<title>openembedded-core.git/scripts/send-pull-request, branch jethro</title>
<subtitle>Mirror of openembedded-core</subtitle>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/'/>
<entry>
<title>send-pull-request: add extra CC argument</title>
<updated>2013-02-06T09:35:01+00:00</updated>
<author>
<name>Martin Jansa</name>
<email>martin.jansa@gmail.com</email>
</author>
<published>2013-02-04T15:06:59+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=52bc47756eb8a81ea07ef4bc06345ef335b30ceb'/>
<id>52bc47756eb8a81ea07ef4bc06345ef335b30ceb</id>
<content type='text'>
* useful e.g. when sending pull-request to release branch with extra CC
  for release maintainer

Signed-off-by: Martin Jansa &lt;Martin.Jansa@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
* useful e.g. when sending pull-request to release branch with extra CC
  for release maintainer

Signed-off-by: Martin Jansa &lt;Martin.Jansa@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: Make -a really imply -c</title>
<updated>2012-08-02T14:21:28+00:00</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2012-08-01T19:18:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=319d2cc8e60450106b665ee4172e2e6dc90e14b5'/>
<id>319d2cc8e60450106b665ee4172e2e6dc90e14b5</id>
<content type='text'>
The switch statement does not fall through as the comment suggests.
This results in -a not implying -c as was intended.

Add the two lines from -c to -a to achieve the intended behavior.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Signed-off-by: Richard Purdie &lt;richard.purdie@linuxfoundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The switch statement does not fall through as the comment suggests.
This results in -a not implying -c as was intended.

Add the two lines from -c to -a to achieve the intended behavior.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Signed-off-by: Richard Purdie &lt;richard.purdie@linuxfoundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>scripts/send-pull-request: fix typo</title>
<updated>2011-12-12T21:50:09+00:00</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-12-10T15:53:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=a82e5abf399f3ced7dcb4d5c543fdea084ef6c63'/>
<id>a82e5abf399f3ced7dcb4d5c543fdea084ef6c63</id>
<content type='text'>
s/No cover letter will be send./No cover letter will be sent./

Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
s/No cover letter will be send./No cover letter will be sent./

Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>scripts/send-pull-request: allow sending of patches without a cover letter</title>
<updated>2011-12-09T17:46:25+00:00</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-12-07T21:19:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=326ef69d1296cd35a86d1eaee0741e6fc956af52'/>
<id>326ef69d1296cd35a86d1eaee0741e6fc956af52</id>
<content type='text'>
Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: default to --supress-cc=all</title>
<updated>2011-07-19T16:33:56+00:00</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-07-14T22:39:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=aaa85bd838f1c1518f8e65c36e265e6b9e141406'/>
<id>aaa85bd838f1c1518f8e65c36e265e6b9e141406</id>
<content type='text'>
Unless the user specifies -c, don't pull in any email addresses from the
patches in the series. This avoids having to remove the email addresses from
patches being pulled in from upstream. If you want the email addresses on the
patch to be added, continue to use the -c option as before.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Unless the user specifies -c, don't pull in any email addresses from the
patches in the series. This avoids having to remove the email addresses from
patches being pulled in from upstream. If you want the email addresses on the
patch to be added, continue to use the -c option as before.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: fix a small typo that fails the script</title>
<updated>2011-06-02T13:50:52+00:00</updated>
<author>
<name>Dexuan Cui</name>
<email>dexuan.cui@intel.com</email>
</author>
<published>2011-06-02T07:56:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=8a1266d49b9c23bfe3fb454f34524869ac9725b2'/>
<id>8a1266d49b9c23bfe3fb454f34524869ac9725b2</id>
<content type='text'>
Signed-off-by: Dexuan Cui &lt;dexuan.cui@intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Dexuan Cui &lt;dexuan.cui@intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: drop sendemail checks</title>
<updated>2011-05-24T06:39:09+00:00</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-05-20T20:36:33+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=b75cf390d75957f5b5718892942f76fc4295270f'/>
<id>b75cf390d75957f5b5718892942f76fc4295270f</id>
<content type='text'>
git send-email has the correct check on it. Basically the From is
taken from the git 'user' and 'email' config values and in case
'sendemail.smtpserver' is not provided it defaults to use local
sendmail command.

Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Acked-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
git send-email has the correct check on it. Basically the From is
taken from the git 'user' and 'email' config values and in case
'sendemail.smtpserver' is not provided it defaults to use local
sendmail command.

Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Acked-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>*pull-request: add copyright, license, and descriptions</title>
<updated>2011-05-19T22:39:00+00:00</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-16T22:18:20+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=66b5a6bed6e55f2c285e2c3644302a6f242db32b'/>
<id>66b5a6bed6e55f2c285e2c3644302a6f242db32b</id>
<content type='text'>
Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: streamline git-send-email usage</title>
<updated>2011-05-19T22:38:41+00:00</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-16T20:34:28+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=71286b32b58d4d1318b0a0a4b09ea65604d0e6fc'/>
<id>71286b32b58d4d1318b0a0a4b09ea65604d0e6fc</id>
<content type='text'>
The script was sending one patch at a time, which defeats the internal
confirmation mechanism of git-send-email (which would otherwise allow
the user to send all patches or abort immediately).

Rework the sending logic to use no more than two commands. Use two
commands when the cover letter is to be sent to all recipients with
the -a argument. Otherwise, send all patches via the same command.

The script duplicates git's send confirmation, eliminate that.

Reported-by: Khem Raj &lt;raj.khem@gmail.com&gt;
Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Joshua Lock &lt;josh@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Khem Raj &lt;raj.khem@gmail.com&gt;
Cc: Joshua Lock &lt;josh@linux.intel.com&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The script was sending one patch at a time, which defeats the internal
confirmation mechanism of git-send-email (which would otherwise allow
the user to send all patches or abort immediately).

Rework the sending logic to use no more than two commands. Use two
commands when the cover letter is to be sent to all recipients with
the -a argument. Otherwise, send all patches via the same command.

The script duplicates git's send confirmation, eliminate that.

Reported-by: Khem Raj &lt;raj.khem@gmail.com&gt;
Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Joshua Lock &lt;josh@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Khem Raj &lt;raj.khem@gmail.com&gt;
Cc: Joshua Lock &lt;josh@linux.intel.com&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>send-pull-request: verify git sendemail config</title>
<updated>2011-05-19T22:37:53+00:00</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-13T22:42:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.multitech.net/cgit/openembedded-core.git/commit/?id=95185de6f149a495201701d260921339e3dfaebd'/>
<id>95185de6f149a495201701d260921339e3dfaebd</id>
<content type='text'>
Perform a quick sanity check to be able to direct users to configure
git.sendemail if they haven't yet.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Perform a quick sanity check to be able to direct users to configure
git.sendemail if they haven't yet.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</pre>
</div>
</content>
</entry>
</feed>
