Skip to content

Commit e16a477

Browse files
committed
Sync with 1.6.3.3
Signed-off-by: Junio C Hamano <[email protected]>
2 parents c5764c0 + cff4231 commit e16a477

File tree

5 files changed

+53
-12
lines changed

5 files changed

+53
-12
lines changed

.gitattributes

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
* whitespace=!indent,trail,space
2-
*.[ch] whitespace
2+
*.[ch] whitespace=indent,trail,space

Documentation/RelNotes-1.6.3.3.txt

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
GIT v1.6.3.3 Release Notes
2+
==========================
3+
4+
Fixes since v1.6.3.2
5+
--------------------
6+
7+
* "git archive" running on Cygwin can get stuck in an infinite loop.
8+
9+
* "git daemon" did not correctly parse the initial line that carries
10+
virtual host request information.
11+
12+
* "git diff --textconv" leaked memory badly when the textconv filter
13+
errored out.
14+
15+
* The built-in regular expressions to pick function names to put on
16+
hunk header lines for java and objc were very inefficiently written.
17+
18+
* in certain error situations git-fetch (and git-clone) on Windows didn't
19+
detect connection abort and ended up waiting indefinitely.
20+
21+
* import-tars script (in contrib) did not import symbolic links correctly.
22+
23+
* http.c used CURLOPT_SSLKEY even on libcURL version 7.9.2, even though
24+
it was only available starting 7.9.3.
25+
26+
* low-level filelevel merge driver used return value from strdup()
27+
without checking if we ran out of memory.
28+
29+
* "git rebase -i" left stray closing parenthesis in its reflog message.
30+
31+
* "git remote show" did not show all the URLs associated with the named
32+
remote, even though "git remote -v" did. Made them consistent by
33+
making the former show all URLs.
34+
35+
* "whitespace" attribute that is set was meant to detect all errors known
36+
to git, but it told git to ignore trailing carriage-returns.
37+
38+
Includes other documentation fixes.

Documentation/git.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,10 @@ unreleased) version of git, that is available from 'master'
4343
branch of the `git.git` repository.
4444
Documentation for older releases are available here:
4545

46-
* link:v1.6.3.2/git.html[documentation for release 1.6.3.2]
46+
* link:v1.6.3.3/git.html[documentation for release 1.6.3.3]
4747

4848
* release notes for
49+
link:RelNotes-1.6.3.3.txt[1.6.3.3],
4950
link:RelNotes-1.6.3.2.txt[1.6.3.2],
5051
link:RelNotes-1.6.3.1.txt[1.6.3.1],
5152
link:RelNotes-1.6.3.txt[1.6.3].

t/t3505-cherry-pick-empty.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,11 @@ test_expect_success setup '
1717
1818
'
1919

20-
test_expect_code 1 'cherry-pick an empty commit' '
21-
22-
git checkout master &&
23-
git cherry-pick empty-branch
24-
20+
test_expect_success 'cherry-pick an empty commit' '
21+
git checkout master && {
22+
git cherry-pick empty-branch
23+
test "$?" = 1
24+
}
2525
'
2626

2727
test_expect_success 'index lockfile was removed' '

ws.c

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,12 @@
1010
static struct whitespace_rule {
1111
const char *rule_name;
1212
unsigned rule_bits;
13+
unsigned loosens_error;
1314
} whitespace_rule_names[] = {
14-
{ "trailing-space", WS_TRAILING_SPACE },
15-
{ "space-before-tab", WS_SPACE_BEFORE_TAB },
16-
{ "indent-with-non-tab", WS_INDENT_WITH_NON_TAB },
17-
{ "cr-at-eol", WS_CR_AT_EOL },
15+
{ "trailing-space", WS_TRAILING_SPACE, 0 },
16+
{ "space-before-tab", WS_SPACE_BEFORE_TAB, 0 },
17+
{ "indent-with-non-tab", WS_INDENT_WITH_NON_TAB, 0 },
18+
{ "cr-at-eol", WS_CR_AT_EOL, 1 },
1819
};
1920

2021
unsigned parse_whitespace_rule(const char *string)
@@ -79,7 +80,8 @@ unsigned whitespace_rule(const char *pathname)
7980
unsigned all_rule = 0;
8081
int i;
8182
for (i = 0; i < ARRAY_SIZE(whitespace_rule_names); i++)
82-
all_rule |= whitespace_rule_names[i].rule_bits;
83+
if (!whitespace_rule_names[i].loosens_error)
84+
all_rule |= whitespace_rule_names[i].rule_bits;
8385
return all_rule;
8486
} else if (ATTR_FALSE(value)) {
8587
/* false (-whitespace) */

0 commit comments

Comments
 (0)