[PATCH] How to keep a project's canonical history correct.

2014-05-09 Thread Stephen P. Smith
During the mail thread about Pull is mostly evil a user asked how the first parent could become reversed. This howto explains how the first parent can get reversed when viewed by the project and then explains a method to keep the history correct. Signed-off-by: Stephen P. Smith isch...@cox.net

Re: [PATCH v1 02/25] contrib: remove 'hg-to-git'

2014-05-09 Thread Miklos Vajna
On Thu, May 08, 2014 at 07:58:13PM -0500, Felipe Contreras felipe.contre...@gmail.com wrote: There hasn't been any real activity on it since 2010. Plus there are better out-of-tree tools. No tests and no real documentation either. ACK, git clone hg::... is what one is supposed to use

Re: Is there any efficient way to track history of a piece of code?

2014-05-09 Thread David Lang
On Thu, 8 May 2014, Jianyu Zhan wrote: That being said, storing abstract syntax tree, instead of raw object in git. But that goes too far... well, there are the clean/smudge filters, you can have it run everything through a prettyprinter as it's checked in. with a little effort, you could

Re: Watchman support for git

2014-05-09 Thread David Lang
On Thu, 8 May 2014, Sebastian Schuberth wrote: On 03.05.2014 05:40, Felipe Contreras wrote: That's very interesting. Do you get similar improvements when doing something similar in Merurial (watchman vs . no watchman). I have not tried it. My understanding is that this is why Facebook

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: No updates since 2010, and no tests. Who benefits from this removal? Is this causing a maintenance burden for Junio? It is cruft that nobody uses and we are not even testing. Plus, foreign SCM tools should live

Re: Conforming to pep8

2014-05-09 Thread Felipe Contreras
William Giokas wrote: On Thu, May 08, 2014 at 11:36:29PM -0500, Felipe Contreras wrote: William Giokas wrote: E401: Multi-line imports seems like something that would just be changing one line Yes, and make the code very annoying. It's 1 extra line in git-remote-hg, and 4 lines in

Re: Output from git blame A..B -- path for the bottom commit is misleading

2014-05-09 Thread David Kastrup
Jeff King p...@peff.net writes: I'd actually be inclined to say the opposite of what Junio is saying there: that -b should blank the author field as well as the commit sha1. I'd even go so far as to say that -b should probably be the default when boundary commits are in use. I cannot think of

Re: Conforming to pep8

2014-05-09 Thread William Giokas
On Fri, May 09, 2014 at 02:18:54AM -0500, Felipe Contreras wrote: William Giokas wrote: On Thu, May 08, 2014 at 11:36:29PM -0500, Felipe Contreras wrote: William Giokas wrote: E401: Multi-line imports seems like something that would just be changing one line Yes, and make the

Re: Conforming to pep8

2014-05-09 Thread Felipe Contreras
William Giokas wrote: On Fri, May 09, 2014 at 02:18:54AM -0500, Felipe Contreras wrote: William Giokas wrote: On Thu, May 08, 2014 at 11:36:29PM -0500, Felipe Contreras wrote: William Giokas wrote: E401: Multi-line imports seems like something that would just be changing one

Re: [PATCH] contrib/subtree bugfix: Can't `add` annotated tag

2014-05-09 Thread James Denholm
Junio C Hamano gits...@pobox.com wrote: The rev (not revs) seems to be used by more things than the final commit-tree state. Are we losing some useful information by peeling it too early like this patch does? (...) You're not wrong, actually, peeling at the last minute (or at least later)

Re: Conforming to pep8

2014-05-09 Thread William Giokas
On Fri, May 09, 2014 at 02:35:34AM -0500, Felipe Contreras wrote: William Giokas wrote: Yeah. In fact, for the mercurial stuff the `from mercurial import changegroup` line should be on the same line as the other `from mercurial import ...` line. I think the line is too big, it should

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras felipe.contre...@gmail.com wrote: No activity since 2010, no documentation, no tests. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/buildsystems/Generators.pm| 42 -- contrib/buildsystems/Generators/QMake.pm

Re: Conforming to pep8

2014-05-09 Thread John Keeping
On Thu, May 08, 2014 at 08:54:29PM -0500, William Giokas wrote: So I have been looking into the python code in the git tree recently (contrib and core tree) and noticed that almost none of the files fully conform to pep8. Now I'm not just saying this because I like the code to be clean,

[Bug] git-email: sendemail.bcc in config file overrides command line option --bcc

2014-05-09 Thread Adam Lee
BugLink: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=747068 --bcc should have higher priority than sendemail.bcc. --bcc=address Specify a Bcc: value for each email. Default is the value of sendemail.bcc. The --bcc option must be repeated for each user you want on the bcc list.

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras felipe.contre...@gmail.com wrote: No activity since 2010, no documentation, no tests. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/buildsystems/Generators.pm| 42 --

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Eric Wong
Felipe Contreras felipe.contre...@gmail.com wrote: Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: No updates since 2010, and no tests. Who benefits from this removal? Is this causing a maintenance burden for Junio? It is cruft that nobody uses and we are not

Re: [PATCH v1 18/25] contrib: remove 'emacs'

2014-05-09 Thread Felipe Contreras
David Kågedal wrote: What problem does this removal solve? Please do not top post. a) What problem does it solve by staying? b) Where are the tests? c) Why it cannot be moved to an outside repository like may other git-related tools? 2014-05-09 2:58 GMT+02:00 Felipe Contreras

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 2:58 AM, Felipe Contreras felipe.contre...@gmail.com wrote: No activity since 2010, no documentation, no tests. Signed-off-by: Felipe Contreras

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: No updates since 2010, and no tests. Who benefits from this removal? Is this causing a maintenance burden for Junio? It is cruft that

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: If you want this script to remain in contrib, please: a) Write at least a few tests b) Write some documentation c) Explain why it cannot live outside the git.git repository like

Re: [PATCH v1 18/25] contrib: remove 'emacs'

2014-05-09 Thread Felipe Contreras
David Kågedal wrote: 2014-05-09 10:29 GMT+02:00 Felipe Contreras felipe.contre...@gmail.com: David Kågedal wrote: What problem does this removal solve? Please do not top post. a) What problem does it solve by staying? b) Where are the tests? c) Why it cannot be moved to

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Eric Wong
Felipe Contreras felipe.contre...@gmail.com wrote: As a minimal token that anybody might possibly be using it, I would like to see it work at least once. Since you said you have arch repos, can you confirm that it does something? Those repos are in offline/offsite storage and I do not have

Re: t5539 fails on ubuntu for v2.0.0-rc2 - the SAME on 12.04 server - this test was disabled up to 1.9.2

2014-05-09 Thread Fabio D'Alfonso
Hi, this test was disabled up to 1.9.2 so it did not fail as it did not run by default. I have set the variable to run and got an error also in previous version, while never had problem in use. So I simply go on as there will no worst than before. It seems that no one is interested to this

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: If you want this script to remain in contrib, please: a) Write at least a few tests b) Write

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras felipe.contre...@gmail.com writes: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: If you want this script to remain in contrib, please: a) Write at least a few tests b) Write some documentation c) Explain

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: As a minimal token that anybody might possibly be using it, I would like to see it work at least once. Since you said you have arch repos, can you confirm that it does something? Those repos are in offline/offsite

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: If you want this script to remain in contrib, please: a) Write at

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
David Kastrup wrote: Felipe Contreras felipe.contre...@gmail.com writes: Besides, you say No activity since 2010 - this is not the case, bc380fc is from November 2013. You think changing the execution bit of a file is considered activity? For an indication of this software has actual

Re: Bash v4 on msysgit?

2014-05-09 Thread Thomas Braun
How can I get Bash v4 for msysgit 1.9.2? I need it for 'globstar' shopt support. Thanks in advance. Unfortunately you can't. Newer bash versions don't work with the current msys layer. See http://mingw.5.n7.nabble.com/bash-4-x-for-MSYS-td5605.html for a more in depth discussion. -- To

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras felipe.contre...@gmail.com writes: David Kastrup wrote: The idea of removing software from distribution is to get rid of stuff without a user base rather than punishing lazy developers. No. So we have you on record that you would want to get rid of stuff _with_ a user base

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com

Re: [PATCH 00/32] Split index mode for very large indexes

2014-05-09 Thread Duy Nguyen
On Mon, Apr 28, 2014 at 02:18:44PM -0700, Shawn Pearce wrote: The read penalty is not addressed here, so I still pay 14MB hashing cost. But that's an easy problem. We could cache the validated index in a daemon. Whenever git needs to load an index, it pokes the daemon. The daemon verifies

Re: [PATCH v1 25/25] contrib: remove 'mw-to-git'

2014-05-09 Thread Matthieu Moy
Felipe Contreras felipe.contre...@gmail.com writes: No chance of ever graduating. I see no relationship between the chance of graduating and the removal from contrib/. If you want to remove mw-to-git from contrib, then a good starting point would be to explain why you want to do so in the

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
David Kastrup wrote: Felipe Contreras felipe.contre...@gmail.com writes: David Kastrup wrote: The idea of removing software from distribution is to get rid of stuff without a user base rather than punishing lazy developers. No. So we have you on record that you would want to get

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Felipe Contreras
Erik Faye-Lund wrote: On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras You think changing the execution bit of a file is considered activity? Well, now we're getting into

Re: [PATCH v1 25/25] contrib: remove 'mw-to-git'

2014-05-09 Thread Felipe Contreras
Matthieu Moy wrote: Felipe Contreras felipe.contre...@gmail.com writes: No chance of ever graduating. I see no relationship between the chance of graduating and the removal from contrib/. Read contrib/README. If you want to remove mw-to-git from contrib, then a good starting point

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread David Kastrup
Felipe Contreras felipe.contre...@gmail.com writes: David Kastrup wrote: Felipe Contreras felipe.contre...@gmail.com writes: David Kastrup wrote: The idea of removing software from distribution is to get rid of stuff without a user base rather than punishing lazy developers. No.

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Erik Faye-Lund
On Fri, May 9, 2014 at 12:57 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:48 AM, Felipe Contreras You think changing the

Re: [PATCH v1 25/25] contrib: remove 'mw-to-git'

2014-05-09 Thread Stefan Beller
On 09.05.2014 12:59, Felipe Contreras wrote: Matthieu Moy wrote: Felipe Contreras felipe.contre...@gmail.com writes: No chance of ever graduating. I see no relationship between the chance of graduating and the removal from contrib/. Read contrib/README. If you want to remove mw-to-git

Re: Bash v4 on msysgit?

2014-05-09 Thread Robert Dailey
Thanks for the info. Really quite disappointing that the discussion linked was from 3 years ago but still no progress. I wish I could use cygwin to use GIT but LESS doesn't work for log, reflog, and other commands. On Fri, May 9, 2014 at 4:37 AM, Thomas Braun thomas.br...@virtuell-zuhause.de

Re: [PATCH v1 00/25] contrib: cleanup

2014-05-09 Thread Stefan Beller
For discussing these larger changes in the first version you may want to use the -D option of git format-patch? 2014-05-09 4:01 GMT+02:00 Felipe Contreras felipe.contre...@gmail.com: Martin Langhoff wrote: On Thu, May 8, 2014 at 8:58 PM, Felipe Contreras felipe.contre...@gmail.com wrote:

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Stefan Beller
It is serious. The purpose of the 'contrib/' area is not clear. The statemens coming from Junio don't match what is on 'contrib/README'. So we have a huge variance of quality all over 'contrib/'. Some tools in contrib have higher quality than what is part of the core (e.g. they have tests,

Re: [PATCH] How to keep a project's canonical history correct.

2014-05-09 Thread Stephen Smith
On May 8, 2014, at 11:08 PM, Stephen P. Smith isch...@cox.net wrote: During the mail thread about Pull is mostly evil a user asked how the first parent could become reversed. This howto explains how the first parent can get reversed when viewed by the project and then explains a method

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Felipe Contreras
Stefan Beller wrote: It is serious. The purpose of the 'contrib/' area is not clear. The statemens coming from Junio don't match what is on 'contrib/README'. So we have a huge variance of quality all over 'contrib/'. Some tools in contrib have higher quality than what is part of the

[ANNOUNCE] git-remote-hg 0.2

2014-05-09 Thread Felipe Contreras
Hi, git-remote-hg is a bidirectional bridge between Git and Mercurial. It is production-ready, has been widely tested, and was previously part of git.git. Junio C Hamano has retracted from his previous statements where he wanted these tools to become part of the Git core and distributed by

Re: [PATCH v1 18/25] contrib: remove 'emacs'

2014-05-09 Thread Felipe Contreras
Alexandre Julliard wrote: David Kågedal dav...@lysator.liu.se writes: 2014-05-09 10:29 GMT+02:00 Felipe Contreras felipe.contre...@gmail.com: David Kågedal wrote: What problem does this removal solve? Please do not top post. a) What problem

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Jeff King
On Fri, May 09, 2014 at 04:20:40AM -0500, Felipe Contreras wrote: Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: As a minimal token that anybody might possibly be using it, I would like to see it work at least once. Since you said you have arch repos, can you

Re: [PATCH v1 23/25] contrib: remove 'hooks/multimail'

2014-05-09 Thread Michael Haggerty
On 05/09/2014 02:58 AM, Felipe Contreras wrote: No tests. No chance of ever graduating. Already out-of-tree. Cc: Michael Haggerty mhag...@alum.mit.edu Signed-off-by: Felipe Contreras felipe.contre...@gmail.com Thank you for your input. git-multimail is maintained outside of the Git

Re: [PATCH v1 23/25] contrib: remove 'hooks/multimail'

2014-05-09 Thread David Kastrup
Michael Haggerty mhag...@alum.mit.edu writes: On 05/09/2014 02:58 AM, Felipe Contreras wrote: No tests. No chance of ever graduating. Already out-of-tree. Cc: Michael Haggerty mhag...@alum.mit.edu Signed-off-by: Felipe Contreras felipe.contre...@gmail.com Thank you for your input.

Re: [PATCH v1 18/25] contrib: remove 'emacs'

2014-05-09 Thread Alexandre Julliard
David Kågedal dav...@lysator.liu.se writes: 2014-05-09 10:29 GMT+02:00 Felipe Contreras felipe.contre...@gmail.com: David Kågedal wrote: What problem does this removal solve? Please do not top post. a) What problem does it solve by staying? b) Where

Re: [PATCH v1 06/25] contrib: remove 'diffall'

2014-05-09 Thread Tim Henigan
On Thu, May 8, 2014 at 5:58 PM, Felipe Contreras felipe.contre...@gmail.com wrote: No activity since 2010, no tests. Cc: Tim Henigan tim.heni...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/diffall/README | 31 -- contrib/diffall/git-diffall |

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Jeff King wrote: On Fri, May 09, 2014 at 04:20:40AM -0500, Felipe Contreras wrote: Eric Wong wrote: Felipe Contreras felipe.contre...@gmail.com wrote: As a minimal token that anybody might possibly be using it, I would like to see it work at least once. Since you said you have arch

Re: [PATCH v1 23/25] contrib: remove 'hooks/multimail'

2014-05-09 Thread Felipe Contreras
Michael Haggerty wrote: On 05/09/2014 02:58 AM, Felipe Contreras wrote: No tests. No chance of ever graduating. Already out-of-tree. Cc: Michael Haggerty mhag...@alum.mit.edu Signed-off-by: Felipe Contreras felipe.contre...@gmail.com Thank you for your input. git-multimail is

Re: [PATCH v1 07/25] contrib: remove 'git-jump'

2014-05-09 Thread Jeff King
On Thu, May 08, 2014 at 09:12:36PM -0500, Felipe Contreras wrote: Jeff King wrote: On Thu, May 08, 2014 at 07:58:18PM -0500, Felipe Contreras wrote: No activity, no tests. Like diff-highlight, I don't think no activity is a useful indicator. I use this daily, and several people

Re: [PATCH v1 06/25] contrib: remove 'diffall'

2014-05-09 Thread Felipe Contreras
Tim Henigan wrote: On Thu, May 8, 2014 at 5:58 PM, Felipe Contreras felipe.contre...@gmail.com wrote: No activity since 2010, no tests. Cc: Tim Henigan tim.heni...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/diffall/README | 31 --

Re: [PATCH v1 00/25] contrib: cleanup

2014-05-09 Thread Jeff King
On Thu, May 08, 2014 at 09:01:32PM -0500, Felipe Contreras wrote: Are you planning on CC'ing the (inactive) authors/maintainers so they know that if they care they should host those elsewhere? They are already Cc'ed. I don't think you were very thorough on this. Of the three remaining

Re: [PATCH v1 23/25] contrib: remove 'hooks/multimail'

2014-05-09 Thread Michael Haggerty
On 05/09/2014 05:04 PM, David Kastrup wrote: Michael Haggerty mhag...@alum.mit.edu writes: On 05/09/2014 02:58 AM, Felipe Contreras wrote: No tests. No chance of ever graduating. Already out-of-tree. Cc: Michael Haggerty mhag...@alum.mit.edu Signed-off-by: Felipe Contreras

Re: Output from git blame A..B -- path for the bottom commit is misleading

2014-05-09 Thread Jeff King
On Fri, May 09, 2014 at 07:04:05AM +0200, David Kastrup wrote: Arguably if the user explicitly limited the range, he knows what he's looking at. Admittedly, I don't know offhand which options _will_ produce boundary commit indications: there may be some without explicit range limitation, and

Re: [Bug] git-email: sendemail.bcc in config file overrides command line option --bcc

2014-05-09 Thread Jeff King
On Fri, May 09, 2014 at 04:13:31PM +0800, Adam Lee wrote: BugLink: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=747068 --bcc should have higher priority than sendemail.bcc. --bcc=address Specify a Bcc: value for each email. Default is the value of sendemail.bcc. The

Re: t5539 fails on ubuntu for v2.0.0-rc2 - the SAME on 12.04 server - this test was disabled up to 1.9.2

2014-05-09 Thread Jeff King
On Fri, May 09, 2014 at 10:39:32AM +0200, Fabio D'Alfonso wrote: this test was disabled up to 1.9.2 so it did not fail as it did not run by default. Yes, in v1.9.2 we turned on http tests to run by default, but only if we could succeed in setting up the http server automatically (and if we

Re: [PATCH v1 00/25] contrib: cleanup

2014-05-09 Thread Felipe Contreras
Jeff King wrote: On Thu, May 08, 2014 at 09:01:32PM -0500, Felipe Contreras wrote: Are you planning on CC'ing the (inactive) authors/maintainers so they know that if they care they should host those elsewhere? They are already Cc'ed. I don't think you were very thorough on this.

Re: t5539 fails on ubuntu for v2.0.0-rc2

2014-05-09 Thread Jeff King
On Thu, May 08, 2014 at 08:02:28AM +0200, Fabio D'Alfonso wrote: this is the error in httpd error.log [Wed May 07 20:44:10 2014] [alert] getpwuid: couldn't determine user name from uid 4294967295, you probably need to modify the User directive [Wed May 07 20:44:10 2014] [notice]

Re: Conforming to pep8

2014-05-09 Thread W. Trevor King
On Fri, May 09, 2014 at 02:44:02AM -0500, William Giokas wrote: Maybe a time to use something like:: from mercurial import foo \ bar \ baz \ ... Would make that import into quite a few lines, but would help

Re: [PATCH v1 07/25] contrib: remove 'git-jump'

2014-05-09 Thread Felipe Contreras
Jeff King wrote: On Thu, May 08, 2014 at 09:12:36PM -0500, Felipe Contreras wrote: Jeff King wrote: On Thu, May 08, 2014 at 07:58:18PM -0500, Felipe Contreras wrote: No activity, no tests. Like diff-highlight, I don't think no activity is a useful indicator. I use this

Re: Conforming to pep8

2014-05-09 Thread Felipe Contreras
W. Trevor King wrote: The indentation for the closing parenthesis is optional [2]. You can of course do things like: from mercurial import ( bar, baz, foo, ) I prefer: from mercurial foo, bar, ... from mercurial baz, ... -- Felipe Contreras -- To unsubscribe

[Closed] git-email: sendemail.bcc in config file overrides command line option --bcc

2014-05-09 Thread Adam Lee
On Fri, May 09, 2014 at 11:45:09AM -0400, Jeff King wrote: On Fri, May 09, 2014 at 04:13:31PM +0800, Adam Lee wrote: BugLink: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=747068 --bcc should have higher priority than sendemail.bcc. --bcc=address Specify a Bcc: value for

[ANNOUNCE] git-remote-bzr 0.2

2014-05-09 Thread Felipe Contreras
Hi, git-remote-bzr is a bidirectional bridge between Git and Bazaar. It is production-ready, has been widely tested, and was previously part of git.git. As I already explained[1], there is no path forward for git-remote-hg and git-remote-bzr; Junio C Hamano has retracted from his previous

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Felipe Contreras wrote: All right, I guess that' something, but I get: Use of each() on hash after insertion without resetting hash iterator results in undefined behavior, Perl interpreter: 0x1fec010 at /usr/lib/git-core/git-archimport line 129. And a ton of: WARNING: no rule

Re: [PATCH v1 23/25] contrib: remove 'hooks/multimail'

2014-05-09 Thread James Denholm
Felipe Contreras felipe.contre...@gmail.com wrote: Michael Haggerty wrote: What's more, it has a maintainer who doesn't routinely insult other people on the mailing list, Aaand skiping the rest. Good bye. Aww, Felipe, it's no fair when you ignore other people's slag. How ever will anyone argue

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Thu, May 08, 2014 at 07:58:30PM -0500, Felipe Contreras wrote: No activity since 2012, no tests, no chance of ever graduating. I don't think no activity is an interesting indicator. This tool _is_ actively maintained, but it has not needed any fixes since

Re: [PATCH v1 00/25] contrib: cleanup

2014-05-09 Thread Martin Langhoff
On Fri, May 9, 2014 at 11:57 AM, Felipe Contreras felipe.contre...@gmail.com wrote: I already explained: That's right, and they are Cc'ed so they can respond. Some tools have only one commit or two, and in those I didn't even bother Cc'ing anyone. contrib/persistent-https consist of a

Re: What's cooking in git.git (Apr 2014, #08; Fri, 25)

2014-05-09 Thread Felipe Contreras
Michael Haggerty wrote: On 04/26/2014 01:19 AM, Jeff King wrote: On Fri, Apr 25, 2014 at 03:50:26PM -0700, Junio C Hamano wrote: [...] * fc/publish-vs-upstream (2014-04-21) 8 commits - sha1_name: add support for @{publish} marks - sha1_name: simplify track finding - sha1_name:

Re: [PATCH v1 00/25] contrib: cleanup

2014-05-09 Thread Felipe Contreras
Martin Langhoff wrote: On Fri, May 9, 2014 at 11:57 AM, Felipe Contreras felipe.contre...@gmail.com wrote: I already explained: That's right, and they are Cc'ed so they can respond. Some tools have only one commit or two, and in those I didn't even bother Cc'ing anyone.

Re: [PATCH 0/4] remote-hg: more improvements

2014-05-09 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: I already said this multiple times, but let me be clear once more: MASTER HAS A REGRESSION (for all versions of Mercurial). As you said, that is not a

Re: Watchman support for git

2014-05-09 Thread David Turner
On Fri, 2014-05-09 at 00:08 -0700, David Lang wrote: On Thu, 8 May 2014, Sebastian Schuberth wrote: On 03.05.2014 05:40, Felipe Contreras wrote: That's very interesting. Do you get similar improvements when doing something similar in Merurial (watchman vs . no watchman). I have not

Re: Output from git blame A..B -- path for the bottom commit is misleading

2014-05-09 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Fri, May 09, 2014 at 07:04:05AM +0200, David Kastrup wrote: Arguably if the user explicitly limited the range, he knows what he's looking at. Admittedly, I don't know offhand which options _will_ produce boundary commit indications: there may be some

Re: t5539 fails on ubuntu for v2.0.0-rc2

2014-05-09 Thread Fabio D'Alfonso
Hi, yes, the problem comes from root. I made a make test using git user in my 12.04 server where there is a gitlab and the test went on for 5539. I think the 3 could be the better solution. This is not the place to discuss about the sudo mania, a disease I did not get and hope not to get, but

Re: [PATCH v1 07/25] contrib: remove 'git-jump'

2014-05-09 Thread Junio C Hamano
Jeff King p...@peff.net writes: And speaking of wasted time, I do not plan on responding further to you in this thread. I am telling you ahead of time that this is the case, because elsewhere[1] I saw you complaining that Junio did not respond to your emails, which you seemed to think was

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Junio C Hamano
Eric Wong normalper...@yhbt.net writes: Felipe Contreras felipe.contre...@gmail.com wrote: No updates since 2010, and no tests. Who benefits from this removal? Is this causing a maintenance burden for Junio? No. See http://thread.gmane.org/gmane.comp.version-control.git/248587 Plus,

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: There is no prove yourself is worthy or get evicted purge going on in the contrib/ area. I saw contrib/README referred to a few times in the near-by threads, and I think these patches are done primarily by deliberately misinterpreting one part of it in order to grab

Re: What's cooking in git.git (Apr 2014, #08; Fri, 25)

2014-05-09 Thread Philip Oakley
From: Michael Haggerty mhag...@alum.mit.edu On 04/26/2014 01:19 AM, Jeff King wrote: On Fri, Apr 25, 2014 at 03:50:26PM -0700, Junio C Hamano wrote: [...] * fc/publish-vs-upstream (2014-04-21) 8 commits - sha1_name: add support for @{publish} marks - sha1_name: simplify track finding -

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: Eric Wong normalper...@yhbt.net writes: Felipe Contreras felipe.contre...@gmail.com wrote: No updates since 2010, and no tests. Who benefits from this removal? Is this causing a maintenance burden for Junio? No. See

Re: [PATCH 00/32] Split index mode for very large indexes

2014-05-09 Thread Junio C Hamano
Duy Nguyen pclo...@gmail.com writes: On Mon, Apr 28, 2014 at 02:18:44PM -0700, Shawn Pearce wrote: The read penalty is not addressed here, so I still pay 14MB hashing cost. But that's an easy problem. We could cache the validated index in a daemon. Whenever git needs to load an index, it

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: *You* said this[1]: If you read the context you omitted from the quote, and realize that it was a counter-suggestion to give a middle ground to a more draconian let's divide them into two, neither which I said I want to see go forward

Re: Watchman support for git

2014-05-09 Thread David Lang
On Fri, 9 May 2014, David Turner wrote: On Fri, 2014-05-09 at 00:08 -0700, David Lang wrote: On Thu, 8 May 2014, Sebastian Schuberth wrote: On 03.05.2014 05:40, Felipe Contreras wrote: That's very interesting. Do you get similar improvements when doing something similar in Merurial

Re: Watchman support for git

2014-05-09 Thread David Turner
On Fri, 2014-05-09 at 11:08 -0700, David Lang wrote: On Fri, 9 May 2014, David Turner wrote: On Fri, 2014-05-09 at 00:08 -0700, David Lang wrote: On Thu, 8 May 2014, Sebastian Schuberth wrote: On 03.05.2014 05:40, Felipe Contreras wrote: That's very interesting. Do you get similar

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: *You* said this[1]: If you read the context you omitted from the quote, and realize that it was a counter-suggestion to give a middle ground to a more draconian let's divide them into two, neither which I said I

Re: Watchman support for git

2014-05-09 Thread David Lang
On Fri, 9 May 2014, David Turner wrote: On Fri, 2014-05-09 at 11:08 -0700, David Lang wrote: On Fri, 9 May 2014, David Turner wrote: On Fri, 2014-05-09 at 00:08 -0700, David Lang wrote: On Thu, 8 May 2014, Sebastian Schuberth wrote: On 03.05.2014 05:40, Felipe Contreras wrote: That's

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes: Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: *You* said this[1]: If you read the context you omitted from the quote, and realize that it was a counter-suggestion to give a middle ground to a more draconian

Re: [PATCH v1 04/25] contrib: remove 'buildsystems'

2014-05-09 Thread Jonathan Nieder
Hi, Erik Faye-Lund wrote: On Fri, May 9, 2014 at 10:14 AM, Felipe Contreras felipe.contre...@gmail.com wrote: Erik Faye-Lund wrote: Please don't. This script is useful to build with the MSVC IDE, which enables us to use their excellent debugger. If you want this script to remain in

Re: [PATCH v1 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: *You* said this[1]: If you read the context you omitted from the quote, and realize that it was a counter-suggestion to give a

Re: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Thomas Adam
On 9 May 2014 18:52, Felipe Contreras felipe.contre...@gmail.com wrote: Such incredible double standards. I think I speak for everyone when I say: fuck off. I'm sick and tired of seeing you pop up here with polemic and rhetoric, publicly outing those who are making Git great, just because

Re: Watchman support for git

2014-05-09 Thread David Turner
On Fri, 2014-05-09 at 11:27 -0700, David Lang wrote: That's not my understanding from Durham Goode's talk in January. Yes, operations involving history go to the server. But the client also maintains a copy of the working tree, and it is for this that watchman is used. Otherwise, why

Re: [PATCH v1 06/25] contrib: remove 'diffall'

2014-05-09 Thread Jonathan Nieder
Hi Tim, Tim Henigan wrote: On Thu, May 8, 2014 at 5:58 PM, Felipe Contreras felipe.contre...@gmail.com wrote: contrib/diffall/README | 31 -- contrib/diffall/git-diffall | 257 2 files changed, 288 deletions(-) delete mode 100644

[PATCH v2 01/17] contrib: remove outdated README

2014-05-09 Thread Felipe Contreras
There is no guideline as for what should be part of contrib. Some tools are actively maintained, others consist of a single commit. Some tools have active user-base, some aren't used by anyone. Some tools are on the path towards the core, others will never get there. Some tools are already

[PATCH v2 00/17] contrib: cleanup

2014-05-09 Thread Felipe Contreras
The contrib area is full of accumulatted cruft. Let's remove what is not used and what is already maintained externay. Felipe Contreras (17): contrib: remove outdated README contrib: remove 'vim' contrib: remove 'emacs' contrib: remove 'diffall' contrib: remove 'hg-to-git' contrib:

[PATCH v2 02/17] contrib: remove 'vim'

2014-05-09 Thread Felipe Contreras
There are only instructions for old versions of vim (7.2) which don't apply since six years. The vast majority of people don't need these instructions. Let's remove them. Cc: Jonathan Nieder jrnie...@gmail.com Cc: Jeff King p...@peff.net Signed-off-by: Felipe Contreras

[PATCH v2 05/17] contrib: remove 'hg-to-git'

2014-05-09 Thread Felipe Contreras
There hasn't been any real activity on it since 2010. Plus there are better out-of-tree tools. No tests and no real documentation either. Cc: Miklos Vajna vmik...@frugalware.org Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/hg-to-git/hg-to-git.py | 255

[PATCH v2 04/17] contrib: remove 'diffall'

2014-05-09 Thread Felipe Contreras
There is no more need for this tool since the --dir-dirr option was introduced. Cc: Tim Henigan tim.heni...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/diffall/README | 31 -- contrib/diffall/git-diffall | 257

[PATCH v2 10/17] contrib: reomve 'thunderbird-patch-inline'

2014-05-09 Thread Felipe Contreras
No activity, no tests. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/thunderbird-patch-inline/README | 20 contrib/thunderbird-patch-inline/appp.sh | 55 2 files changed, 75 deletions(-) delete mode 100644

  1   2   >