<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/git.git/apply.c, branch v0.99.8e</title>
<subtitle>Git
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v0.99.8e</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/atom?h=v0.99.8e'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/'/>
<updated>2005-10-16T18:05:14Z</updated>
<entry>
<title>Update git-apply to use C-style quoting for funny pathnames.</title>
<updated>2005-10-16T18:05:14Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-10-16T18:05:14Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=ea56188a2488eff449661e2b4de656f38c88e62d'/>
<id>urn:sha1:ea56188a2488eff449661e2b4de656f38c88e62d</id>
<content type='text'>
This is a backport so that maintenance branch can understand
diff output that uses C-style quoting produced by newer tools.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>[PATCH] Flag empty patches as errors</title>
<updated>2005-10-01T06:52:48Z</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@osdl.org</email>
</author>
<published>2005-10-01T06:25:23Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=1fea629f794cda57cc161979dab903ec7460cc7c'/>
<id>urn:sha1:1fea629f794cda57cc161979dab903ec7460cc7c</id>
<content type='text'>
A patch that contains no actual diff, and that doesn't change any
meta-data is bad. It shouldn't be a patch at all, and git-apply shouldn't
just accept it.

This caused a corrupted patch to be silently applied as an empty change in
the kernel, because the corruption ended up making the patch look empty.

An example of such a patch is one that contains the patch header, but
where the initial fragment header (the "@@ -nr,.." line) is missing,
causing us to not parse any fragments.

The real "patch" program will also flag such patches as bad, with the
message

	patch: **** Only garbage was found in the patch input.

and we should do likewise.

Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>[PATCH] Make git-apply understand incomplete lines in non-C locales</title>
<updated>2005-09-04T18:00:28Z</updated>
<author>
<name>Fredrik Kuivinen</name>
<email>freku045@student.liu.se</email>
</author>
<published>2005-09-04T17:29:02Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=433ef8a2fb93627e311217b2a41b3132933e6b0b'/>
<id>urn:sha1:433ef8a2fb93627e311217b2a41b3132933e6b0b</id>
<content type='text'>
The message "\ No newline at end of file" used by diff(1) to mark
an incomplete line is locale dependent. We can't assume more than
that it begins with "\ ".

For example, given two files, "foo" and "bar", with appropriate
contents, 'diff -u foo bar' will produce the following output on
my system:

    --- foo 2005-09-04 18:59:38.000000000 +0200
    +++ bar 2005-09-04 18:59:16.000000000 +0200
    @@ -1 +1 @@
    -foobar
    +foo
    \ Ingen nyrad vid filslut

[jc: the check for the marker still uses the line length being no less
than 12 bytes for a sanity check, but I think it is safe to assume
that in other locales. I haven't checked the .po files from diff, tho'.]

Signed-off-by: Fredrik Kuivinen &lt;freku045@student.liu.se&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Merge refs/heads/portable from http://www.cs.berkeley.edu/~ejr/gits/git.git </title>
<updated>2005-08-29T06:02:01Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-08-29T06:02:01Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=10d781b9caa4f71495c7b34963bef137216f86a8'/>
<id>urn:sha1:10d781b9caa4f71495c7b34963bef137216f86a8</id>
<content type='text'>
</content>
</entry>
<entry>
<title>[PATCH] Fix git patch header processing in git-apply.</title>
<updated>2005-08-28T21:24:01Z</updated>
<author>
<name>Robert Fitzsimons</name>
<email>robfitz@273k.net</email>
</author>
<published>2005-08-28T15:24:27Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=e70a165d3db26dbab62e3430553a21d5e97b74ec'/>
<id>urn:sha1:e70a165d3db26dbab62e3430553a21d5e97b74ec</id>
<content type='text'>
Stop processing and return NULL if we encounter a '\n' character
before we have two matching names in the git header.

Signed-off-by: Robert Fitzsimons &lt;robfitz@273k.net&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>Fix ?: statements.</title>
<updated>2005-08-24T03:41:12Z</updated>
<author>
<name>Jason Riedy</name>
<email>ejr@cs.berkeley.edu</email>
</author>
<published>2005-08-23T20:34:07Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=c7c81b3a51de2778535c8bb18d42a4bb7dfd57bc'/>
<id>urn:sha1:c7c81b3a51de2778535c8bb18d42a4bb7dfd57bc</id>
<content type='text'>
Omitting the first branch in ?: is a GNU extension.  Cute,
but not supported by other compilers.  Replaced mostly
by explicit tests.  Calls to getenv() simply are repeated
on non-GNU compilers.

Signed-off-by: Jason Riedy &lt;ejr@cs.berkeley.edu&gt;
</content>
</entry>
<entry>
<title>[PATCH] When copying or renaming, keep the mode, please</title>
<updated>2005-08-17T19:13:12Z</updated>
<author>
<name>Johannes Schindelin</name>
<email>Johannes.Schindelin@gmx.de</email>
</author>
<published>2005-08-17T07:01:07Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=35cc4bcd10862db190df9685b7ad221f2a25404f'/>
<id>urn:sha1:35cc4bcd10862db190df9685b7ad221f2a25404f</id>
<content type='text'>
Without this patch, git-apply does not retain the mode when renaming or
copying files.

[jc: Good catch, Johannes.  I added a test case to demonstrate the
breackage in the original.]

Signed-off-by: Johannes Schindelin &lt;Johannes.Schindelin@gmx.de&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>[PATCH] -Werror fixes</title>
<updated>2005-08-10T05:28:19Z</updated>
<author>
<name>Timo Sirainen</name>
<email>tss@iki.fi</email>
</author>
<published>2005-08-09T15:30:22Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=4ec99bf080f2d95c7c3d706f65eea1c6b9302f23'/>
<id>urn:sha1:4ec99bf080f2d95c7c3d706f65eea1c6b9302f23</id>
<content type='text'>
GCC's format __attribute__ is good for checking errors, especially
with -Wformat=2 parameter. This fixes most of the reported problems
against 2005-08-09 snapshot.
</content>
</entry>
<entry>
<title>[PATCH] Make git-apply --stat less butt-ugly with long filenames</title>
<updated>2005-07-29T04:05:06Z</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@osdl.org</email>
</author>
<published>2005-07-29T03:37:23Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=629170973d6fca813a7593c3aac9bb18a8e476a8'/>
<id>urn:sha1:629170973d6fca813a7593c3aac9bb18a8e476a8</id>
<content type='text'>
When git-apply was printing out long filenames, it used to just truncate
them to show the last "max_len" characters of the filename. Which can be
really quite ugly (note the two filenames that have just been silently
truncated from the beginning - it looks even worse when there are lots
of them, like there were in the current v2.6.13-rc4 cris arch update):

 Documentation/video4linux/README.saa7134           |    9
 Documentation/video4linux/bttv/Cards               |   74
 umentation/video4linux/hauppauge-wintv-cx88-ir.txt |   54
 Documentation/video4linux/lifeview.txt             |   42
 mentation/video4linux/not-in-cx2388x-datasheet.txt |   41
 Documentation/w1/w1.generic                        |  107

With this patch it now looks like so:

 Documentation/video4linux/README.saa7134           |    9
 Documentation/video4linux/bttv/Cards               |   74
 .../video4linux/hauppauge-wintv-cx88-ir.txt        |   54
 Documentation/video4linux/lifeview.txt             |   42
 .../video4linux/not-in-cx2388x-datasheet.txt       |   41
 Documentation/w1/w1.generic                        |  107

ie we've made it clear with an ellipsis that we've cut off something from
the beginning, and it also tries to do it cleanly at a subdirectory level.

Signed-off-by: Linus "good taste" Torvalds &lt;torvalds@osdl.org&gt;
Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
</content>
</entry>
<entry>
<title>[PATCH] apply.c: --exclude=fnmatch-pattern option.</title>
<updated>2005-07-22T17:26:01Z</updated>
<author>
<name>Junio C Hamano</name>
<email>junkio@cox.net</email>
</author>
<published>2005-07-22T16:56:57Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/git.git/commit/?id=d854f783af2a441827d66b4b70e9d0d9e3434b15'/>
<id>urn:sha1:d854f783af2a441827d66b4b70e9d0d9e3434b15</id>
<content type='text'>
Adds --exclude=pattern option to the "git-apply" command.  This
was useful while reimporting the BKCVS patchset dump of the
Linux kernel, starting at 2.4.0 and ending at 2.6.12-rc2 Ingo
announced some time ago to exclude BitKeeper directory.

Signed-off-by: Junio C Hamano &lt;junkio@cox.net&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
</feed>
