Re: Watchman support for git

2014-05-09 Thread Duy Nguyen
On Sat, May 3, 2014 at 6:14 AM, wrote: > The most sigificant patch uses Facebook's watchman daemon[1] to monitor > the repository work tree for changes. This makes allows git status > to avoid traversing the entire work tree to find changes. I got "warning: Watchman watch error: Got bad JSON fr

Re: [PATCH v1 20/25] contrib: remove 'contacts'

2014-05-09 Thread Felipe Contreras
brian m. carlson wrote: > On Thu, May 08, 2014 at 07:58:31PM -0500, Felipe Contreras wrote: > > There are better out-of-tree tools, and this tool is not planned to move > > into the core anyway. > > I have used this once or twice, and I have seen others indicate their > use of it as well. I am no

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

2014-05-09 Thread Stephen & Linda Smith
On Friday, May 09, 2014 02:05:44 PM Junio C Hamano wrote: > I needed a few tweaks on top while queuing. You will find the > result on 'pu' after I push it out. > > In addition to one typofix ("because" lacking "c"), here are what I > did: > > - Typeset concrete command e.g. `git pull` in monosp

Re: [PATCH v1 20/25] contrib: remove 'contacts'

2014-05-09 Thread brian m. carlson
On Thu, May 08, 2014 at 07:58:31PM -0500, Felipe Contreras wrote: > There are better out-of-tree tools, and this tool is not planned to move > into the core anyway. I have used this once or twice, and I have seen others indicate their use of it as well. I am not volunteering to maintain this, but

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

2014-05-09 Thread Felipe Contreras
Eric Wong wrote: > Thomas Adam wrote: > > I think I speak for everyone when I say: fuck off. > > I wouldn't put it so harshly... > > Felipe: I suggest you take a long vacation away from development. Nah, I'm done. > Whatever good you may be able to contribute today is drowned out by your > beh

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

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > Felipe Contreras writes: > > > 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 towar

[PATCH 2/2] Make it possible to update git_wcwidth()

2014-05-09 Thread Torsten Bögershausen
The function git_wcwidth() returns for a given unicode code point the width on the display: -1 for control characters, 0 for combining or other non-visible code points 1 for e.g. ASCII 2 for double-width code points. This table had been originally been extracted for one Unicode version, probabl

[PATCH 1/2] utf8.c: use a table for double_width

2014-05-09 Thread Torsten Bögershausen
Refactor git_wcwidth() and replace the if-else-if chain. Use the table double_width which is scanned by the bisearch() function, which is already used to find combining code points. Signed-off-by: Torsten Bögershausen --- utf8.c | 41 ++--- 1 file changed, 18

RE: What's cooking in git.git (May 2014, #02; Fri, 9)

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > * fc/remote-helpers-hg-bzr-graduation (2014-04-29) 11 commits > - remote-hg: trivial cleanups > - remote-hg: make sure we omit multiple heads > - git-remote-hg: use internal clone's hgrc > - t: remote-hg: split into setup test > - remote-hg: properly detect missing cont

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

2014-05-09 Thread Junio C Hamano
Jeff King writes: > Does running the tests as a non-root user fix it? If so, I think we have > a few options in git: > > 1. Add a User directive to our httpd.conf. I doubt this is a good > idea to do unconditionally, as a non-root apache would probably be > unhappy with it. > > 2. A

[ANNOUNCE] Git v2.0.0-rc3

2014-05-09 Thread Junio C Hamano
A release candidate Git v2.0.0-rc3 is now available for testing at the usual places. This hopefully will be the last -rc before the real 2.0. The tarballs are found at: https://www.kernel.org/pub/software/scm/git/testing/ The following public repositories all have a copy of the 'v2.0.0-rc3'

What's cooking in git.git (May 2014, #02; Fri, 9)

2014-05-09 Thread Junio C Hamano
Here are the topics that have been cooking. Commits prefixed with '-' are only in 'pu' (proposed updates) while commits prefixed with '+' are in 'next'. The tip of the 'master' branch is at v2.0.0-rc3, which is in sync with v1.9.3 with respect to bugfixes that was also tagged today. Hopefully we

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

2014-05-09 Thread Junio C Hamano
"Stephen P. Smith" writes: > 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-

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

2014-05-09 Thread Tim Henigan
Hi Jonathan, On Fri, May 9, 2014 at 1:12 PM, Jonathan Nieder wrote: > Hi, > > Tim Henigan wrote: > >> However, I like the idea of simply removing the directory from contrib >> and pointing to 'difftool --dir-diff' in the release notes. > > Thanks. I believe Junio uses commit messages as referenc

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

2014-05-09 Thread Jonathan Nieder
Hi, Tim Henigan wrote: > However, I like the idea of simply removing the directory from contrib > and pointing to 'difftool --dir-diff' in the release notes. Thanks. I believe Junio uses commit messages as reference when writing release notes so hopefully this should be enough to make that happ

[ANNOUNCE] Git v1.9.3

2014-05-09 Thread Junio C Hamano
The latest maintenance release Git v1.9.3 is now available at the usual places. The tarballs are found at: https://www.kernel.org/pub/software/scm/git/testing/ The following public repositories will all have a copy of the 'v1.9.3' tag and the 'maint' branch that the tag points at: url = h

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

2014-05-09 Thread Junio C Hamano
Felipe Contreras writes: > 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 t

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

2014-05-09 Thread Johannes Sixt
I know I'm late in the show, but can we please, PLEASE *NOT* feed the troll? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: Pull is Mostly Evil

2014-05-09 Thread Marc Branchaud
After poking this hornet's nest I pretty much have stood back and not participated in the ensuing discussions. But having unleashed the hornets I feel I should at least say something, if only to assure people that I'm not ignoring their plight. There have been various proposals to modify git-pull

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

2014-05-09 Thread Tim Henigan
On Fri, May 9, 2014 at 12:11 PM, Felipe Contreras wrote: > There is no more need for this tool since the --dir-dirr option was > introduced. s/--dir-dirr/--dir-diff/ -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majord

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

2014-05-09 Thread Tim Henigan
Hi Jonathan, On Fri, May 9, 2014 at 11:50 AM, Jonathan Nieder wrote: > Hi Tim, > > Tim Henigan wrote: >> On Thu, May 8, 2014 at 5:58 PM, Felipe Contreras >> wrote: > >>> contrib/diffall/README | 31 -- >>> contrib/diffall/git-diffall | 257 >>>

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

2014-05-09 Thread Eric Wong
Thomas Adam wrote: > I think I speak for everyone when I say: fuck off. I wouldn't put it so harshly... Felipe: I suggest you take a long vacation away from development. Whatever good you may be able to contribute today is drowned out by your behavior. The projects you are involved with will g

[PATCH v2 17/17] contrib: remove 'contacts'

2014-05-09 Thread Felipe Contreras
There are better out-of-tree tools, and this tool is not planned to move into the core anyway. No tests either. Cc: Eric Sunshine Signed-off-by: Felipe Contreras --- contrib/contacts/git-contacts | 203 -- contrib/contacts/git-contacts.txt | 94

[PATCH v2 16/17] contrib: remove 'git-resurrect'

2014-05-09 Thread Felipe Contreras
No activity, no documentation, no tests, no chance of ever graduating. Signed-off-by: Felipe Contreras --- contrib/git-resurrect.sh | 182 --- 1 file changed, 182 deletions(-) delete mode 100755 contrib/git-resurrect.sh diff --git a/contrib/git-resur

[PATCH v2 15/17] contrib: remove 'persistent-https'

2014-05-09 Thread Felipe Contreras
No activity. No tests. No chance of ever moving into the core because it uses Go. Cc: Colby Ranger Signed-off-by: Felipe Contreras --- contrib/persistent-https/LICENSE | 202 - contrib/persistent-https/Makefile | 38 --- contrib/persistent-https/READ

[PATCH v2 13/17] contrib: remove 'rerere-train'

2014-05-09 Thread Felipe Contreras
No activity, no nothing. Signed-off-by: Felipe Contreras --- contrib/rerere-train.sh | 52 - 1 file changed, 52 deletions(-) delete mode 100755 contrib/rerere-train.sh diff --git a/contrib/rerere-train.sh b/contrib/rerere-train.sh deleted file mo

[PATCH v2 12/17] contrib: remove 'svn-fe'

2014-05-09 Thread Felipe Contreras
No activity since 2012, no tests. Cc: Jonathan Nieder Signed-off-by: Felipe Contreras --- contrib/svn-fe/.gitignore | 4 --- contrib/svn-fe/Makefile| 63 - contrib/svn-fe/svn-fe.c| 18 --- contrib/svn-fe/svn-fe.txt | 71

[PATCH v2 09/17] contrib: remove 'git-shell-commands'

2014-05-09 Thread Felipe Contreras
No activity, no tests. Signed-off-by: Felipe Contreras --- contrib/git-shell-commands/README | 18 -- contrib/git-shell-commands/help | 18 -- contrib/git-shell-commands/list | 10 -- 3 files changed, 46 deletions(-) delete mode 100644 contrib/git-she

[PATCH v2 14/17] contrib: remove 'remotes2config'

2014-05-09 Thread Felipe Contreras
No activity since 2007. No documentation, no tests. Signed-off-by: Felipe Contreras --- contrib/remotes2config.sh | 33 - 1 file changed, 33 deletions(-) delete mode 100755 contrib/remotes2config.sh diff --git a/contrib/remotes2config.sh b/contrib/remotes2config

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

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

[PATCH v2 08/17] contrib: remove 'convert-objects'

2014-05-09 Thread Felipe Contreras
No activity since 2010, no tests, no documentation. Signed-off-by: Felipe Contreras --- contrib/convert-objects/convert-objects.c | 329 contrib/convert-objects/git-convert-objects.txt | 29 --- 2 files changed, 358 deletions(-) delete mode 100644 contrib/convert

[PATCH v2 11/17] contrib: remove 'workdir'

2014-05-09 Thread Felipe Contreras
No activity since 2010, no documentation, no tests. Signed-off-by: Felipe Contreras --- contrib/workdir/git-new-workdir | 82 - 1 file changed, 82 deletions(-) delete mode 100755 contrib/workdir/git-new-workdir diff --git a/contrib/workdir/git-new-workdi

[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 Signed-off-by: Felipe Contreras --- contrib/diffall/README | 31 -- contrib/diffall/git-diffall | 257 2 files changed, 288 deletions(-) delete

[PATCH v2 07/17] contrib: remove 'stats'

2014-05-09 Thread Felipe Contreras
No real activity since 2012 (or ever), no tests, no documentation. Signed-off-by: Felipe Contreras --- contrib/stats/git-common-hash | 26 -- contrib/stats/mailmap.pl | 70 -- contrib/stats/packinfo.pl | 212 -- 3 files changed,

[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 Signed-off-by: Felipe Contreras --- contrib/hg-to-git/hg-to-git.py | 255 contrib/hg-to-git/hg-t

[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 Cc: Jeff King Signed-off-by: Felipe Contreras --- contrib/vim/README | 22 -- 1

[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: re

[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 out-of-

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 > wrote: >> contrib/diffall/README | 31 -- >> contrib/diffall/git-diffall | 257 >> >> 2 files changed, 288 deletions(-) >> delete mode 100644 contrib/diffall/

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. Otherw

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

2014-05-09 Thread Thomas Adam
On 9 May 2014 18:52, Felipe Contreras 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 you're not getting your own w

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

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > Felipe Contreras writes: > > > Junio C Hamano wrote: > >> Felipe Contreras 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 > >> drac

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 > 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 contrib, please: >>

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

2014-05-09 Thread Junio C Hamano
Felipe Contreras writes: > Junio C Hamano wrote: >> Felipe Contreras 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", neithe

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 ver

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

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > Felipe Contreras 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 fo

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 interesti

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 (watchma

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

2014-05-09 Thread Junio C Hamano
Felipe Contreras 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 immediately, you see that this m

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

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > Felipe Contreras writes: > > > Junio C Hamano wrote: > >> Felipe Contreras 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: [PATCH 00/32] Split index mode for very large indexes

2014-05-09 Thread Junio C Hamano
Duy Nguyen 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 pokes the

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

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: > Eric Wong writes: > > > Felipe Contreras 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 This has not

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

2014-05-09 Thread Philip Oakley
From: "Michael Haggerty" 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: cleanup inte

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: [PATCH v1 1/2] Remove 'git archimport'

2014-05-09 Thread Junio C Hamano
Eric Wong writes: > Felipe Contreras 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, foreign SCM tools should live out-of-tree a

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

2014-05-09 Thread Junio C Hamano
Jeff King 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 because he ca

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, bu

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

2014-05-09 Thread Junio C Hamano
Jeff King 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 without expl

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).

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

2014-05-09 Thread Junio C Hamano
Felipe Contreras writes: > Junio C Hamano wrote: >> Felipe Contreras 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 regression, isn't it? It is an old >> bre

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 > 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-htt

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 > >> -

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

2014-05-09 Thread Martin Langhoff
On Fri, May 9, 2014 at 11:57 AM, Felipe Contreras 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 *single* commit, I didn

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

2014-05-09 Thread Junio C Hamano
Jeff King 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 2012.

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

2014-05-09 Thread Michael Haggerty
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: cleanup interpret_branch_n

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

2014-05-09 Thread James Denholm
Felipe Contreras 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 with you if you don

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: n

[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 stateme

[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= > > > Specify a "Bcc:

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 unsubscr

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 indica

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: 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] Apac

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 thorou

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 ca

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= > > Specify a "Bcc:" value for each email. Default is the value of > > sendemail.bcc. > > > >

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, a

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 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 >>> Signed-off-by: Felipe Contreras >> >> Thank you for your input.

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 remainin

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 > wrote: > > > No activity since 2010, no tests. > > > > Cc: Tim Henigan > > Signed-off-by: Felipe Contreras > > --- > > contrib/diffall/README | 31 -- > > contrib/diffall/git-diffall | 257 > > ---

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 se

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 > > Signed-off-by: Felipe Contreras > > Thank you for your input. > > git-multimail is maintained outside of the Git

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 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

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 wrote: > No activity since 2010, no tests. > > Cc: Tim Henigan > Signed-off-by: Felipe Contreras > --- > contrib/diffall/README | 31 -- > contrib/diffall/git-diffall | 257 > > 2 files chan

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

2014-05-09 Thread Alexandre Julliard
David Kågedal writes: > 2014-05-09 10:29 GMT+02:00 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 canno

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

2014-05-09 Thread David Kastrup
Michael Haggerty 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 >> Signed-off-by: Felipe Contreras > > Thank you for your input. > > git-multimail is maintained outside of the Git proje

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 > Signed-off-by: Felipe Contreras Thank you for your input. git-multimail is maintained outside of the Git project and is only distributed along with Git

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 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 doe

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

2014-05-09 Thread Felipe Contreras
Alexandre Julliard wrote: > David Kågedal writes: > > > 2014-05-09 10:29 GMT+02:00 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? >

[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 defaul

Re: Conforming to pep8

2014-05-09 Thread Michael Haggerty
On 05/09/2014 04:09 AM, Jonathan Nieder wrote: > (cc-ing Pete Wyckoff who maintains git-p4 and Michael Haggerty > who maintains git-multimail) > William Giokas wrote: > >> - We follow PEP-8 (http://www.python.org/dev/peps/pep-0008/). >> >> It's even the first thing that you see when you go lo

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

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" 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 to ke

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 t

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 : > Martin Langhoff wrote: >> On Thu, May 8, 2014 at 8:58 PM, Felipe Contreras > > wrote: >> >> > Let us be honest, the vast majority of tools

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 wrote: >> How can I get Bash v4 for m

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 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 co

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 wrote: > Erik Faye-Lund wrote: >> On Fri, May 9, 2014 at 11:32 AM, Felipe Contreras >> 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 "

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

2014-05-09 Thread David Kastrup
Felipe Contreras writes: > David Kastrup wrote: >> Felipe Contreras 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 reco

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

2014-05-09 Thread Felipe Contreras
Matthieu Moy wrote: > Felipe Contreras 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 > would be to explain w

  1   2   >