Re: [PATCH v5 00/14] New remote-hg helper

2012-11-01 Thread Felipe Contreras
On Tue, Oct 30, 2012 at 7:00 PM, Chris Webb ch...@arachsys.com wrote: Felipe Contreras felipe.contre...@gmail.com writes: Yes, it seems this is an API issue; repo.branchtip doesn't exist in python 2.2. Hi. Presumably this is a problem with old mercurial not a problem with old python as

Re: [OT] How to get the discussion details via notes

2012-11-01 Thread Peter Baumann
On Wed, Oct 31, 2012 at 10:10:24AM -0400, Jeff King wrote: On Wed, Oct 31, 2012 at 10:53:27AM +0100, Peter Baumann wrote: covers the basics (current behavior and intent of the change) in its first two paragraphs and anyone wanting more detail can use

[PATCH v2 0/3] Introduce diff.submodule

2012-11-01 Thread Ramkumar Ramachandra
Hi, v1 is here: http://mid.gmane.org/1349196670-2844-1-git-send-email-artag...@gmail.com I've fixed the issues pointed out in the review by Jens. Sorry about the delay; need to find a new dayjob. Ramkumar Ramachandra (3): Documentation: move diff.wordRegex from config.txt to diff-config.txt

[PATCH 1/3] Documentation: move diff.wordRegex from config.txt to diff-config.txt

2012-11-01 Thread Ramkumar Ramachandra
19299a8 (Documentation: Move diff.driver.* from config.txt to diff-config.txt, 2011-04-07) moved the diff configuration options to diff-config.txt, but forgot about diff.wordRegex, which was left behind in config.txt. Fix this. Signed-off-by: Ramkumar Ramachandra artag...@gmail.com ---

[PATCH 2/3] diff: introduce diff.submodule configuration variable

2012-11-01 Thread Ramkumar Ramachandra
Introduce a diff.submodule configuration variable corresponding to the '--submodule' command-line option of 'git diff'. Signed-off-by: Ramkumar Ramachandra artag...@gmail.com --- Documentation/diff-config.txt|7 +++ Documentation/diff-options.txt |3 ++- builtin/diff.c

[PATCH 3/3] submodule: display summary header in bold

2012-11-01 Thread Ramkumar Ramachandra
Currently, 'git diff --submodule' displays output with a bold diff header for non-submodules. So this part is in bold: diff --git a/file1 b/file1 index 30b2f6c..2638038 100644 --- a/file1 +++ b/file1 For submodules, the header looks like this: Submodule submodule1

Re: [git-users] Git clone fails with bad pack header, how to get remote log

2012-11-01 Thread Jeff King
On Wed, Oct 31, 2012 at 03:34:22PM +0100, kevin molcard wrote: I forgot to mention that I am using scm manager: https://bitbucket.org/sdorra/scm-manager/wiki/Home So that maybe the custom layer you are talking about. Yeah, that is very important. If I am reading the scm-manager code right

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread René Scharfe
Am 01.11.2012 03:58, schrieb Felipe Contreras: On Thu, Nov 1, 2012 at 2:32 AM, Junio C Hamano gits...@pobox.com wrote: Johannes Schindelin johannes.schinde...@gmx.de wrote: On Wed, 31 Oct 2012, Felipe Contreras wrote: It doesn't get any more obvious than that. But to each his own. In my

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Tomas Carnecky
On Thu, Nov 1, 2012 at 1:46 PM, René Scharfe rene.scha...@lsrfire.ath.cx wrote: Also, and I'm sure you didn't know that, Jedem das Seine (to each his own) was the slogan of the Buchenwald concentration camp. For that reason some (including me) hear the unspoken cynical half-sentence and some

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Martin Langhoff
On Thu, Nov 1, 2012 at 9:46 AM, René Scharfe rene.scha...@lsrfire.ath.cx wrote: You probably didn't intend it, but your sentences at the top can be read more like: This is a logical consequence. If you don't understand that, your mental capabilities must be lacking.. That's obviously (ha!) a

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Felipe Contreras
On Thu, Nov 1, 2012 at 2:46 PM, René Scharfe rene.scha...@lsrfire.ath.cx wrote: Am 01.11.2012 03:58, schrieb Felipe Contreras: On Thu, Nov 1, 2012 at 2:32 AM, Junio C Hamano gits...@pobox.com wrote: Johannes Schindelin johannes.schinde...@gmx.de wrote: On Wed, 31 Oct 2012, Felipe Contreras

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Martin Langhoff
Felipe, I'll invite you to reread some of your words: That being said, I did wonder what must be going through his mind to not see that as obvious, (...) Following the guideline of always assuming good faith So perhaps it does apply that you could try to assume good intellectual faith in

Re: [git-users] Git clone fails with bad pack header, how to get remote log

2012-11-01 Thread kevin molcard
Hi Peff, thanks for this information. I will report the issue to scm-manager. In the meantime I will try to use ssh protocol to connect to my remote server. Thanks again, Kevin 2012/11/1 Jeff King p...@peff.net: On Wed, Oct 31, 2012 at 03:34:22PM +0100, kevin molcard wrote: I forgot to

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Felipe Contreras
On Thu, Nov 1, 2012 at 3:47 PM, Martin Langhoff martin.langh...@gmail.com wrote: Felipe, I'll invite you to reread some of your words: That being said, I did wonder what must be going through his mind to not see that as obvious, (...) Following the guideline of always assuming good faith

Re: Wrap commit messages on `git commit -m`

2012-11-01 Thread Thomas Adam
Hi, On 1 November 2012 16:07, Ramkumar Ramachandra artag...@gmail.com wrote: Hi, Some of my colleagues are lazy to fire up an editor and write proper commit messages- they often write one-liners using `git commit -m`. However, that line turns out to be longer than 72 characters, and the

Re: What's cooking in git.git (Oct 2012, #09; Mon, 29)

2012-11-01 Thread Ramsay Jones
Jeff King wrote: What's cooking in git.git (Oct 2012, #09; Mon, 29) -- [snip] * cr/cvsimport-local-zone (2012-10-16) 1 commit - git-cvsimport: allow author-specific timezones Allows cvsimport to read per-author timezone from the author

Re: crash on git diff-tree -Ganything tree for new files with textconv filter

2012-11-01 Thread Ramsay Jones
Jeff King wrote: Thinking on it more, my patch, hacky thought it seems, may not be the worst solution. Here are the options that I see: 1. Use a regex library that does not require NUL termination. If we are bound by the regular regexec interface, this is not feasible. But the

checkout-index: unable to create file foo (File exists)

2012-11-01 Thread Brian J. Murrell
When we use git on a network filesystem, occasionally and sporadically we will see the following from a git checkout command: error: git checkout-index: unable to create file foo (File exists) Through a very basic grepping and following of the source it seems that the core of the error message

Re: Lack of netiquette, was Re: [PATCH v4 00/13] New remote-hg helper

2012-11-01 Thread Jonathan Nieder
Junio C Hamano wrote: We've been hoping we can do without a rigid code of conduct written down to maintain cordial community focused on technical merits, and instead relied on people's common sense, but sense may not be so common, unfortunately, so we may have to have one. I think that

Re: Wrap commit messages on `git commit -m`

2012-11-01 Thread James Nylen
On Thu, Nov 1, 2012 at 2:12 PM, Thomas Adam tho...@xteddy.org wrote: Hi, On 1 November 2012 16:07, Ramkumar Ramachandra artag...@gmail.com wrote: Hi, Some of my colleagues are lazy to fire up an editor and write proper commit messages- they often write one-liners using `git commit -m`.

Re: [PATCH v2 0/3] Introduce diff.submodule

2012-11-01 Thread Jens Lehmann
Am 01.11.2012 11:43, schrieb Ramkumar Ramachandra: Hi, v1 is here: http://mid.gmane.org/1349196670-2844-1-git-send-email-artag...@gmail.com I've fixed the issues pointed out in the review by Jens. Thanks, looking good to me. -- To unsubscribe from this list: send the line unsubscribe git

Re: Wrap commit messages on `git commit -m`

2012-11-01 Thread Lars Gullik Bjønnes
Ramkumar Ramachandra artag...@gmail.com writes: | Hi, | Some of my colleagues are lazy to fire up an editor and write proper | commit messages- they often write one-liners using `git commit -m`. | However, that line turns out to be longer than 72 characters, and the | resulting `git log` output

Re: Wrap commit messages on `git commit -m`

2012-11-01 Thread Kevin
As I see it, the problem is not the possibility to add new lines, but colleagues being too lazy to add them. On Thu, Nov 1, 2012 at 5:29 PM, Lars Gullik Bjønnes lar...@gullik.org wrote: Ramkumar Ramachandra artag...@gmail.com writes: | Hi, | Some of my colleagues are lazy to fire up an

Request: git fetch --timeout

2012-11-01 Thread H. Peter Anvin
Right now, there are events which can cause git fetch to hang forever. This is really problematic in automatic use. A git fetch --timeout which would exit with a nonzero exit code if no forward progress is made would be really helpful. -hpa -- To unsubscribe from this list: send the line

Re: Wrap commit messages on `git commit -m`

2012-11-01 Thread Jonathan Nieder
Kevin wrote: As I see it, the problem is not the possibility to add new lines, but colleagues being too lazy to add them. I suspect the underlying problem is that we make it too hard to tell git which text editor to run. Ram, what platform do your colleagues use? Thanks, Jonathan -- To

Re: What's cooking in git.git (Oct 2012, #09; Mon, 29)

2012-11-01 Thread Chris Rorvick
On Thu, Nov 1, 2012 at 1:40 PM, Ramsay Jones ram...@ramsay1.demon.co.uk wrote: Jeff King wrote: What's cooking in git.git (Oct 2012, #09; Mon, 29) -- [snip] * cr/cvsimport-local-zone (2012-10-16) 1 commit - git-cvsimport: allow

Overlong lines with git-merge --log

2012-11-01 Thread Tim Janik
Hey all. Using git-merge --log to merge commit messages that spawn multiple lines will produce overlong summary lines. That's because each summary line for a commit includes the entire commit message (all lines concatenated). According to git-fmt-merge-msg(1), --log should 'populate the log

[PATCH v4 00/14] fast-export and remote-testgit improvements

2012-11-01 Thread Felipe Contreras
Hi, I found more issues in fast-export. remote-testgit, and eventually I decided there's no reason to use this python script, so I wrote a much simpler version that does the same, and more. I'm not going to list all the reasons because apparently opinions are not welcome in the list any more. For

[PATCH v4 01/14] fast-export: avoid importing blob marks

2012-11-01 Thread Felipe Contreras
We want to be able to import, and then export, using the same marks, so that we don't push things that the other side already received. Unfortunately, fast-export doesn't store blobs in the marks, but fast-import does. This creates a mismatch when fast export is reusing a mark that were

[PATCH v4 02/14] remote-testgit: fix direction of marks

2012-11-01 Thread Felipe Contreras
Basically this is what we want: == pull == testgit transport-helper * export - import # testgit.marks git.marks == push == testgit transport-helper * import- export #

[PATCH v4 03/14] Rename git-remote-testgit to git-remote-testpy

2012-11-01 Thread Felipe Contreras
This script is not really exercising the remote-helper functionality, but more the python framework for remote helpers that live in git_remote_helpers. It's also not a good example of how to write remote-helpers, unless you are planning to use python, and even then you might not want to use this

[PATCH v4 04/14] Add new simplified git-remote-testgit

2012-11-01 Thread Felipe Contreras
It's way simpler. It exerceises the same features of remote helpers. It's easy to read and understand. It doesn't depend on python. It does _not_ exercise the python remote helper framework; there's another tool and another test for that. For now let's just copy the old remote-helpers test

[PATCH v4 06/14] remote-testgit: remove irrelevant test

2012-11-01 Thread Felipe Contreras
Only makes sense for remote-testpy. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- t/t5801-remote-helpers.sh | 13 - 1 file changed, 13 deletions(-) diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh index 2eca42e..a03d087 100755 ---

[PATCH v4 07/14] remote-testgit: cleanup tests

2012-11-01 Thread Felipe Contreras
We don't need a bare 'server' and an intermediary 'public'. The repos can talk to each other directly; that's what we want to exercise. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- t/t5801-remote-helpers.sh | 67 +++ 1 file changed,

[PATCH v4 08/14] remote-testgit: exercise more features

2012-11-01 Thread Felipe Contreras
Unfortunately they do not work. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- git-remote-testgit| 18 + t/t5801-remote-helpers.sh | 49 +++ 2 files changed, 63 insertions(+), 4 deletions(-) mode change 100755 =

[PATCH v4 09/14] remote-testgit: report success after an import

2012-11-01 Thread Felipe Contreras
Doesn't make a difference for the tests, but it does for the ones seeking reference. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- git-remote-testgit | 11 +++ 1 file changed, 11 insertions(+) diff --git a/git-remote-testgit b/git-remote-testgit index 6c348b0..4e8b356

[PATCH v4 10/14] remote-testgit: make clear the 'done' feature

2012-11-01 Thread Felipe Contreras
People seeking for reference would find it useful. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- git-remote-testgit | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/git-remote-testgit b/git-remote-testgit index 4e8b356..1116587 100755 ---

[PATCH v4 12/14] fast-export: fix comparison in tests

2012-11-01 Thread Felipe Contreras
First the expected, then the actual, otherwise the diff would be the opposite of what we want. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- t/t9350-fast-export.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/t/t9350-fast-export.sh

[PATCH v4 13/14] fast-export: make sure updated refs get updated

2012-11-01 Thread Felipe Contreras
When an object has already been exported (and thus is in the marks) it's flagged as SHOWN, so it will not be exported again, even if in a later time it's exported through a different ref. We don't need the object to be exported again, but we want the ref updated, which doesn't happen. Since we

[PATCH v4 11/14] fast-export: trivial cleanup

2012-11-01 Thread Felipe Contreras
Setting 'commit' to 'commit' is a no-op. It might have been there to avoid a compiler warning, but if so, it was the compiler to blame, and it's certainly not there any more. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- builtin/fast-export.c | 2 +- 1 file changed, 1

[PATCH v4 14/14] fast-export: don't handle uninteresting refs

2012-11-01 Thread Felipe Contreras
They have been marked as UNINTERESTING for a reason, lets respect that. Currently the first ref is handled properly, but not the rest, so: % git fast-export master ^master Would currently throw a reset for master (2nd ref), which is not what we want. % git fast-export master ^foo ^bar ^roo