Re: [RFC/PATCH 2/5] upload-pack: support out of band client capability requests

2015-02-27 Thread Kyle J. McKay
On Feb 27, 2015, at 17:01, Stefan Beller wrote: From: Nguyễn Thái Ngọc Duy pclo...@gmail.com The only difference from the original protocol client capabilities are negotiated before initial refs advertisment. Client capabilities are sent out of band (upload-pack receives it as the second

Re: [PATCH] l10n: de.po: fix negation for commit -a with paths

2015-02-27 Thread Ralf Thielow
2015-02-27 17:30 GMT+01:00 Michael J Gruber g...@drmicha.warpmail.net: Signed-off-by: Michael J Gruber g...@drmicha.warpmail.net --- po/de.po | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/po/de.po b/po/de.po index 11fbd0f..aff3109 100644 --- a/po/de.po +++ b/po/de.po

Re: Easy Non-Fast-Forward Pushes

2015-02-27 Thread Stefan Beller
On Fri, Feb 27, 2015 at 8:20 AM, Lasse Kliemann la...@lassekliemann.de wrote: As far as I understand, a push will always modify (or add) a ref in the remote repository. When pushing to branch B, then the ref pointing to the last commit in this branch will be moved, provided that this can be

Easy Non-Fast-Forward Pushes

2015-02-27 Thread Lasse Kliemann
As far as I understand, a push will always modify (or add) a ref in the remote repository. When pushing to branch B, then the ref pointing to the last commit in this branch will be moved, provided that this can be done in a fast-forward way. Otherwise the push will fail. The following options

[PATCH] l10n: de.po: fix negation for commit -a with paths

2015-02-27 Thread Michael J Gruber
Signed-off-by: Michael J Gruber g...@drmicha.warpmail.net --- po/de.po | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/po/de.po b/po/de.po index 11fbd0f..aff3109 100644 --- a/po/de.po +++ b/po/de.po @@ -4613,7 +4613,7 @@ msgstr Ungültiger \cleanup\ Modus %s #:

Re: Deadlock during remote update

2015-02-27 Thread Jeff King
On Fri, Feb 27, 2015 at 06:27:28PM +0700, Duy Nguyen wrote: 31638 git-remote-https upstream https://git.openstack.org/openstack/nova has pipe:[170381707] (fd 1), waiting for read from pipe:[170384472] 31642 git fetch-pack --stateless-rpc --lock-pack --include-tag --thin --no-progress

[PATCH] grep: correct help string for --exclude-standard

2015-02-27 Thread Nguyễn Thái Ngọc Duy
The current help string is about --no-exclude-standard. But git grep -h would show --exclude-standard instead. Flip the string. See 0a93fb8 (grep: teach --untracked and --exclude-standard options - 2011-09-27) for more info about these options. Signed-off-by: Nguyễn Thái Ngọc Duy

Re: Git gc removes all packs

2015-02-27 Thread Jeff King
On Fri, Feb 27, 2015 at 11:16:09AM +0100, Dmitry Neverov wrote: I followed your advice and removed a symlink ref from my repository. But didn't help.. automatic GC has just removed all packs again. May alternates cause such a behavior? Are any ways to make gc log somewhere why it removes

Re: [PATCH] sequencer: preserve commit messages

2015-02-27 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: Without any config being set the result is certainly what I'm after. What I'm still wondering about is the case without --edit but with commit.cleanup: It seems to me that git commit being involved in a conflict-less cherry-pick is solely an

Re: [PATCH] sequencer: preserve commit messages

2015-02-27 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 26.02.2015 20:49: Michael J Gruber g...@drmicha.warpmail.net writes: Hmm. With --edit, current config being in effect should be expected, right? So how about: In case of no conflict: force cleanup=verbatim unless --edit is used? Perhaps something like

Re: feature request: excluding files/paths from git grep

2015-02-27 Thread Michael J Gruber
Junio C Hamano venit, vidit, dixit 26.02.2015 21:59: Michael J Gruber g...@drmicha.warpmail.net writes: So, as a summary of the discussion, it seems it's time to switch the default to --textconv for git grep? Hmmm, why? Nobody seems to be asking for such a change in this thread. The

Re: Easy Non-Fast-Forward Pushes

2015-02-27 Thread Junio C Hamano
Lasse Kliemann la...@lassekliemann.de writes: As far as I understand, a push will always modify (or add) a ref in the remote repository. When pushing to branch B, then the ref pointing to the last commit in this branch will be moved, provided that this can be done in a fast-forward way.

Re: feature request: excluding files/paths from git grep

2015-02-27 Thread Junio C Hamano
Michael J Gruber g...@drmicha.warpmail.net writes: Junio C Hamano venit, vidit, dixit 26.02.2015 21:59: So that does not sound to me a summary of the discussion at all. Well, your conditional I do not recall its conclusion, but it it were Yes, that is what it means, then it might be

Re: [PATCH v2 2/2] index-pack: kill union delta_base to save memory

2015-02-27 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: Notice that with recent Git versions, ofs-delta objects are preferred over ref-delta objects and ref-delta objects have no reason to be present in a clone pack. It is true that we try to use ofs-delta as much as possible, but where does have no

Re: [PATCH] versionsort: support reorder prerelease suffixes

2015-02-27 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy pclo...@gmail.com writes: Signed-off-by: Nguyễn Thái Ngọc Duy pclo...@gmail.com --- Second round. Looking better. We can do 1.0-pre12 1.0-rc0 1.0 1.0-post1 too but it relies on config key's loading order, a bit iffy. Documentation/config.txt | 7 +++

Re: [PATCH 2/2] diffcore-rename: avoid processing duplicate destinations

2015-02-27 Thread Junio C Hamano
Jeff King p...@peff.net writes: Like I mentioned before, I'm OK with not checking the actual diff output in the test. It's not like it was planned, and is just what diff_tree() happens to produce. It does make sense, though When the topic is on processing broken input, I do not think It

Re: [PATCH v6 0/4] commit: Add commit.verbose configuration

2015-02-27 Thread Torstein Hegge
On Tue, Jun 17, 2014 at 14:38:56 -0500, Caleb Thompson wrote: This patch allows people to set commit.verbose to implicitly send --verbose to git-commit. It introduces several cleanup patches to t/t7505-commit-verbose.sh to bring it closer to the current state of the tests as they have been

Deadlock during remote update

2015-02-27 Thread Heald, Mike
Hi, We have a cron job that runs remote update on a number of repositories. Sometimes, the processes deadlock and we have to go -TERM them. Here's a breakdown of what state the processes end up in when the deadlock happens, from one of our production systems yesterday: 31629 git

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Duy Nguyen
On Sat, Feb 28, 2015 at 6:05 AM, Junio C Hamano gits...@pobox.com wrote: Just for fun, I was trying to see if there is a hole in the current protocol that allows a new client to talk a valid v1 protocol exchange with existing, deployed servers without breaking, while letting it to know a new

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Stefan Beller
+git@vger.kernel.org On Thu, Feb 26, 2015 at 5:42 PM, Duy Nguyen pclo...@gmail.com wrote: https://github.com/pclouds/git/commits/uploadpack2 I rebased your branch, changed the order of commits slightly and started to add some. they are found at

Re: Any chance for a Git v2.1.5 release?

2015-02-27 Thread Philip Oakley
From: Kyle J. McKay mack...@gmail.com On Feb 26, 2015, at 12:54, Junio C Hamano wrote: Kyle J. McKay mack...@gmail.com writes: I would like to better understand how the various heads are maintained. I've read MaintNotes and I've got the concepts, but I'm still a little fuzzy on some

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Junio C Hamano
Junio C Hamano gits...@pobox.com writes: I do not think v1 can be fixed by send one ref with capability, newer client may respond immediately so we can stop enumerating remaining refs and older one will get stuck so we can have a timeout to see if the connection is from the newer one, and

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Junio C Hamano
On Fri, Feb 27, 2015 at 4:07 PM, Duy Nguyen pclo...@gmail.com wrote: There may be another hole, if we send want empty-tree, it looks like it will go through without causing errors. It's not exactly no-op because an empty tree object will be bundled in result pack. But that makes no difference

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Junio C Hamano
On Fri, Feb 27, 2015 at 3:44 PM, Stefan Beller sbel...@google.com wrote: On Fri, Feb 27, 2015 at 3:05 PM, Junio C Hamano gits...@pobox.com wrote: I am _not_ proposing that we should go this route, at least not yet. I am merely pointing out that an in-place sidegrade from v1 to a protocol that

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Stefan Beller
On Fri, Feb 27, 2015 at 4:33 PM, Junio C Hamano gits...@pobox.com wrote: On Fri, Feb 27, 2015 at 3:44 PM, Stefan Beller sbel...@google.com wrote: On Fri, Feb 27, 2015 at 3:05 PM, Junio C Hamano gits...@pobox.com wrote: I am _not_ proposing that we should go this route, at least not yet. I am

[RFC/PATCH 3/5] connect.c: connect to a remote service with some flags

2015-02-27 Thread Stefan Beller
If this is over git protocol, the flags is appended as the next parameter after host=. If it's ssh, a new argument is appended to the command line. None of the callers use this now though. [sb: originally by pclouds, rebased as jk implemented 1823bea10, (git_connect: use argv_array), so any

[RFC/PATCH 2/5] upload-pack: support out of band client capability requests

2015-02-27 Thread Stefan Beller
From: Nguyễn Thái Ngọc Duy pclo...@gmail.com The only difference from the original protocol client capabilities are negotiated before initial refs advertisment. Client capabilities are sent out of band (upload-pack receives it as the second command line argument). The server sends one pkt-line

[RFC/PATCH 1/5] upload-pack: only accept capabilities on the first want line

2015-02-27 Thread Stefan Beller
From: Nguyễn Thái Ngọc Duy pclo...@gmail.com pack-protocol.txt says so and fetch-pack also follows it even though upload-pack is a bit lax. Fix it. Signed-off-by: Stefan Beller sbel...@google.com --- upload-pack.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[RFC/PATCH 4/5] daemon.c: accept extra service arguments

2015-02-27 Thread Stefan Beller
Before 73bb33a (daemon: Strictly parse the extra arg part of the command - 2009-06-04) a client sending extra arguments could DoS git-daemon. 73bb33a fixed it by forbidding extra arguments. Allow arguments other than host= again as a preparation step for upload-pack2. host= if present must be the

[RFC/PATCH 0/5] protocol v2 for upload-pack

2015-02-27 Thread Stefan Beller
Heavily inspired by the ideas of Duy, who wrote the first patches nearly a year ago. Nguyễn Thái Ngọc Duy (2): upload-pack: only accept capabilities on the first want line upload-pack: support out of band client capability requests Stefan Beller (3): connect.c: connect to a remote service

[RFC/PATCH 5/5] WIP/Document the http protocol change

2015-02-27 Thread Stefan Beller
Signed-off-by: Stefan Beller sbel...@google.com --- Documentation/technical/http-protocol.txt | 4 ++-- Documentation/technical/protocol-capabilities.txt | 4 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/Documentation/technical/http-protocol.txt

Re: feature request: excluding files/paths from git grep

2015-02-27 Thread Trevor Saunders
On Wed, Feb 25, 2015 at 02:11:08PM -0500, Jeff King wrote: On Wed, Feb 25, 2015 at 11:01:22AM -0800, Junio C Hamano wrote: Jeff King p...@peff.net writes: So I think _if_ using diff attributes is enough for this purpose, then there is no code to be written. But if somebody wants to

Re: Git gc removes all packs

2015-02-27 Thread Dmitry Neverov
I followed your advice and removed a symlink ref from my repository. But didn't help.. automatic GC has just removed all packs again. May alternates cause such a behavior? Are any ways to make gc log somewhere why it removes packs? On Thu, Feb 5, 2015 at 9:03 PM, Jeff King p...@peff.net wrote:

Re: [RFC/PATCH 0/3] protocol v2

2015-02-27 Thread Stefan Beller
On Fri, Feb 27, 2015 at 3:05 PM, Junio C Hamano gits...@pobox.com wrote: Junio C Hamano gits...@pobox.com writes: I do not think v1 can be fixed by send one ref with capability, newer client may respond immediately so we can stop enumerating remaining refs and older one will get stuck so we

Re: Deadlock during remote update

2015-02-27 Thread Duy Nguyen
On Fri, Feb 27, 2015 at 6:03 PM, Heald, Mike mike.he...@hp.com wrote: Hi, We have a cron job that runs remote update on a number of repositories. Sometimes, the processes deadlock and we have to go -TERM them. Here's a breakdown of what state the processes end up in when the deadlock