Re: [RFC/PATCH] Add interpret-trailers builtin

2013-11-05 Thread Christian Couder
From: Junio C Hamano > Christian Couder writes: >> >> * "trailer" seems better than "commitTrailer" as the config key because >> it looks like all the config keys are lower case and "committrailer" is not >> very readable. > > And closes the door for other things from later acquiring trailers?

Re: Inquiry

2013-11-05 Thread Max Baroi
You probably don't want to store your favorite movies or other large media files with out-of-the-box git. On Tue, Nov 5, 2013 at 9:00 PM, Ilya Silvestrov wrote: > > No, git only works for C, because they use git to store Linux kernel sources > :) > > Seriously, it's about storing any files incl

Re: Inquiry

2013-11-05 Thread Ilya Silvestrov
No, git only works for C, because they use git to store Linux kernel sources :) Seriously, it's about storing any files including your favorite movies. On 11/05/2013 07:58 PM, Altaf Hussain Sayyed wrote: Hi, I have following query. Can git repository can be used for revision control of all ki

Inquiry

2013-11-05 Thread Altaf Hussain Sayyed
Hi, I have following query. Can git repository can be used for revision control of all kind of programming language coding including but not limited to followings: ASP.NET Java iOS PHP Regards! -- Best Wishes, ALTAF-HUSSAIN SAYYED Founder and CEO Nimetler Technologies ,Mumbai Branch =

Re: [PATCH 2/2] git_connect: factor out discovery of the protocol and its parts

2013-11-05 Thread Johannes Sixt
Am 05.11.2013 21:45, schrieb Torsten Bögershausen: > On 2013-11-05 20.39, Johannes Sixt wrote: > Thanks for picking this up, please see some minor nits inline, > and git_connect() is at the end > >> -struct child_process *git_connect(int fd[2], const char *url_orig, >> -

[PATCH v3 1/2] builtin/remote: remove postfixcmp() and use suffixcmp() instead

2013-11-05 Thread Christian Couder
Commit 8cc5b290 (git merge -X, 25 Nov 2009) introduced suffixcmp() with nearly the same implementation as postfixcmp() that already existed since commit 211c8968 (Make git-remote a builtin, 29 Feb 2008). The only difference between the two implementations is that, when the string is smaller than t

[PATCH v3 2/2] Rename suffixcmp() to has_suffix() and invert its result

2013-11-05 Thread Christian Couder
Now has_suffix() returns 1 when the suffix is present and 0 otherwise. The old name followed the pattern anything-cmp(), which suggests a general comparison function suitable for e.g. sorting objects. But this was not the case for suffixcmp(). Signed-off-by: Christian Couder --- builtin/clone.c

Re: [PATCH 2/2] git_connect: factor out discovery of the protocol and its parts

2013-11-05 Thread Torsten Bögershausen
On 2013-11-05 20.39, Johannes Sixt wrote: Thanks for picking this up, please see some minor nits inline, and git_connect() is at the end > -struct child_process *git_connect(int fd[2], const char *url_orig, > - const char *prog, int flags) > +static enum protocol pars

[PATCH] git-cat-file: fix output when format string contains no variables

2013-11-05 Thread Sven Brauch
>From 2e7b5aed771faeff654a447346bb0b57570d9569 Mon Sep 17 00:00:00 2001 From: Sven Brauch Date: Tue, 5 Nov 2013 20:06:21 +0100 Subject: [PATCH] git-cat-file: fix output when format string contains no variables When the format string for git-cat-object --batch-check contained no variables, the fu

[PATCH 2/2] git_connect: factor out discovery of the protocol and its parts

2013-11-05 Thread Johannes Sixt
git_connect has grown large due to the many different protocols syntaxes that are supported. Move the part of the function that parses the URL to connect to into a separate function for readability. Signed-off-by: Johannes Sixt --- Apart from this "simplification", the protocol parsing code is a

[PATCH 1/2] git_connect: remove artificial limit of a remote command

2013-11-05 Thread Johannes Sixt
Since day one, function git_connect() had a limit on the command line of the command that is invoked to make a connection. 7a33bcbe converted the code that constructs the command to strbuf. This would have been the right time to remove the limit, but it did not happen. Remove it now. git_connect()

Re: Splitting files merge with branches

2013-11-05 Thread Junio C Hamano
Javier Domingo writes: > I have been using git for now 4 years, and one feature I miss a lot, > that would increase the usability of git in many cases, would be > having it detect "inter-file" movements, so that if I, in a single > commit just part one file into many, git can track that change. >

Re: [PATCH v2] Rename suffixcmp() to has_suffix() and inverse its result

2013-11-05 Thread Junio C Hamano
Max Horn writes: > +1 for the change. I find the resulting code easier to understand, too. > ... > > Taking one step back, shouldn't the commit message rather explain > the new status, instead of referring so much to the past? If I > imagine somebody reading this in a year, they might not even kn

Re: [PATCH] Add the commit.gpgsign option to sign all commits

2013-11-05 Thread Junio C Hamano
Nicolas Vigier writes: > If the problem is users having to type their passphrase to sign each > commit, we can suggest using an agent in the option description: Yeah, that is probably a good idea. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to major

Re: [RFC/PATCH] Add interpret-trailers builtin

2013-11-05 Thread Junio C Hamano
Johan Herland writes: > On Mon, Nov 4, 2013 at 8:12 PM, Junio C Hamano wrote: >> Johan Herland writes: +{ + char *end = strchr(arg, '='); + if (!end) + end = strchr(arg, ':'); >>> >>> So both '=' (preferred) and ':' are accepted as field/value >>> s

Re: [PATCH v2] Rename suffixcmp() to has_suffix() and inverse its result

2013-11-05 Thread Max Horn
+1 for the change. I find the resulting code easier to understand, too. On 05.11.2013, at 05:57, Christian Couder wrote: > As suffixcmp() should not be used as an ordering comparison function, > and anything-cmp() ought to be usable as an ordering comparison function, > suffixcmp() should be re

git svn clone with funky tags layout

2013-11-05 Thread Jim Garrison
I'm doing a one-time migration of an svn project. For historical reasons our repo layout is weird: trunk/reporting/reporting_app tags/something_else tags/reporting_app-2.3.45 tags/reporting_app-2.4.46 tags/reporting_app-2.4.0 tags/reporting_app-2.4.1

Bug with git svn fetch? "error:too many matches for svn-remote.svn.added-placeholder"

2013-11-05 Thread Jess Hottenstein
Hello, I'm running a git svn fetch of a large svn repo and it is grinding to a halt with thousands of "error: too many matches for svn-remote.svn.added-placeholder". My .git/config has grown to ~5 lines of added-placeholders. I also had to up my linux file limit because I have ~2500 open fil

Re: rcs

2013-11-05 Thread Jakub Narębski
Finnerty, James M Mr CTR USA USASOC-SOAR wrote Jeff King [mailto:p...@peff.net]wrote: On Tue, Oct 29, 2013 at 11:35:21AM -0500, Finnerty, James M Mr CTR USA USASOC-SOAR wrote: Hi. I'm going to attempt to import a git database into Razor which is linux rcs based. Does the linux version of git

Re: [PATCH] Add the commit.gpgsign option to sign all commits

2013-11-05 Thread Nicolas Vigier
On Mon, 04 Nov 2013, Junio C Hamano wrote: > Nicolas Vigier writes: > > > If you want to GPG sign all your commits, you have to add the -S option > > all the time. The commit.gpgsign config option allows to sign all > > commits automatically. > > I'm somewhat horrified to imagine the end-user e

Re: [PATCH v3] push: Enhance unspecified push default warning

2013-11-05 Thread Matthieu Moy
- Original Message - > Greg Jacobson writes: > > > Is there anything I could do to improve this patch? Thank you. > > My vague recollection is that we started from an excerpt from the > documentation page, not unlike this patch attempts to, but because > such an excerpt has to be less c

Re: [PATCH v3] push: Enhance unspecified push default warning

2013-11-05 Thread Matthieu Moy
- Original Message - > Greg Jacobson writes: > > > Is there anything I could do to improve this patch? Thank you. > > My vague recollection is that we started from an excerpt from the > documentation page, not unlike this patch attempts to, but because > such an excerpt has to be less c

Splitting files merge with branches

2013-11-05 Thread Javier Domingo
Hi, I have been using git for now 4 years, and one feature I miss a lot, that would increase the usability of git in many cases, would be having it detect "inter-file" movements, so that if I, in a single commit just part one file into many, git can track that change. I suppose this is quite diff