<feed xmlns='http://www.w3.org/2005/Atom'>
<title>git/t/t4010-diff-pathspec.sh, branch v2.21.3</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/git/git.git/
</subtitle>
<id>https://git.shady.money/git/atom?h=v2.21.3</id>
<link rel='self' href='https://git.shady.money/git/atom?h=v2.21.3'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/'/>
<updated>2018-08-20T18:33:48Z</updated>
<entry>
<title>Merge branch 'ab/test-must-be-empty-for-master'</title>
<updated>2018-08-20T18:33:48Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2018-08-20T18:33:48Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=14677d25ab50efcf57621ae69d6ba858a5f86165'/>
<id>urn:sha1:14677d25ab50efcf57621ae69d6ba858a5f86165</id>
<content type='text'>
Test updates.

* ab/test-must-be-empty-for-master:
  tests: make use of the test_must_be_empty function
</content>
</entry>
<entry>
<title>tests: make use of the test_must_be_empty function</title>
<updated>2018-07-30T18:18:41Z</updated>
<author>
<name>Ævar Arnfjörð Bjarmason</name>
<email>avarab@gmail.com</email>
</author>
<published>2018-07-27T17:48:11Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=d3c6751b18170276f80e3578d16877d43ad22709'/>
<id>urn:sha1:d3c6751b18170276f80e3578d16877d43ad22709</id>
<content type='text'>
Change various tests that use an idiom of the form:

    &gt;expect &amp;&amp;
    test_cmp expect actual

To instead use:

    test_must_be_empty actual

The test_must_be_empty() wrapper was introduced in ca8d148daf ("test:
test_must_be_empty helper", 2013-06-09). Many of these tests have been
added after that time. This was mostly found with, and manually pruned
from:

    git grep '^\s+&gt;.*expect.* &amp;&amp;$' t

Signed-off-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t: drop unnecessary terminating semicolon in subshell</title>
<updated>2018-07-03T19:38:04Z</updated>
<author>
<name>Eric Sunshine</name>
<email>sunshine@sunshineco.com</email>
</author>
<published>2018-07-02T00:23:44Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=02779185d5b280488cd1eafb1cdbb35babee5efe'/>
<id>urn:sha1:02779185d5b280488cd1eafb1cdbb35babee5efe</id>
<content type='text'>
Signed-off-by: Eric Sunshine &lt;sunshine@sunshineco.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>test-lib.sh: introduce and use $EMPTY_TREE</title>
<updated>2016-07-18T20:45:32Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2016-07-16T05:06:24Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=f9e7d9f8c3e32280418a0bedd9d37f2a0cc2e2b9'/>
<id>urn:sha1:f9e7d9f8c3e32280418a0bedd9d37f2a0cc2e2b9</id>
<content type='text'>
This is a special SHA1. Let's keep it at one place, easier to replace
later when the hash change comes, easier to recognize.

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 'jc/diff-test-updates'</title>
<updated>2015-03-05T20:45:43Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2015-03-05T20:45:43Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=fa8baa4b2acf6f8be8f28bf84493e0889e45eb91'/>
<id>urn:sha1:fa8baa4b2acf6f8be8f28bf84493e0889e45eb91</id>
<content type='text'>
Test clean-up.

* jc/diff-test-updates:
  test_ln_s_add: refresh stat info of fake symbolic links
  t4008: modernise style
  t/diff-lib: check exact object names in compare_diff_raw
  tests: do not borrow from COPYING and README from the real source
  t4010: correct expected object names
  t9300: correct expected object names
  t4008: correct stale comments
</content>
</entry>
<entry>
<title>t4010: correct expected object names</title>
<updated>2015-02-15T23:38:10Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2015-02-15T21:35:19Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=bb8db1b5538a455a0892312b25d73d51f2650429'/>
<id>urn:sha1:bb8db1b5538a455a0892312b25d73d51f2650429</id>
<content type='text'>
The output the test expects is bogus.

It was left unnoticed only because compare_diff_raw, which only
cares about the add/delete/rename/copy was used to check the result.

Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>t4010-diff-pathspec.sh: use the $( ... ) construct for command substitution</title>
<updated>2014-04-30T18:08:05Z</updated>
<author>
<name>Elia Pinto</name>
<email>gitter.spiros@gmail.com</email>
</author>
<published>2014-04-30T16:22:57Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=38b2e5d12cbe681bff8f31cf10e3de8bd6f6db52'/>
<id>urn:sha1:38b2e5d12cbe681bff8f31cf10e3de8bd6f6db52</id>
<content type='text'>
The Git CodingGuidelines prefer the $(...) construct for command
substitution instead of using the backquotes `...`.

The backquoted form is the traditional method for command
substitution, and is supported by POSIX.  However, all but the
simplest uses become complicated quickly.  In particular, embedded
command substitutions and/or the use of double quotes require
careful escaping with the backslash character.

The patch was generated by:

for _f in $(find . -name "*.sh")
do
   sed -i 's@`\(.*\)`@$(\1)@g' ${_f}
done

and then carefully proof-read.

Signed-off-by: Elia Pinto &lt;gitter.spiros@gmail.com&gt;
Reviewed-by: Matthieu Moy &lt;Matthieu.Moy@imag.fr&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'nd/submodule-pathspec-ending-with-slash'</title>
<updated>2014-02-27T22:01:15Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-02-27T22:01:15Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=cbaeafc3251d08b479c5eb735444bb6ad6210c93'/>
<id>urn:sha1:cbaeafc3251d08b479c5eb735444bb6ad6210c93</id>
<content type='text'>
Allow "git cmd path/", when the 'path' is where a submodule is
bound to the top-level working tree, to match 'path', despite the
extra and unnecessary trailing slash.

* nd/submodule-pathspec-ending-with-slash:
  clean: use cache_name_is_other()
  clean: replace match_pathspec() with dir_path_match()
  pathspec: pass directory indicator to match_pathspec_item()
  match_pathspec: match pathspec "foo/" against directory "foo"
  dir.c: prepare match_pathspec_item for taking more flags
  pathspec: rename match_pathspec_depth() to match_pathspec()
  pathspec: convert some match_pathspec_depth() to dir_path_match()
  pathspec: convert some match_pathspec_depth() to ce_path_match()
</content>
</entry>
<entry>
<title>pathspec: pass directory indicator to match_pathspec_item()</title>
<updated>2014-02-24T22:37:19Z</updated>
<author>
<name>Nguyễn Thái Ngọc Duy</name>
<email>pclouds@gmail.com</email>
</author>
<published>2014-01-24T13:40:33Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=ae8d0824217bdf97c69ead49568cd03fc140627b'/>
<id>urn:sha1:ae8d0824217bdf97c69ead49568cd03fc140627b</id>
<content type='text'>
This patch activates the DO_MATCH_DIRECTORY code in m_p_i(), which
makes "git diff HEAD submodule/" and "git diff HEAD submodule" produce
the same output. Previously only the version without trailing slash
returns the difference (if any).

That's the effect of new ce_path_match(). dir_path_match() is not
executed by the new tests. And it should not introduce regressions.

Previously if path "dir/" is passed in with pathspec "dir/", they
obviously match. With new dir_path_match(), the path becomes
_directory_ "dir" vs pathspec "dir/", which is not executed by the old
code path in m_p_i(). The new code path is executed and produces the
same result.

The other case is pathspec "dir" and path "dir/" is now turned to
"dir" (with DO_MATCH_DIRECTORY). Still the same result before or after
the patch.

So why change? Because of the next patch about clean.c.

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 'as/tree-walk-fix-aggressive-short-cut'</title>
<updated>2014-01-27T18:48:32Z</updated>
<author>
<name>Junio C Hamano</name>
<email>gitster@pobox.com</email>
</author>
<published>2014-01-27T18:48:32Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/git/commit/?id=8bba7206b503b63a8208355f66bc09c981072d51'/>
<id>urn:sha1:8bba7206b503b63a8208355f66bc09c981072d51</id>
<content type='text'>
* as/tree-walk-fix-aggressive-short-cut:
  tree_entry_interesting: match against all pathspecs
</content>
</entry>
</feed>
