<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/t/t7501-commit.sh, branch v1.6.3</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v1.6.3</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v1.6.3'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2009-04-09T05:42:16Z</updated>
<entry>
<title>tests: skip perl tests if NO_PERL is defined</title>
<updated>2009-04-09T05:42:16Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2009-04-03T19:33:59Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1b19ccd236e3369ac77d74ded207406ffbf9feca'/>
<id>urn:sha1:1b19ccd236e3369ac77d74ded207406ffbf9feca</id>
<content type='text'>
These scripts all test git programs that are written in
perl, and thus obviously won't work if NO_PERL is defined.
We pass NO_PERL to the scripts from the building Makefile
via the GIT-BUILD-OPTIONS file.

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t7501-commit.sh: explicitly check that -F prevents invoking the editor</title>
<updated>2009-01-11T01:45:02Z</updated>
<author>
<name>Adeodato Simó</name>
<email>dato@net.com.org.es</email>
</author>
<published>2009-01-09T17:30:05Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=7eb5bbdb645e04d746bc0edea102744f50a5e529'/>
<id>urn:sha1:7eb5bbdb645e04d746bc0edea102744f50a5e529</id>
<content type='text'>
The "--signoff" test case in t7500-commit.sh was setting VISUAL while
using -F -, which indeed tested that the editor is not spawned with -F.
However, having it there was confusing, since there was no obvious reason
to the casual reader for it to be there.

This commits removes the setting of VISUAL from the --signoff test, and
adds in t7501-commit.sh a dedicated test case, where the rest of tests for
-F are.

Signed-off-by: Adeodato Simó &lt;dato@net.com.org.es&gt;
Okay-then-by: Johannes Schindelin &lt;Johannes.Schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t7501: always use test_cmp instead of diff</title>
<updated>2008-09-10T18:16:10Z</updated>
<author>
<name>Miklos Vajna</name>
<email>vmiklos@frugalware.org</email>
</author>
<published>2008-09-10T17:32:44Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=1e368681bd8c35202b91f8998e07d8bbb3de6c7c'/>
<id>urn:sha1:1e368681bd8c35202b91f8998e07d8bbb3de6c7c</id>
<content type='text'>
This should make the output more readable (by default using diff -u)
when some tests fail.

Also changed the diff order from "current expected" to "expected
current".

Signed-off-by: Miklos Vajna &lt;vmiklos@frugalware.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>tests: use "git xyzzy" form (t7200 - t9001)</title>
<updated>2008-09-03T21:51:48Z</updated>
<author>
<name>Nanako Shiraishi</name>
<email>nanako3@lavabit.com</email>
</author>
<published>2008-09-03T08:59:33Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=47a528ad24185133867ebb5bb7692db2cb8bef39'/>
<id>urn:sha1:47a528ad24185133867ebb5bb7692db2cb8bef39</id>
<content type='text'>
Converts tests between t7201-t9001.

Signed-off-by: Nanako Shiraishi &lt;nanako3@lavabit.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t/: Use "test_must_fail git" instead of "! git"</title>
<updated>2008-07-13T20:21:26Z</updated>
<author>
<name>Stephan Beyer</name>
<email>s-beyer@gmx.net</email>
</author>
<published>2008-07-12T15:47:52Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d492b31cafe9aa5ce001b1d48815f4c0bb40d01a'/>
<id>urn:sha1:d492b31cafe9aa5ce001b1d48815f4c0bb40d01a</id>
<content type='text'>
This patch changes every occurrence of "! git" -- with the meaning
that a git call has to gracefully fail -- into "test_must_fail git".

This is useful to

 - make sure the test does not fail because of a signal,
   e.g. SIGSEGV, and

 - advertise the use of "test_must_fail" for new tests.

Signed-off-by: Stephan Beyer &lt;s-beyer@gmx.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'bd/tests'</title>
<updated>2008-05-14T20:45:16Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2008-05-14T20:45:16Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=761adeb4dba3305d0810b5340a71783d29484fb0'/>
<id>urn:sha1:761adeb4dba3305d0810b5340a71783d29484fb0</id>
<content type='text'>
* bd/tests:
  Rename the test trash directory to contain spaces.
  Fix tests breaking when checkout path contains shell metacharacters
  Don't use the 'export NAME=value' in the test scripts.
  lib-git-svn.sh: Fix quoting issues with paths containing shell metacharacters
  test-lib.sh: Fix some missing path quoting
  Use test_set_editor in t9001-send-email.sh
  test-lib.sh: Add a test_set_editor function to safely set $VISUAL
  git-send-email.perl: Handle shell metacharacters in $EDITOR properly
  config.c: Escape backslashes in section names properly
  git-rebase.sh: Fix --merge --abort failures when path contains whitespace

Conflicts:

	t/t9115-git-svn-dcommit-funky-renames.sh
</content>
</entry>
<entry>
<title>fix bsd shell negation</title>
<updated>2008-05-14T04:44:48Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2008-05-14T04:01:22Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bbf08124e0a87658a76b53a05d67227fa3b4e7b9'/>
<id>urn:sha1:bbf08124e0a87658a76b53a05d67227fa3b4e7b9</id>
<content type='text'>
On some shells (notably /bin/sh on FreeBSD 6.1), the
construct

  foo &amp;&amp; ! bar | baz

is true if

  foo &amp;&amp; baz

whereas for most other shells (such as bash) is true if

  foo &amp;&amp; ! baz

We can work around this by specifying

  foo &amp;&amp; ! (bar | baz)

which works everywhere.

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Fix tests breaking when checkout path contains shell metacharacters</title>
<updated>2008-05-05T21:37:02Z</updated>
<author>
<name>Bryan Donlan</name>
<email>bdonlan@fushizen.net</email>
</author>
<published>2008-05-04T05:37:59Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f69e836fab2b634281d92a0d304de4d768e479cc'/>
<id>urn:sha1:f69e836fab2b634281d92a0d304de4d768e479cc</id>
<content type='text'>
This fixes the remainder of the issues where the test script itself is at
fault for failing when the git checkout path contains whitespace or other
shell metacharacters.

The majority of git svn tests used the idiom

  test_expect_success "title" "test script using $svnrepo"

These were changed to have the test script in single-quotes:

  test_expect_success "title" 'test script using "$svnrepo"'

which unfortunately makes the patch appear larger than it really is.

One consequence of this change is that in the verbose test output the
value of $svnrepo (and in some cases other variables, too) is no
longer expanded, i.e. previously we saw

  * expecting success:
	test script using /path/to/git/t/trash/svnrepo

but now it is:

  * expecting success:
	test script using "$svnrepo"

Signed-off-by: Bryan Donlan &lt;bdonlan@fushizen.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>add test_cmp function for test scripts</title>
<updated>2008-03-13T07:57:52Z</updated>
<author>
<name>Jeff King</name>
<email>peff@peff.net</email>
</author>
<published>2008-03-12T21:36:36Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=82ebb0b6ec7470cab96a013d3d719c109003ef83'/>
<id>urn:sha1:82ebb0b6ec7470cab96a013d3d719c109003ef83</id>
<content type='text'>
Many scripts compare actual and expected output using
"diff -u". This is nicer than "cmp" because the output shows
how the two differ. However, not all versions of diff
understand -u, leading to unnecessary test failure.

This adds a test_cmp function to the test scripts and
switches all "diff -u" invocations to use it. The function
uses the contents of "$GIT_TEST_CMP" to compare its
arguments; the default is "diff -u".

On systems with a less-capable diff, you can do:

  GIT_TEST_CMP=cmp make test

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'maint'</title>
<updated>2008-02-03T08:57:23Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2008-02-03T08:57:23Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d8534adac756a71a897b3a433eaea7992a137b71'/>
<id>urn:sha1:d8534adac756a71a897b3a433eaea7992a137b71</id>
<content type='text'>
* maint:
  Fix "git-commit -C $tag"
  Documentation/git-stash.txt: Adjust SYNOPSIS command syntax (2)
</content>
</entry>
</feed>
