RE: Git abuses qt4-ssh-askpass

2014-05-10 Thread Felipe Contreras
to access the remote repository (eg. my GitHub password) in the second question. This is a bug, and very misleading behavior. Same in seahorse (two password prompts). -- Felipe Contreras-- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord

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 out

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

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

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

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

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

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

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 19/25] contrib: remove 'diff-highlight'

2014-05-09 Thread Felipe Contreras
contrib/examples section), 3rdPartyTools/, Bridges/ (for cooperating with other VCS). That won't fix the issue that these tools are not maintained. A separate repository 'git-cruft' might do. -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git in the body of a message

[ANNOUNCE] git-remote-hg 0.2

2014-05-09 Thread Felipe Contreras
for failed imports If you use ArchLinux, you can use the package I wrote[1]. Enjoy :) https://github.com/felipec/git-remote-hg Daniel Liew (1): Use internal clone's hgrc Felipe Contreras (22): Reorganize tests Add README build: add install target doc: add manpage

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

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

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

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

[ANNOUNCE] git-remote-bzr 0.2

2014-05-09 Thread Felipe Contreras
active in Git v2.0 * Add support for older versions of bzr If you use ArchLinux, you can use the package I wrote[2]. Enjoy :) https://github.com/felipec/git-remote-bzr Felipe Contreras (10): Reorganize test Add README build: add install instructions doc: add manpage

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: What's cooking in git.git (Apr 2014, #08; Fri, 25)

2014-05-09 Thread Felipe Contreras
that this has been in the works for a long long tie and nobody has proposed a better name. Yes. One reason I think @{p} makes sense for publish is: % git push -u, @{u}, @{upstream} % git push -p, @{p}, @{publish} -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git

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

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

2014-05-09 Thread Felipe Contreras
/gmane.comp.version-control.git/248233 [2] http://article.gmane.org/gmane.comp.version-control.git/248242 -- Felipe Contreras -- 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

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 http://thread.gmane.org/gmane.comp.version

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

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

2014-05-09 Thread Felipe Contreras
to find out if a tool belongs in contrib or not is to as Junio. Cc: Junio C Hamano gits...@pobox.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/README | 43 --- 1 file changed, 43 deletions(-) delete mode 100644 contrib/README diff

[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 felipe.contre

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

[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 felipe.contre...@gmail.com --- contrib/convert-objects/convert-objects.c | 329 contrib/convert-objects/git-convert-objects.txt | 29 --- 2 files changed, 358 deletions(-) delete

[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 felipe.contre...@gmail.com --- contrib/workdir/git-new-workdir | 82 - 1 file changed, 82 deletions(-) delete mode 100755 contrib/workdir/git-new-workdir diff --git

[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 felipe.contre...@gmail.com --- contrib/stats/git-common-hash | 26 -- contrib/stats/mailmap.pl | 70 -- contrib/stats/packinfo.pl | 212

[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 felipe.contre...@gmail.com --- contrib/remotes2config.sh | 33 - 1 file changed, 33 deletions(-) delete mode 100755 contrib/remotes2config.sh diff --git a/contrib

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

2014-05-09 Thread Felipe Contreras
No activity, no nothing. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- 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

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

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

[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 cran...@google.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/persistent-https/LICENSE | 202 - contrib/persistent-https/Makefile

[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 sunsh...@sunshineco.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/contacts/git-contacts | 203

[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 felipe.contre...@gmail.com --- contrib/git-resurrect.sh | 182 --- 1 file changed, 182 deletions(-) delete mode 100755 contrib/git-resurrect.sh

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

2014-05-09 Thread Felipe Contreras
since the very beginning. -- Felipe Contreras -- 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: [PATCH v2 01/17] contrib: remove outdated README

2014-05-09 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com 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

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

2014-05-09 Thread Felipe Contreras
, nothing changed. The projects you are involved with will get by fine without you. git-remote-hg didn't get by so fine without me. -- Felipe Contreras -- 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

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

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-08 Thread Felipe Contreras
a hack. -- Felipe Contreras -- 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: [PATCH 0/4] remote-hg: more improvements

2014-05-08 Thread Felipe Contreras
that explains why it now uses LookupError (not ManifestLookupError), or do you want to reroll the original by squashing it? I don't want to do anything for a contrib tool. It's already broken in v2.0 anyway. -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git

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

2014-05-08 Thread Felipe Contreras
contrib tool, then I'll threat it as one. And unfortunately Junio would rather let an important part of Git die a slow death rather than admit he was wrong. Just watch him ignore the problem. -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git in the body of a message

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

2014-05-08 Thread Felipe Contreras
and they are going to move out of contrib and be distributed by default? If that's the case I'll resume the fixes because the current sitution is not good. -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org More

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

2014-05-08 Thread Felipe Contreras
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 regression, isn't it? It is an old breakage that existed

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-08 Thread Felipe Contreras
David Lang wrote: On Thu, 8 May 2014, Felipe Contreras wrote: If submodules were an integral part of Git that would be a possibility, but they are more like a hack. Well, if git.git can't use them, then how can anyone else be expected to. That is a very good question. I haven't been

[PATCH v1 00/25] contrib: cleanup

2014-05-08 Thread Felipe Contreras
apparently should live out-of-tree. After the cleanup, the only tools that remain are 'completion', 'credential' and 'subtree', which might eventually graduate. Felipe Contreras (25): Remove remote-helpers contrib: remove 'hg-to-git' contrib: remove 'stats' contrib: remove 'buildsystems

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

2014-05-08 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 v1 06/25] contrib: remove 'diffall'

2014-05-08 Thread Felipe Contreras
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 | 257 2 files changed, 288 deletions

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

2014-05-08 Thread Felipe Contreras
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 | 189 - contrib/buildsystems/Generators/Vcproj.pm | 626

[PATCH v1 03/25] contrib: remove 'stats'

2014-05-08 Thread Felipe Contreras
No real activity since 2012 (or ever), no tests, no documentation. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/stats/git-common-hash | 26 -- contrib/stats/mailmap.pl | 70 -- contrib/stats/packinfo.pl | 212

[PATCH v1 05/25] contrib: remove 'convert-objects'

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

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

2014-05-08 Thread Felipe Contreras
No activity, no tests. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/git-jump/README | 92 --- contrib/git-jump/git-jump | 69 --- 2 files changed, 161 deletions(-) delete mode 100644 contrib

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

2014-05-08 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 sunsh...@sunshineco.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/contacts/git-contacts | 203

[PATCH v1 22/25] contrib: remove 'fast-import'

2014-05-08 Thread Felipe Contreras
No tests, no documentation. No chance of ever graduating. Cc: Johannes Schindelin johannes.schinde...@gmx.de Cc: David Aguilar dav...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/fast-import/git-import.perl | 64 - contrib/fast-import/git

[PATCH v1 08/25] contrib: remove 'git-shell-commands'

2014-05-08 Thread Felipe Contreras
No activity, no tests. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/git-shell-commands/README | 18 -- contrib/git-shell-commands/help | 18 -- contrib/git-shell-commands/list | 10 -- 3 files changed, 46 deletions(-) delete

[PATCH v1 15/25] contrib: remove 'remotes2config'

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

[PATCH v1 10/25] contrib: reomve 'thunderbird-patch-inline'

2014-05-08 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 contrib

[PATCH v1 11/25] contrib: remove 'workdir'

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

[PATCH v1 16/25] contrib: remove 'persistent-https'

2014-05-08 Thread Felipe Contreras
No activity. No tests. No chance of ever moving into the core because it uses Go. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/persistent-https/LICENSE | 202 - contrib/persistent-https/Makefile | 38 --- contrib/persistent

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

2014-05-08 Thread Felipe Contreras
No activity since 2012, no tests, no chance of ever graduating. Cc: Jeff King p...@peff.net Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/diff-highlight/README | 152 - contrib/diff-highlight/diff-highlight | 173

[PATCH v1 09/25] contrib: remove 'gitview'

2014-05-08 Thread Felipe Contreras
No activity since 2007. Better out-of-tree tools out there. Cc: Aneesh Kumar K.V aneesh.ku...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/gitview/gitview | 1305 --- contrib/gitview/gitview.txt | 57 -- 2 files

[PATCH v1 24/25] contrib: remove 'hooks'

2014-05-08 Thread Felipe Contreras
No activity since 2008, no tests, no documentation. Cc: Gerrit Pape p...@smarden.org Cc: Shawn O. Pearce spea...@spearce.org Cc: Andy Parkins andypark...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/hooks/post-receive-email | 759

[PATCH v1 13/25] contrib: remove 'svn-fe'

2014-05-08 Thread Felipe Contreras
No activity since 2012, no tests. Cc: Jonathan Nieder jrnie...@gmail.com Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/svn-fe/.gitignore | 4 --- contrib/svn-fe/Makefile| 63 - contrib/svn-fe/svn-fe.c| 18

[PATCH v1 17/25] contrib: remove 'git-resurrect'

2014-05-08 Thread Felipe Contreras
No activity, no documentation, no tests, no chance of ever graduating. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/git-resurrect.sh | 182 --- 1 file changed, 182 deletions(-) delete mode 100755 contrib/git-resurrect.sh

[PATCH v1 12/25] contrib: remove 'vim'

2014-05-08 Thread Felipe Contreras
There's nothing there. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/vim/README | 22 -- 1 file changed, 22 deletions(-) delete mode 100644 contrib/vim/README diff --git a/contrib/vim/README b/contrib/vim/README deleted file mode 100644 index

[PATCH v1 14/25] contrib: remove 'rerere-train'

2014-05-08 Thread Felipe Contreras
No activity, no nothing. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- 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

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

2014-05-08 Thread Felipe Contreras
No updates since 2010, and no tests. Plus, foreign SCM tools should live out-of-tree anyway. Cc: Eric Wong normalper...@yhbt.net Cc: Martin Langhoff mar...@laptop.org Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- .gitignore |1 - Documentation/git

[PATCH v1 0/2] Remove foreign SCMs

2014-05-08 Thread Felipe Contreras
. Felipe Contreras (2): Remove 'git archimport' Remove 'git quiltimport' .gitignore|2 - Documentation/git-archimport.txt | 112 Documentation/git-quiltimport.txt | 54 -- Makefile |2 - command-list.txt |2

[PATCH v1 2/2] Remove 'git quiltimport'

2014-05-08 Thread Felipe Contreras
No updates since 2009 and no tests. Foreign SCM tools should live out-of-tree anyway. Cc: Johannes Schindelin johannes.schinde...@gmx.de Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- .gitignore| 1 - Documentation/git-quiltimport.txt | 54

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

2014-05-08 Thread Felipe Contreras
Jeff King wrote: 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. I use

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

2014-05-08 Thread Felipe Contreras
Martin Langhoff wrote: On Thu, May 8, 2014 at 8:58 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Let us be honest, the vast majority of tools in 'contrib/' have no chance of ever graduating, so let's remove them. I am curious -- have you checked what parts of contrib

RE: Conforming to pep8

2014-05-08 Thread Felipe Contreras
pep8 in git-remote-{hg,bzr}, but only to some extent. I do this: [pep8] ignore = E401,E302,E201,E202,E203,E126,E128 max-line-length = 160 That said there's a couple of issues present that I didn't notice. Thanks for checking. -- Felipe Contreras -- To unsubscribe from this list: send

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

2014-05-08 Thread Felipe Contreras
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 have commented off-list to me that they use it, too. Add tests

[PATCH] fixup! completion: graduate out of contrib

2014-05-08 Thread Felipe Contreras
Actually we need a symlink for gitk. At least ArchLinux doesn't do that, so we would be fixing at least one bug. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- Makefile | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Makefile b/Makefile index 2690903..a3d27d8 100644

Re: Conforming to pep8

2014-05-08 Thread Felipe Contreras
William Giokas wrote: On Thu, May 08, 2014 at 09:10:25PM -0500, Felipe Contreras wrote: William Giokas wrote: Which is a whole bunch of errors and warnings thrown by pep8. Is pep8 just getting put by the wayside? I would much rather have these scripts conform to that and have an actual

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-07 Thread Felipe Contreras
is wrong. Moreover, if you are cross-compiling you won't be able to run the target's `git` binary. If anything, it should be `pkg-config --variable=exec-path git`. -- Felipe Contreras -- To unsubscribe from this list: send the line unsubscribe git in the body of a message to majord...@vger.kernel.org

[PATCH] Standardize python shebangs

2014-05-07 Thread Felipe Contreras
It's better if all our scripts use the same '/usr/bin/env python'. Signed-off-by: Felipe Contreras felipe.contre...@gmail.com --- contrib/hooks/multimail/README | 6 +++--- contrib/hooks/multimail/git_multimail.py| 2 +- contrib/hooks/multimail/migrate-mailhook-config

Re: [PATCH] completion: move out of contrib

2014-05-07 Thread Felipe Contreras
Felipe Contreras wrote: Yes, *if* they have been packaging them, they have a way. But what if they haven't been doing so? And for the ones that have a way, now they need one hack less. As an example of all the hacks needed by a real distribution package, here's the stuff ArchLinux packagers

Re: [PATCH] Standardize python shebangs

2014-05-07 Thread Felipe Contreras
Johan Herland wrote: On Wed, May 7, 2014 at 12:03 PM, Felipe Contreras felipe.contre...@gmail.com wrote: It's better if all our scripts use the same '/usr/bin/env python'. Only if they are source compatible with both Python2 and Python3. See PEP394 URL: http://legacy.python.org/dev/peps

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

2014-05-07 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: Here's a bunch of tests more, and a fixes for Mercurial v3.0. I think the discussion with John Keeping hints that we shouldn't be rushing fc/remote-helpers-hg-bzr-graduation Really? Based on what reasoning? I have

Re: [PATCH] Standardize python shebangs

2014-05-07 Thread Felipe Contreras
Matthieu Moy wrote: Felipe Contreras felipe.contre...@gmail.com writes: It's better if all our scripts use the same '/usr/bin/env python'. Why? Using python2 for git_multimail.py is a deliberate decision: If you want to use python2, then use '/usr/bin/env python2'. The git-multimail

Re: [PATCH] completion: move out of contrib

2014-05-07 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: As an example of all the hacks needed by a real distribution package, here's the stuff ArchLinux packagers have to do: # bash completion mkdir -p $pkgdir/usr/share/bash-completion/completions/ install

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-07 Thread Felipe Contreras
to match $MAN_PATH and $PATH. Using `git --html-path` for that is wrong. -- Felipe Contreras -- 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: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-07 Thread Felipe Contreras
, that starts to be a real burden. It doesn't *need* them to build. The Mercurial/Bazaar dependencies are optional, both at build-time and at run-time. Most distributions would want to test the functionality they are distributing, and for testing they do need these dependencies. -- Felipe Contreras

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-07 Thread Felipe Contreras
/332412470c6e084f10ac2f8dc11e86ab4680974a -- Felipe Contreras -- 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: [PATCH 0/4] remote-hg: more improvements

2014-05-07 Thread Felipe Contreras
Junio C Hamano wrote: Felipe Contreras felipe.contre...@gmail.com writes: Really? Based on what reasoning? I have proven his reasoning to be basically wrong. Perhaps s/proven/convinced myself only/; you didn't prove it to me and I doubt you proved it to John. And you are still

Re: [PATCH] Standardize python shebangs

2014-05-07 Thread Felipe Contreras
Matthieu Moy wrote: Felipe Contreras felipe.contre...@gmail.com writes: If you want to use python2, then use '/usr/bin/env python2'. Err, yes, this is what the code does before your patch. Not for all the instances. If you are going to follow practices different than git.git

Re: What's cooking in git.git (Apr 2014, #09; Tue, 29)

2014-05-07 Thread Felipe Contreras
John Keeping wrote: On Wed, May 07, 2014 at 03:26:15PM -0500, Felipe Contreras wrote: Junio C Hamano wrote: Your git-integrate might turn into something I could augment my workflow with with some additions. - specifying a merge strategy per branch being merged; git

<    1   2   3   4   5   6   7   8   9   10   >