Re: [PATCH] submodule update - don't run git-fetch if sha1 available

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/12/07, Junio C Hamano <gitster@xxxxxxxxx> wrote:

> unnecessary work), you may want to add a more lightweight
> built-in command to do so.  Perhaps...
>
> -- >8 --
> Subject: ls-tree --check
>
> You can run "ls-tree --check" to make sure that all necessary
> objects exist in the repository to complete a tree object.  This
> may be useful to make sure that the subproject commit bound to a
> superproject index can be checked out in its entirety.

This looks good, and the subproject part of it should be trivial to
implement but i'm struggling with applying your patch.

If I copy the whole mail with headers (I get them with a menu item
"show original" in gmail web client), paste it into a file ("*p in
vim) I get the following:

$ git apply junio.mail
Adds trailing whitespace.
junio.mail:103:#define LS_CHECK 32
Adds trailing whitespace.
junio.mail:112: "git-ls-tree [-d] [-r] [-t] [-l] [-z] [--check]
[--name-only] [--name-status] [--full-name] [--abbrev[=<n>]] <tree-i
sh> [path...]";
Adds trailing whitespace.
junio.mail:120:         } else if (ls_options & LS_CHECK) {
Adds trailing whitespace.
junio.mail:121:                 if (S_ISGITLINK(mode))
Adds trailing whitespace.
junio.mail:122:                         /*
error: patch failed: builtin-ls-tree.c:16
error: builtin-ls-tree.c: patch does not apply

git am gives me similar behaviour.

$ git am junio.mail

Applying submodule update - don't run git-fetch if sha1 available

Adds trailing whitespace.
.dotest/patch:14:#define LS_CHECK 32
Adds trailing whitespace.
.dotest/patch:23:       "git-ls-tree [-d] [-r] [-t] [-l] [-z]
[--check] [--name-only] [--name-status] [--full-name] [--abbrev[=<n>]]
 <tree-ish> [path...]";
Adds trailing whitespace.
.dotest/patch:31:               } else if (ls_options & LS_CHECK) {
Adds trailing whitespace.
.dotest/patch:32:                       if (S_ISGITLINK(mode))
Adds trailing whitespace.
.dotest/patch:33:                               /*
error: patch failed: builtin-ls-tree.c:16
error: builtin-ls-tree.c: patch does not apply
Patch failed at 0001.
When you have resolved this problem run "git-am --resolved".
If you would prefer to skip this patch, instead run "git-am --skip".



Is the "-- >8 --" -line something git-apply looks for?

Short of manual patching, abandoning the web-client or switch to
another mail-account, is there any simple way to get around the above
issues ?    [I'm trying to do this on windows/msysgit]


Thank you for good support on this list!

//Torgil
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux