<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/t/t7502-commit.sh, branch v2.0.2</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.0.2</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.0.2'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2014-02-25T17:35:20Z</updated>
<entry>
<title>commit: add --cleanup=scissors</title>
<updated>2014-02-25T17:35:20Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2014-02-17T12:15:32Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=75df1f434f8cfdac1c8eeacae259ab375c01385b'/>
<id>urn:sha1:75df1f434f8cfdac1c8eeacae259ab375c01385b</id>
<content type='text'>
Since 1a72cfd (commit -v: strip diffs and submodule shortlogs from the
commit message - 2013-12-05) we have a less fragile way to cut out
"git status" at the end of a commit message but it's only enabled for
stripping submodule shortlogs.

Add new cleanup option that reuses the same mechanism for the entire
"git status" without accidentally removing lines starting with '#'.

Signed-off-by: Nguyễn Thái Ngọc Duy &lt;pclouds@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'rs/commit-m-no-edit'</title>
<updated>2013-06-05T21:59:53Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-06-05T21:59:53Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bf9923171e299fec694153c68f06bfc1650e412d'/>
<id>urn:sha1:bf9923171e299fec694153c68f06bfc1650e412d</id>
<content type='text'>
"git commit --allow-empty-message -m ''" should not start an
editor.

* rs/commit-m-no-edit:
  commit: don't start editor if empty message is given with -m
</content>
</entry>
<entry>
<title>commit: don't start editor if empty message is given with -m</title>
<updated>2013-05-28T21:33:01Z</updated>
<author>
<name>René Scharfe</name>
<email>rene.scharfe@lsrfire.ath.cx</email>
</author>
<published>2013-05-25T21:43:34Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=25206778aac776fc6cc4887653fdae476c7a9b5a'/>
<id>urn:sha1:25206778aac776fc6cc4887653fdae476c7a9b5a</id>
<content type='text'>
If an empty message is specified with the option -m of git commit then
the editor is started.  That's unexpected and unnecessary.  Instead of
using the length of the message string for checking if the user
specified one, directly remember if the option -m was given.

Reported-by: Mislav Marohnić &lt;mislav.marohnic@gmail.com&gt;
Signed-off-by: René Scharfe &lt;rene.scharfe@lsrfire.ath.cx&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'bc/append-signed-off-by'</title>
<updated>2013-04-01T15:59:24Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-04-01T15:59:23Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=900c8ecb5c98c3cae256dcc030476392fdf2bbf2'/>
<id>urn:sha1:900c8ecb5c98c3cae256dcc030476392fdf2bbf2</id>
<content type='text'>
Consolidate codepaths that inspect log-message-to-be and decide to
add a new Signed-off-by line in various commands.

* bc/append-signed-off-by:
  git-commit: populate the edit buffer with 2 blank lines before s-o-b
  Unify appending signoff in format-patch, commit and sequencer
  format-patch: update append_signoff prototype
  t4014: more tests about appending s-o-b lines
  sequencer.c: teach append_signoff to avoid adding a duplicate newline
  sequencer.c: teach append_signoff how to detect duplicate s-o-b
  sequencer.c: always separate "(cherry picked from" from commit body
  sequencer.c: require a conforming footer to be preceded by a blank line
  sequencer.c: recognize "(cherry picked from ..." as part of s-o-b footer
  t/t3511: add some tests of 'cherry-pick -s' functionality
  t/test-lib-functions.sh: allow to specify the tag name to test_commit
  commit, cherry-pick -s: remove broken support for multiline rfc2822 fields
  sequencer.c: rework search for start of footer to improve clarity
</content>
</entry>
<entry>
<title>Merge branch 'yd/use-test-config-unconfig'</title>
<updated>2013-03-28T21:38:27Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-03-28T21:38:27Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=631bc94e67383b66da190550866566f09d32f299'/>
<id>urn:sha1:631bc94e67383b66da190550866566f09d32f299</id>
<content type='text'>
Bulk-update of the test suite.

* yd/use-test-config-unconfig:
  t5520: use test_config to set/unset git config variables (leftover bits)
  t7600: use test_config to set/unset git config variables
  t7502: remove clear_config
  t7502: use test_config to set/unset git config variables
  t9500: use test_config to set/unset git config variables
  t7508: use test_config to set/unset git config variables
  t7500: use test_config to set/unset git config variables
  t5541: use test_config to set/unset git config variables
  t5520: use test_config to set/unset git config variables
  t4202: use test_config/test_unconfig to set/unset git config variables
  t4034: use test_config/test_unconfig to set/unset git config variables
  t4304: use test_config to set/unset git config variables
  t3400: use test_config to set/unset git config variables
</content>
</entry>
<entry>
<title>t7502: remove clear_config</title>
<updated>2013-03-25T15:50:54Z</updated>
<author>
<name>Yann Droneaud</name>
<email>ydroneaud@opteya.com</email>
</author>
<published>2013-03-24T21:06:14Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=e023a31de6acdc53c006efe231acc5257740d3b0'/>
<id>urn:sha1:e023a31de6acdc53c006efe231acc5257740d3b0</id>
<content type='text'>
Using test_config ensure the configuration variable are removed
at the end of the test, there's no need to remove variable
at the beginning of the test.

Signed-off-by: Yann Droneaud &lt;ydroneaud@opteya.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t7502: use test_config to set/unset git config variables</title>
<updated>2013-03-25T15:50:54Z</updated>
<author>
<name>Yann Droneaud</name>
<email>ydroneaud@opteya.com</email>
</author>
<published>2013-03-24T21:06:10Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=464be6307c898ff314c723626b5ee61387dab9b4'/>
<id>urn:sha1:464be6307c898ff314c723626b5ee61387dab9b4</id>
<content type='text'>
Instead of using construct such as:
    test_when_finished "git config --unset &lt;key&gt;"
    git config &lt;key&gt; &lt;value&gt;
uses
    test_config &lt;key&gt; &lt;value&gt;
The latter takes care of removing &lt;key&gt; at the end of the test.

Signed-off-by: Yann Droneaud &lt;ydroneaud@opteya.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'rt/commit-cleanup-config'</title>
<updated>2013-03-21T21:02:53Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2013-03-21T21:02:53Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=dcf0d12aed0d4627dcbfdbc52cea75ce74103e46'/>
<id>urn:sha1:dcf0d12aed0d4627dcbfdbc52cea75ce74103e46</id>
<content type='text'>
Fix tests that contaminated their environments and affected new
tests introduced later in the sequence by containing their effects
in their own subshells.

* rt/commit-cleanup-config:
  t7502: perform commits using alternate editor in a subshell
</content>
</entry>
<entry>
<title>git-commit: populate the edit buffer with 2 blank lines before s-o-b</title>
<updated>2013-02-23T08:03:00Z</updated>
<author>
<name>Brandon Casey</name>
<email>drafnel@gmail.com</email>
</author>
<published>2013-02-22T22:05:27Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8c613fd5efa1c3bbd48845ed91c533192e699c08'/>
<id>urn:sha1:8c613fd5efa1c3bbd48845ed91c533192e699c08</id>
<content type='text'>
'commit -s' populates the edit buffer with a blank line before the
Signed-off-by line, to allow the user to immediately start typing
the log message.  But commit 33f2f9ab removed this space, forcing
the user to first push the Signed-off-by line down to open a place
to type the log message.

Fix this regression and let's ensure that the Signed-off-by line is
preceded by two blank lines, instead of just one, to hint that
something should be filled in, and that a blank line should separate
it from the body and the Signed-off-by line.

Add a test for this behavior.

Reported-by: John Keeping &lt;john@keeping.me.uk&gt;
Signed-off-by: Brandon Casey &lt;drafnel@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t7502: perform commits using alternate editor in a subshell</title>
<updated>2013-02-23T08:00:16Z</updated>
<author>
<name>Brandon Casey</name>
<email>drafnel@gmail.com</email>
</author>
<published>2013-02-22T23:13:00Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=24e099f475692059dc2cb0c1c0d1a74a3fde8ac6'/>
<id>urn:sha1:24e099f475692059dc2cb0c1c0d1a74a3fde8ac6</id>
<content type='text'>
These tests call test_set_editor to set an alternate editor script, but
they appear to presume that the assignment is of a temporary nature and
will not have any effect outside of each individual test.  That is not
the case.  All of the test functions within a test script share a single
environment, so any variables modified in one, are visible in the ones
that follow.

So, let's protect the test functions that follow these, which set an
alternate editor, by performing the test_set_editor and 'git commit'
in a subshell.

Signed-off-by: Brandon Casey &lt;drafnel@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
</feed>
