<feed xmlns='http://www.w3.org/2005/Atom'>
<title>openembedded-core.git/scripts/send-pull-request, branch master</title>
<subtitle></subtitle>
<id>https://trygvis.io/git/2012/05/openembedded-core.git/atom?h=master</id>
<link rel='self' href='https://trygvis.io/git/2012/05/openembedded-core.git/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/'/>
<updated>2011-12-12T21:50:09Z</updated>
<entry>
<title>scripts/send-pull-request: fix typo</title>
<updated>2011-12-12T21:50:09Z</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-12-10T15:53:12Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=a82e5abf399f3ced7dcb4d5c543fdea084ef6c63'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>scripts/send-pull-request: allow sending of patches without a cover letter</title>
<updated>2011-12-09T17:46:25Z</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-12-07T21:19:52Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=326ef69d1296cd35a86d1eaee0741e6fc956af52'/>
<id>urn:sha1:326ef69d1296cd35a86d1eaee0741e6fc956af52</id>
<content type='text'>
Signed-off-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
</entry>
<entry>
<title>send-pull-request: default to --supress-cc=all</title>
<updated>2011-07-19T16:33:56Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-07-14T22:39:55Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=aaa85bd838f1c1518f8e65c36e265e6b9e141406'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>send-pull-request: fix a small typo that fails the script</title>
<updated>2011-06-02T13:50:52Z</updated>
<author>
<name>Dexuan Cui</name>
<email>dexuan.cui@intel.com</email>
</author>
<published>2011-06-02T07:56:18Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=8a1266d49b9c23bfe3fb454f34524869ac9725b2'/>
<id>urn:sha1:8a1266d49b9c23bfe3fb454f34524869ac9725b2</id>
<content type='text'>
Signed-off-by: Dexuan Cui &lt;dexuan.cui@intel.com&gt;
</content>
</entry>
<entry>
<title>send-pull-request: drop sendemail checks</title>
<updated>2011-05-24T06:39:09Z</updated>
<author>
<name>Otavio Salvador</name>
<email>otavio@ossystems.com.br</email>
</author>
<published>2011-05-20T20:36:33Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=b75cf390d75957f5b5718892942f76fc4295270f'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>*pull-request: add copyright, license, and descriptions</title>
<updated>2011-05-19T22:39:00Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-16T22:18:20Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=66b5a6bed6e55f2c285e2c3644302a6f242db32b'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>send-pull-request: streamline git-send-email usage</title>
<updated>2011-05-19T22:38:41Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-16T20:34:28Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=71286b32b58d4d1318b0a0a4b09ea65604d0e6fc'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>send-pull-request: verify git sendemail config</title>
<updated>2011-05-19T22:37:53Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-13T22:42:37Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=95185de6f149a495201701d260921339e3dfaebd'/>
<id>urn:sha1: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>
</entry>
<entry>
<title>send-pull-request: don't send all patches to everyone even with -a</title>
<updated>2011-05-19T22:37:27Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-13T20:58:36Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=90ef7136087f1a16da3c8fc2decbed27a5debcd8'/>
<id>urn:sha1:90ef7136087f1a16da3c8fc2decbed27a5debcd8</id>
<content type='text'>
Rather than sending every patch to every recipient of the entire series when
-a is used, only send the cover letter to everyone and use git's
--signed-off-by-cc feature to generate an auto cc list for the individual
patches.

Add a -c option to use --signed-off-by-cc to auto cc recipeients at the
individual patch level. This is implied by -a.

Using git to harvest the Cc list means only collecting Signed-off-by and Cc
lines, rather than the more generic *-by lines previously. This is a fair
trade-off for significantly reduced complexity. If users want to add Acked-by
and Tested-by lines and want to use the -a feature, they should include those
recipients as Cc lines as well.

Now that we rely on git for auto-cc for the individual patches,
make sure the user is prompted before sending each patch by forcing
--confirm=always.

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Khem Raj &lt;raj.khem@gmail.com&gt;
Cc: Koen Kooi &lt;koen@dominion.thruhere.net&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
</entry>
<entry>
<title>send-pull-request: fix greedy auto-cc regex</title>
<updated>2011-05-19T22:37:08Z</updated>
<author>
<name>Darren Hart</name>
<email>dvhart@linux.intel.com</email>
</author>
<published>2011-05-13T21:03:07Z</published>
<link rel='alternate' type='text/html' href='https://trygvis.io/git/2012/05/openembedded-core.git/commit/?id=e155cdedb0ac7ce8e0d6cb0ed5f22e9367f5e343'/>
<id>urn:sha1:e155cdedb0ac7ce8e0d6cb0ed5f22e9367f5e343</id>
<content type='text'>
A greedy regular expression caused emails to be harvested from patches
that were quoted in the commit message. Ensure only tags that start at the
beginning of the line are considered for harvesting.

NOTE: users are still responsible for verifying the recipients list and to
      ensure they do not spam people!

Signed-off-by: Darren Hart &lt;dvhart@linux.intel.com&gt;
Acked-by: Bruce Ashfield &lt;bruce.ashfield@windriver.com&gt;
Acked-by: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
Cc: Richard Purdie &lt;richard.purdie@linuxfoundation.org&gt;
Cc: Saul Wold &lt;sgw@linux.intel.com&gt;
Cc: Bruce Ashfield &lt;bruce.ashfield@windriver.com&gt;
Cc: Otavio Salvador &lt;otavio@ossystems.com.br&gt;
</content>
</entry>
</feed>
